配列クラスを整備する

[berryzplus]

PR対象

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

カテゴリ

• 改善

PR の背景

• 先行 issue はとくに作成していません。

仕様・動作説明

• 独自の可変長配列クラス StaticVector を整備します。
• 独自の固定長文字列バッファクラス StaticString を整備します。
• 独自の可変長文字列バッファクラス CNativeW を整備します。

「生配列の代わりに使えるクラスを整備する」の目的で始めた整備でしたが、
CNativeWも合わせて整備するのが妥当と思われたため一緒に含めることにしました。

最大の変更点はNULLの扱いです。

変更前） 文字列は LPCWSTR で受け取る。
変更後） 文字列は std::wstring_view で受け取る。

LPCWSTR はwchar_tポインタなので、NULL値になり得ます。
std::wstring_view は文字列参照なので、NULL値になりません。

旧来の実装では NULL を有効な無効値として扱うために色々苦労していました。
思い切って 「NULL値は無効」 に倒してしまうことで整理しやすくします。

LPCWSTR pszText = nullptr;
SFilePath szFile{};
szFile = pszText; // ←従来なら有効だったコード。

今後は無効で、やったらクラッシュします。

「いい悪い」の判断は不可能と思うので、
本件も勝手にやります。

PR の影響範囲

テスト内容

関連 issue, PR

参考資料

[berryzplus]

上げてから思い出す。GitHub Actionsの英語版だと何故かこうなるのよね。

組み込み演算子 operator [](size_t) と
独自定義演算子 operator [](int) の
どちらを使えばよいか分からんのでエラー、というやつ。

[berryzplus]

全然違う話だった。

理由：
x86ビルドをまったく考慮していなかったため。

前提がひっくり返るので取り下げます。

[berryzplus]

あとでコミットの再編をするとして、対策済み修正をあげておくことに。

[github-actions]

Test Results

1 358 tests  +1   1 358 ✅ +1   7m 3s ⏱️ -25s
  122 suites ±0       0 💤 ±0 
    1 files   ±0       0 ❌ ±0 

Results for commit 49ec2dc. ± Comparison against base commit 80132bc.

This pull request removes 5 and adds 6 tests. Note that renamed tests count towards both.

CNativeW ‑ AppendStringNullPointer
CNativeW ‑ AssignStringNullLiteral
CNativeW ‑ AssignStringNullPointer
CNativeW ‑ CompareWithStringPtr
CNativeW ‑ ConstructWithStringNull

CNativeW ‑ CompareWithString
CNativeW ‑ length001
CNativeW ‑ length101
StaticVector ‑ test101
StaticVector ‑ test102
StaticVector ‑ test103

[berryzplus]

ひっくり返った前提

前提： StaticVectorを生配列の代わりとして使いたい。
発覚： 生配列の代わりとして使おうとすると x86 がビルドできない。

StaticVectorを生配列として扱ってたのはCRecentで、
もともと暗黙変換ではなく取得メソッドを利用していた。
しかも、CRecent自体に整理したほうがよさげな気配がある。

どうまとめるかは考え中。

[sonarqubecloud]

Quality Gate passed

Issues
23 New issues
6 Accepted issues

Measures
0 Security Hotspots
95.0% Coverage on New Code
1.0% Duplication on New Code

See analysis details on SonarQube Cloud