Vistaスタイルのファイルダイアログでカスタマイズ部分が表示されない場合がある問題の修正

[beru]

#844 で報告した問題に対する対処です。

メンバ変数の初期化が漏れていて不定値のままになっていたのが原因でした。

[berryzplus]

初期化漏れは了解です。
対処は入れる方向でいきたいです。

気になっているのは、これで全部？ということです。

問題の構造体 CustomizeSetting にはメンバが6つあります。
今回の修正箇所は、そのうちの3つだけを初期化していたコードで「1つだけ漏れてました:smile:」という修正を行うわけです。「6個中3個を初期化」⇒「6個中4個を初期化」です。たぶん大丈夫だとは思うんですが、あと2個は設定しなくていいんですよね？が気になります。

初期化漏れ発覚時の対処は、「意味がなくても全対処」が基本なのでどうなんだろ？と思っております。bShowReadOnlyを右クリックで Find All Reference するとすべての利用箇所にアクセスできます。（これ、知ってると思いますが一応。）

というわけで、たぶん問題ないんだろーなーと思いつつ、チェックの前に見解が聞きたいです。

[beru]

気になっているのは、これで全部？ということです。

問題の構造体 CustomizeSetting にはメンバが6つあります。
今回の修正箇所は、そのうちの3つだけを初期化していたコードで「1つだけ漏れてました😄」という修正を行うわけです。「6個中3個を初期化」⇒「6個中4個を初期化」です。たぶん大丈夫だとは思うんですが、あと2個は設定しなくていいんですよね？が気になります。

調べてみました。あと2個は CDlgOpenFile_CommonItemDialog::DoModalOpenDlgImpl1 で false に初期化されているので問題無さそうです。

初期化漏れ発覚時の対処は、「意味がなくても全対処」が基本なのでどうなんだろ？と思っております。bShowReadOnlyを右クリックで Find All Reference するとすべての利用箇所にアクセスできます。（これ、知ってると思いますが一応。）

というわけで、たぶん問題ないんだろーなーと思いつつ、チェックの前に見解が聞きたいです。

おそらく問題無いんじゃないかなと思います。

[berryzplus]

気になっているのは、これで全部？ということです。

問題の構造体 CustomizeSetting にはメンバが6つあります。
今回の修正箇所は、そのうちの3つだけを初期化していたコードで「1つだけ漏れてました😄」という修正を行うわけです。「6個中3個を初期化」⇒「6個中4個を初期化」です。たぶん大丈夫だとは思うんですが、あと2個は設定しなくていいんですよね？が気になります。

調べてみました。あと2個は CDlgOpenFile_CommonItemDialog::DoModalOpenDlgImpl1 で false に初期化されているので問題無さそうです。

確認ありがとうございます。
よく見てみると残り2つは 保存時専用 っぽいですね。

• bUseEol 保存時の改行コード選択UIを表示するかのフラグ
• bUseBom 保存時のBOM選択UIを表示するかのフラグ

CDlgOpenFile_CommonItemDialog::DoModalOpenDlgImpl1 で一律falseになってるので問題なし、で良さそうに思いました。

おそらく問題無いんじゃないかなと思います。

まぁ、なんかあったらその時対応するでいいと考えています。
いまやれる範囲で明らかな見落としがないか一応はちゃんとみたいなと思って確認してただけです。

[berryzplus]

LGTMです。

[beru]

確認ありがとうございます。それでは Merge します。
もし問題が見つかったら別PRで対応します。