suconbu commented 6 months ago

PR対象

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

カテゴリ

ビルド関連

PR の背景

MinGWビルド時に以下のエラーが出るため直します。
(test-ccodebase.cpp で1か所、test-ccodebase.cpp で2か所)

D:/a/1/s/tests/unittests/test-ccodebase.cpp:818:88: error: converting to execution character set: Illegal byte sequence
  818 |         EXPECT_STREQ(L"E8919BF3A08480", pCodeBase->CodeToHex(L"葛󠄀", sStatusbar).c_str());

修正方法としては、該当の文字を直接書く代わりに \U... (ユニバーサル文字名) の形式で指定するようにします。

仕様・動作説明

ビルドエラー修正のみのため省略します。

PR の影響範囲

特にありません。

テスト内容

「PR の背景」に書いたエラーが、ローカルビルド、CIビルドそれぞれで解消されることを確認します。

参考資料

AppVeyorBot commented 6 months ago

:white_check_mark: Build sakura 1.0.4337 completed (commit https://github.com/sakura-editor/sakura/commit/296fd7e4a4 by @suconbu)

AppVeyorBot commented 6 months ago

:white_check_mark: Build sakura 1.0.4338 completed (commit https://github.com/sakura-editor/sakura/commit/cd3c343ef7 by @suconbu)

suconbu commented 6 months ago

const外しのキャストを追加したことで SonarCloud の指摘が1つ (以下リンク) 出てしまっていますが、同ソースファイルの他の箇所でも同様のキャストは行っているので、今回はこのままとさせてください。 ~~https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&types=CODE_SMELL&pullRequest=1948&id=sakura-editor_sakura&open=AY8xsI_-AkPibppNEOhE~~

~~→この指摘は https://github.com/sakura-editor/sakura/pull/1949 で修正します。~~