Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged

Conversation

suconbu
Copy link
Member

@suconbu suconbu commented Apr 28, 2024

PR対象

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

カテゴリ

  • 不具合修正

PR の背景

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

仕様・動作説明

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

PR の影響範囲

特にないと思います。

テスト内容

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

関連 issue, PR

参考資料

Copy link

sonarcloud bot commented Apr 28, 2024

@AppVeyorBot
Copy link

Build sakura 1.0.4335 completed (commit 72722626fe by @suconbu)

Copy link
Member

@kengoide kengoide left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

記載されている欠陥が期待通りに修正されることを確認しました。

@suconbu
Copy link
Member Author

suconbu commented Apr 29, 2024

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

@suconbu suconbu merged commit 5def577 into sakura-editor:master Apr 29, 2024
21 of 24 checks passed
@suconbu suconbu added the 🐛bug🦋 ■バグ修正(Something isn't working) label Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛bug🦋 ■バグ修正(Something isn't working)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants