[スレッド全体]

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点とも問題点を当方でも確認。修正したパッチをアップしました。
> #念のためパッチも上げてありますが、なすこじさん手元のほうの修正でいいような気がします。


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


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