Vistaスタイルのファイルダイアログ使用時に名前を付けて保存の文字コードセット, 改行コード, BOM指定が有効に動作しない問題を修正

[beru]

#873 で報告した問題を解消する修正を行いました。

[beru]

memo : ここの指定は実は不要な事が分かった。pSaveInfo->cEol の値が EOL_NONE で、m_cEol の型 CEol のデフォルトコンストラクタでも EOL_NONE が設定されるので。でも明示的に設定しておいたほうが気分的に安心する。

[berryzplus]

#874 (comment)
指摘1件の対応をお願いしたいです。
他は問題なさげに見えます。

[berryzplus]

対応ありがとうございます。LGTMです。

[ds14050]

初見でもうマージまで済んでいました。関係ありそうなログです。参考に。

• ファイル保存時の改行コード統一は仕様？ | サクラエディタ過去ログ
• 改行コードの変更が反映されない | サクラエディタ過去ログ

[beru]

初見でもうマージまで済んでいました。関係ありそうなログです。参考に。

• ファイル保存時の改行コード統一は仕様？ | サクラエディタ過去ログ
• 改行コードの変更が反映されない | サクラエディタ過去ログ

少しログを見てみましたが、多分本件には直接関係していない話題だと思います。

[berryzplus]

•ファイル保存時の改行コード統一は仕様？ | サクラエディタ過去ログ

「これ俺が質問したときのログやん！」と昔のハンドルをカミングアウトしてみたり。
今回修正の内容は、単にダイアログから内部データへのコピーが行われていない、という話なので、
内部的に「行末の改行を変更する関数」を常に呼んでいる理由が分かった話とは関係なさそうです。

•改行コードの変更が反映されない | サクラエディタ過去ログ

ディープですね。（こっちの話も本件とは関係なさそうです。
ぼくはkobakeさんと近い立ち位置の人間ですが、現役プログラマでもあります。
双方の言い分がわかるので読んでいてかなり「イラ☆イラ」としました。
いま、ちょうどこのあたりの実装見直しを検討しています。

現状で、ビューコマンダーがアンドゥさんを指揮しているのを、
ドックラインマネージャーが指揮するように変えたらどうかな？的なことを考えてます。
この辺は複雑すぎるのでスキル関係なく単純なレビューは不可能だと思います。
ポンチ絵描いて仕組みを解説しつつ、ブランチ切ってそこで開発進めていく感じかなぁ。

というか、かなり強引に単体テストの導入を進めてきたのは、
現実的にレビュー不可能レベルの大規模改修を、
安心して受け入れられるようにするための方便になるんじゃないかと期待するからです。

テストなしで「えいやっ！」って受け入れするとやっぱり不具合が出ますし、要件に対して機能を満たすかの確認を細かく重ねていくことが、仕事じゃなくてもやっぱり大事なのかな、と思っています。