CDlgInput1で文字列長はNUL終端を含まない長さを指定するはずが1文字多いところがあるのを修正

[usagisita]

PR の目的

タイトルの通り、1行入力ダイアログのいくつかの画面に、1文字分長い文字列制限値を指定しているため、入力画面で限界まで文字を入れると、落ちたり落ちなかったり、動作不良だったり、ちょっと不健康です。
このPRでは、不具合として修正します。

カテゴリ

• 不具合修正

PR の背景

異常終了、不安定な動作はなるべく直したいです。

PR のメリット

バグが減ります。

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

特にありません。

仕様・動作説明

履歴ダイアログの「検索」などの文字列はダブルクリックか右クリックで「編集」できます。
同、左下の「追加」や右メニューの「新規追加」でも入力画面が出ます。
同じような感じで、プロファイルマネージャの新規作成、名前変更があります。
ファイルツリー設定画面の「パスの一括置換」も同上です。
印刷プレビューのページ番号の「...」の直接指定のダイアログもです。
以上が、1文字多くなっていたところです。

なおCDlgPrintSettingのCDlgInput1の変数は未使用だったので削除しました。

PR の影響範囲

上記、ダイアログの入力可能文字数が1文字減ります。

テスト内容

手順
元のコードベースで実行し、履歴ダイアログで新規追加はMAX文字入力すると、履歴に追加されません。
PRのコード修正を適用すると、MAX文字(前より1文字少ない)でも追加されます。

テスト1

結合テスト的なものなので、単体テストは書くのが難しいため、ありません。
サクラエディタを実行し、各ダイアログ上を操作して、正常動作するか確認します。

関連 issue, PR

参考資料

[sonarcloud]

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

0.0% Coverage
0.0% Duplication

[AppVeyorBot]

✅ Build sakura 1.0.3554 completed (commit c4a83b1ee7 by @usagisita)

[berryzplus]

発生している事象への対処として適切だと思います。

本質的には CDlgInput1 が提供するインターフェースが分かりづらいことが元凶だと思ったので、別件としていつか対策したいです。

[sanomari]

PR内容は問題ないと思います。

本質的には CDlgInput1 が提供するインターフェースが分かりづらいことが元凶だと思ったので、別件としていつか対策したいです。

前に作成して放置中のコードなんですが、こんなの作りました。
https://github.com/sanomari/sakura-editor/blob/0b3583fbc8cd245584bf287c72e5f07995bf729b/sakura_core/dlg/CDlgProfileMgr.cpp#L55-L66

[usagisita]

リファクタリングをともわない、どんなバグかわかる最低限度の修正ということで、ここはひとつ。

sanomari さんのコードも見ました。ダイアログの処理と、プロファイルマネージャの機能の分離、いいと思います。
コンフリクトしてしまうと思いますが、甘えさせてもらって、先にこれは入れさせてもらおうと思います。

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

[usagisita]

マージします。

[berryzplus]

前に作成して放置中のコードなんですが、こんなの作りました。
https://github.com/sanomari/sakura-editor/blob/0b3583fbc8cd245584bf287c72e5f07995bf729b/sakura_core/dlg/CDlgProfileMgr.cpp#L55-L66

むしろコミットメッセージが気になる感じの修正でした。
　👉 プロファイルマネージャの機能をダイアログから分離する

そろそろ gmock をオンラインにするPRを作成すべきなのかも知れない、と思いました。