sakura-editor / sakura

SAKURA Editor (Japanese text editor for MS Windows)

https://sakura-editor.github.io/

Other

1.24k stars 162 forks source link

最終行を矩形選択して削除すると落ちる問題を修正 #1947

Closed suconbu closed 5 months ago

suconbu commented 5 months ago

PR対象

アプリ(サクラエディタ本体)

カテゴリ

不具合修正

PR の背景

最終行の行全体を矩形選択した後、削除操作 (Delキー押下、切り取りなど) をすると、必ずアプリケーションが異常終了することが判明したため修正します。 sakura_boxselection_crash

仕様・動作説明

ユーザーに対するふるまいの変化はありません。

PR の影響範囲

特にないと思います。

テスト内容

「PR の背景」に書いた手順を行ってもアプリケーションが落ちないことを確認します。(効果確認)
矩形選択領域を削除した時のキャレット位置を補正する処理が、変更前と同様に機能することを確認します。(弊害確認)

削除前	削除後 (期待動作)	(補正がなかった場合の結果)

関連 issue, PR

参考資料

sonarcloud[bot] commented 5 months ago

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

AppVeyorBot commented 5 months ago

:white_check_mark: Build sakura 1.0.4335 completed (commit https://github.com/sakura-editor/sakura/commit/72722626fe by @suconbu)

suconbu commented 5 months ago

レビューありがとうございます。 Azure pipeline の MinGW ビルドが失敗するようですが、確認したところ本PRの変更と関係ない箇所 (CMacro.cpp:2465) のエラーでしたので、このままマージしたいと思います。