[▲前のスレッド]
▼ 2010/10/1 (金) 20:54:10 なすこじ 　
Mozilla/5.0 (Windows; U; Windows NT 5.1; ja; rv:1.9.2.10) Gecko/20100914 Firefox/3.6.10
[5685] パッチを色々確認しました
こんにちは、やっとXP SP3に移行したのでVS2008/VS2010がインストールできるようになりました (^^;

以下のANSI版パッチを確認しました。
　#2835545 Erlang アウトライン解析
　#2992320 存在しないcursor1.curのsakura.dsp登録を解除
　#3008991 Viewウィンドウの遅延作成(ANSI版)
　#3009187 [U済] アウトライン解析（トグル）で「C 関数一覧」画面が閉じない
　#3025944 [U済]折り返し行インデントで折り返すとメモリを食い尽くす
　#3026401 [U済]ツールバー折返しの番号ずれ対応
　#3055719 [U済]Undo/Redoでのメモリーリーク修正
　#3055721 [U済]矩形選択中の長い文字列の貼り付けでクラッシュ

−−　ここまで確認済み　−−

★未確認
　#3056634 [U済]チップウィンドウのリソースリーク修正
　#3056638 Grep不具合の修正
　#3067444 [U済] 「クリップボードから」と「正規表現」がONの置換で「選択始点挿入」「選択終点追加」が効かない
　#3069797 [U済] MSDEVLineSelect形式で「クリップボードから」全置換したときにラインモード貼り付けにならない

★多分破棄（#3069797により該当部分が無くなるため）
　#3055721 [U済]矩形選択中の長い文字列の貼り付けでクラッシュ

特に問題が出なければ10/11以降に代理コミットします。
▼ 2010/10/2 (土) 17:13:17 もか 　
INCM1.23c
[5686] Re: パッチを色々確認しました
>★多分破棄（#3069797により該当部分が無くなるため）
>　#3055721 [U済]矩形選択中の長い文字列の貼り付けでクラッシュ
破棄(Rejected)に変更しました。

#3055711 ユーザの意図しないプログラム実行の抑制(A用)
も候補に入れていただきたく。
意図的にPrivateに設定してありましたが、A/Wとも全く反応がなく、私以外には見えていないのでしょうか？
見えないと困るのでPublicにしました。

# そろそろ 1.6.6.0 のリリース時期かなぁと
▼ 2010/10/3 (日) 00:27:16 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5687] Re2: パッチを色々確認しました
▼ もかさん
> #3055711 ユーザの意図しないプログラム実行の抑制(A用)
> も候補に入れていただきたく。
> 意図的にPrivateに設定してありましたが、A/Wとも全く反応がなく、私以外には見えていないのでしょうか？
> 見えないと困るのでPublicにしました。

見えていませんでしたので、Privateが原因だと思います。

「#3056638 Grep不具合の修正」について、以下の現象が確認できません。
・Grep結果画面の強調表示に単語単位が適用されていない
・Grepの出力順が不正になることがある

パッチの74〜88行目の理由が分かりません。
この時点ではcmemMessageは空な気が……

ではでは。
▼ 2010/10/3 (日) 01:27:28 もか 　
INCM1.23c
[5689] Re3: パッチを色々確認しました
いろいろありがとうございます。
>見えていませんでしたので、Privateが原因だと思います。
プロジェクトメンバーの名前が濃い人(Admin)の人は見えてると思っていたけれど、
Tracker ManagerのAdmin+投稿者でないと見えないということみたいですね。
それだと私とgentaさんとyazakiさんだけに見えると。

>「#3056638 Grep不具合の修正」について、以下の現象が確認できません。
>・Grep結果画面の強調表示に単語単位が適用されていない
行出力にして、単語単位検索でHogeをGrepしたとき
出力の同一行に Hoge と HogeHoge があると HogeHoge にも色がついてしまいます。

>・Grepの出力順が不正になることがある
>パッチの74〜88行目の理由が分かりません。
>この時点ではcmemMessageは空な気が……
移植ミスで場所を間違えたようです。すみません。確認してませんでした。
CEditView::DoGrepTree:7957行目付近
>        /*
>         * サブフォルダを検索する。
>         */
>        if( bGrepSubFolder ){
の直前が正しい位置でした。
この位置で出力しないと、同一フォルダ内の出力がサブフォルダで分断されます。
移動しただけですが変更後のパッチをあげました。
▼ 2010/10/4 (月) 03:09:18 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5690] Re4: パッチを色々確認しました
▼ もかさん
あああ、すみません m(_ _)m
SourceForgeにログインせずに見ていました。

> >「#3056638 Grep不具合の修正」について、以下の現象が確認できません。
> >・Grep結果画面の強調表示に単語単位が適用されていない
> 行出力にして、単語単位検索でHogeをGrepしたとき
> 出力の同一行に Hoge と HogeHoge があると HogeHoge にも色がついてしまいます。

A版r1825やv1.6.4.0、U版r1432などでも試してみましたが、どうにも現象が出ません ＞＜
m_bCurSrchWordOnlyにアクセスする所が少ないので、おっしゃる様な現象は出ない気がするのですが……

XP SP3/VC2008 Express Editionでビルドしています。

・Grepの出力順が不正になることがある
現象を確認できました。
また、修正箇所を移動により修正後の動作を確認できました。

▼ 2010/10/4 (月) 20:08:39 もか 　
INCM1.23c
[5692] Re5: パッチを色々確認しました
お手数おかけします。Grep結果だけ見て不具合の有無のほう確認してませんでした。
なすこじさんの指摘通りで、そのままでは発生しませんでした。

Unicode版では rev1796 >>unicode:1334 検索文字列の色分けで単語単位のフラグがビューごとになっていない
でのビューの表示側を修正した影響で現象が発生するようになりました。
　調べてみたところ、色分けの描画の判定関数
ANSIのCEditView::IsSearchString()内
trunk/sakura_core/CEDitView_New2.cpp:590
>        if( m_pShareData->m_Common.m_bWordOnly ){        /* 検索／置換  1==単語のみ検索 */
になっていて、ANSI側は不具合のあるままです。
m_bCurSrchWordOnlyに変更するとANSIでもGrepの表示不具合を確認しました。

なおANSIのHEADとrev1796より前のUnicodeでは以下のような感じになります。
WindowA: ファイルを開いて「単語単位」でGrep検索
WindowB: 新規Grepウィンドウ。Grep完了。色分けできている(設定=ShareData, 表示=ShareDataだから)
WindowA: 普通に単語チェックをOFFで検索or検索マーク切り替え(で今の実装では単語チェックが外れる)
WindowB: 再作画された部分だけ単語単位色分けが無効になる
もちろんGrep以外の2つの編集ウィンドウ/ビューでも同じです。
▼ 2010/10/5 (火) 01:05:47 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5693] Re6: パッチを色々確認しました
▼ もかさん
Grepの件了解です。また確認しておきます。

その他の問題点
#3008991 Viewウィンドウの遅延作成(ANSI版)VC6でビルドエラー
CreateEditViewBySplit()内にfor(int i=0;…が2個あるのでVC6では2重定義となります。
int i; を外に出して下さい。

#3055711 ユーザの意図しないプログラム実行の抑制(A用)VC2005で起動エラー
StartControlProcess()でCreateProcess()実行時にメモリ参照エラーとなります。
エラーになる理屈は分からないけど、下記の所がおかしい気がします (^^;
パッチ90行目
　　if( !szExeDir[0] ){
　→if( szExeDir[0] ){

パッチ246〜249行目
　Init()は元のままで良い

ではでは。

▼ 2010/10/5 (火) 23:10:28 もか 　
INCM1.23c
[5694] Re7: パッチを色々確認しました
3点とも問題点を当方でも確認。修正したパッチをアップしました。
#念のためパッチも上げてありますが、なすこじさん手元のほうの修正でいいような気がします。

>#3055711 ユーザの意図しないプログラム実行の抑制(A用)VC2005で起動エラー
>StartControlProcess()でCreateProcess()実行時にメモリ参照エラーとなります。
>(中略)
>パッチ90行目
>　　if( !szExeDir[0] ){
>　→if( szExeDir[0] ){
意図と反対ですね。
こちらの実害は少ないけれど0ではないです。

>パッチ246〜249行目
>　Init()は元のままで良い
こっちがビンゴで、メンバ関数呼び出しだったところをpatchで同名のAPIと間違えて置き換えてます。
その影響でメンバ変数が初期化されなくて落ちているようです。

#U版ではInitの部分は実装が違うので問題ないです。
▼ 2010/10/7 (木) 00:34:55 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5695] Re8: パッチを色々確認しました
▼ もかさん
> 3点とも問題点を当方でも確認。修正したパッチをアップしました。
> #念のためパッチも上げてありますが、なすこじさん手元のほうの修正でいいような気がします。

ありがとうございます。確認しました。

▼ 2010/10/3 (日) 00:51:26 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5688] Re:パッチを色々確認しました
「#2992320 存在しないcursor1.curのsakura.dsp登録を解除」については、パッチに問題が無いので確認済みとしていますが、私のVC2010 Express Editionでは>>dev:5673 の現象が出せません。

dswを直接VC2010で開くと「…dspを読み込めません。…」となって変換に失敗してしまうので……

▼ 2010/10/4 (月) 03:10:03 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5691] Re:パッチを色々確認しました
以下のパッチを確認しました。
　#3056634 [U済]チップウィンドウのリソースリーク修正　#3067444 [U済] 「クリップボードから」と「正規表現」がONの置換で「選択始点挿入」「選択終点追加」が効かない　#3069797 [U済] MSDEVLineSelect形式で「クリップボードから」全置換したときにラインモード貼り付けにならない

★一部現象未確認のため保留
　#3056638 Grep不具合の修正
★未確認
　#3055711 ユーザの意図しないプログラム実行の抑制(A用)

▼ 2010/10/7 (木) 00:35:39 なすこじ 　
Mozilla/5.0 Opera/9.5 (KDDI-SH3H; BREW; Opera Mobi; U; ja) Presto/2.4.15 Version/10.00
[5696] Re:パッチを色々確認しました
以下のパッチを確認しました。
　#3056638 Grep不具合の修正　#3055711 ユーザの意図しないプログラム実行の抑制(A用)

確認が終わりましたので、予定通り10/11以降にコミットします。

▼ 2010/10/11 (月) 17:33:07 なすこじ 　
Mozilla/5.0 (Windows; U; Windows NT 5.1; ja; rv:1.9.2.10) Gecko/20100914 Firefox/3.6.10
[5697] コミット完了しました
以下のANSI版パッチをコミットしました。
　#2835545 Erlang アウトライン解析
　#2992320 存在しないcursor1.curのsakura.dsp登録を解除
　#3008991 Viewウィンドウの遅延作成(ANSI版)
　#3009187 [U済] アウトライン解析（トグル）で「C 関数一覧」画面が閉じない
　#3025944 [U済]折り返し行インデントで折り返すとメモリを食い尽くす
　#3026401 [U済]ツールバー折返しの番号ずれ対応
　#3055719 [U済]Undo/Redoでのメモリーリーク修正
　#3056634 [U済]チップウィンドウのリソースリーク修正
　#3056638 Grep不具合の修正
　#3067444 [U済] 「クリップボードから」と「正規表現」がONの置換で「選択始点挿入」「選択終点追加」が効かない
　#3069797 [U済] MSDEVLineSelect形式で「クリップボードから」全置換したときにラインモード貼り付けにならない
　#3055711 ユーザの意図しないプログラム実行の抑制(A用)

以上
[▼次のスレッド]
INCM/CMT
Cyclamen v3.81
[ut:0.020][st:0.010]