文字コードの指定ダイアログのCPチェックの仕様を修正する

[berryzplus]

PR の目的

文字コードの指定ダイアログのCPチェックの仕様を修正することにより、
issue #1037 の不具合(CPチェックが効かないことがある)を解消します。

カテゴリ

• 仕様変更
• 不具合修正

PR の背景

issue #1037 の対策PRです。

対応する不具合内容

• 文字コードの指定ダイアログのCPチェックが効かないことがある

不具合の原因

1. CPチェックの状態を格納するメンバ変数の初期化が漏れている
2. IDC_CHECK_CP を利用する他4つのダイアログと実装が異なっている

#1037 (comment) でコメントしている通り、不具合を解消するだけならずっと少ない修正で事足ります。

この PR では、不具合原因のうち 2. の「他4つのダイアログと実装が違う」に着目し、実装を合わせてみました。
CPチェックの状態を格納するメンバ変数を削除し、IDC_CHECK_CP のイベント実装をシンプルにしました。

PR のメリット

issue #1037 の不具合が解消します。

PR のデメリット (トレードオフとかあれば)

とくにありません。

なお、Windows API CheckDlgButton の使い方が誤っている件についてはスルーします。
※CheckDlgButton関数の第3引数はBOOLではなくintです。TRUEで動作している理由は、BST_CHECKEDの値が1でTRUEの定義とたまたま一致しているためです。

PR の影響範囲

• アプリ(=サクラエディタ)の機能に影響があります。
  • issue 文字コードの指定ダイアログでCPのチェックが効かない事が有る #1037 の不具合が解消します。

関連チケット

close #1037 文字コードの指定ダイアログでCPのチェックが効かない事が有る

参考資料

[AppVeyorBot]

✅ Build sakura 1.0.2278 completed (commit f39b53ddc4 by @berryzplus)

[beru]

問題無いと思います。

動作させて #1037 で報告した問題が解消されたことを確認しました。

[berryzplus]

レビューありがとうございます。
何か問題が見つかったら別PRにて対応しまっす。