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

MinGWのビルドエラーを修正する #1948

Merged

Conversation

suconbu
Copy link
Member

@suconbu suconbu commented May 1, 2024

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ビルドそれぞれで解消されることを確認します。

関連 issue, PR

参考資料

@AppVeyorBot
Copy link

Build sakura 1.0.4337 completed (commit 296fd7e4a4 by @suconbu)

@AppVeyorBot
Copy link

Build sakura 1.0.4338 completed (commit cd3c343ef7 by @suconbu)

@suconbu suconbu marked this pull request as ready for review May 1, 2024 02:55
@suconbu
Copy link
Member Author

suconbu commented May 1, 2024

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

→この指摘は #1949 で修正します。

@suconbu suconbu force-pushed the feature/fix_builderror_mingw branch from 89ea5e3 to c99b200 Compare May 4, 2024 07:42
Copy link

sonarcloud bot commented May 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@AppVeyorBot
Copy link

Build sakura 1.0.4341 completed (commit 32d4d05ab2 by @suconbu)

@suconbu
Copy link
Member Author

suconbu commented May 4, 2024

当初出ていた VariantChangeType 関連のビルドエラーは #1949 により解消されたため、
PRの題目を、IVSに関連するテストコードのビルドエラーの修正に変更しました。

@berryzplus berryzplus merged commit db729fb into sakura-editor:master May 6, 2024
24 checks passed
@suconbu suconbu deleted the feature/fix_builderror_mingw branch May 8, 2024 14:57
@suconbu
Copy link
Member Author

suconbu commented May 8, 2024

レビュー、マージありがとうございました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants