トリプルクリック時に次の行の先頭にURLがあるとそれが選択されてしまう現象が起きないように対策

[beru]

PR の目的

#1745 で報告された問題が起きないようにするのが目的です。

カテゴリ

• 不具合修正

PR の背景

PR のメリット

トリプルクリック時に次の行が選択されてしまうのは挙動として不自然です。

それが起きないようにする事で予期しない挙動が発生するのを防ぎます。

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

マウス左ボタン押下に対応する CEditView::OnLBUTTONDOWN の処理を途中で中断するように return; を入れたけれど、その事によって本来行うべき処理が行われなくなる恐れがある。

仕様・動作説明

PR の影響範囲

マウス左ボタン押下に対応する CEditView::OnLBUTTONDOWN の処理で、トリプルクリックを処理する記述の最後に return; するように記述を追加しました。こうする事でトリプルクリック処理の後に、後続に記述されている処理が実行されないようにして問題の現象を発生する事を回避しました。

テスト内容

テスト1

手順

• 1行目に文字列、2行目にURLを書く。
• 1行目をトリプルクリックする。
• 1行目が選択される事を確認

関連 issue, PR

#1745

参考資料

[beru]

少し愚痴が入ってしまいますが、CEditView::OnLBUTTONDOWN メソッドは470行近くあります。こんだけ記述が長いと処理内容が互いに整合性が取れているか把握するのが厳しいと思いますね。実際に報告された変な現象が発生していますし。こういう長いプログラムの記述を見かける事が結構色々なシチュエーションであります。書いた人は多分その時は（その人の感覚的には）結構把握しているのかもしれませんが、後になってそのコードを読む人は？？？状態になります。

自分が問題回避のために追加した return; で処理を中断する事で他の問題が発生しないか心配ですが、まぁ影響範囲はトリプルクリック時のみなので限定されているかなと思います。

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

0.0% Coverage
0.0% Duplication

[AppVeyorBot]

✅ Build sakura 1.0.3971 completed (commit 7117b33f9b by @beru)

[berryzplus]

修正ここだけですね。

「必要と思われるreturnがなかったので足しました。」ですね。

「必要と思われるreturn」に異論はないです。

[beru]

レビューありがとうございました。Mergeします。
もし問題が見つかったらまた別のPRで対処します。