IsBadStringPtrAの呼出を除去する

[berryzplus]

PR対象

• アプリ(サクラエディタ本体)
• テストコード

カテゴリ

• 不具合修正

PR の背景

• IsBadStringPtrAの呼出を削除したい #2187

興味があれば 関連 issue, PR を見てください。

仕様・動作説明

• CPPA::stdErrorから使用すべきでない古いAPI関数 IsBadStringPtrA の呼出を削除します。

「IsBadStringPtrA 非推奨 理由」でGoogle検索すれば、
この関数が何故ダメなのか、AIが解説してくれます。

本件についてもレビューつかないと思われるので勝手にやるつもりです。

PR の影響範囲

• 言及してない変更を多く含んでいますが、影響は少ないです。
  • 共有メモリ初期化手順の見直しはアプリ側に無影響です。
  • PPAマクロのエラー情報コールバックのテストもアプリ側に無影響です。
  • IsBadStringPtrAの除去の影響は、主に「アプリ審査が通りやすくなる」です。
  • relation_toolの不具合は、サクラエディタ終了時に ひっそりとクラッシュする が発生しなくなります。
    おそらく誰も気付いていないやつなので「実害なし」です。

テスト内容

本PRは単体テストで変更前後の「動作変更なし」を担保します。

関連 issue, PR

• resolves CPPA::stdErrorの処理が不正っぽい #1722 マージすると「不正じゃないです」を証明できます。
• CPPA::stdErrorにテストを導入したい #1811 過去に行った検討の残骸。
  本PRでは省略している細かい説明が色々書いてあるので興味があれば見るといいかも。
• CPPA::stdErrorのテストを追加するにあたっての課題共有 #1812 過去に行った検討の課題事項。
  ざっくり言うと、「表示されたメッセージを確認する方法がない」なんだけど、別PRでマージ済みなのでスルー可能。
• PPAのコンテキストをクラス外から設定できる仕組みを作りたい #1820 過去に行った検討の課題事項。
  本PRではしれっとテスト用関数 CallErrorProc を追加して回避している。
• resolves IsBadStringPtrAの呼出を削除したい #2187
• resolves CPPA::stdErrorのテストを書けるようにしたい #2188

参考資料

[github-actions]

Test Results

622 tests  +1   622 ✅ +1   1m 45s ⏱️ +24s
 78 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit c207a80. ± Comparison against base commit 9961b29.

This pull request removes 1 and adds 2 tests. Note that renamed tests count towards both.

CShareData ‑ test

CPPA ‑ ppaErrorProc
CShareData ‑ initShareData001

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

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

See analysis details on SonarQube Cloud