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

format関数を使えるようにC++20に移行する #2014

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

berryzplus
Copy link
Contributor

PR対象

  • アプリ(サクラエディタ本体)
  • テストコード
  • ビルド手順/CI

カテゴリ

  • 改善

PR の背景

現状のコードベースは C++17 準拠で、C++のフォーマット関数を利用できずに不便です。

仕様・動作説明

コードベースの準拠レベルを C++20 に変更し、C++のフォーマット関数を利用できるようにします。

PR の影響範囲

機能的な影響はありません。
vs2017 ではビルドできなくなります。

テスト内容

関連 issue, PR

参考資料

C++20では u8リテラル文字列 と MBCSリテラル文字列 が別型となり、ビルドが通らなくなるため。
C++20では u8リテラル文字列 と MBCSリテラル文字列 が別型となり、ビルドが通らなくなるため。
C++コンパイラーのC++準拠レベルをC++20にして std::format を使えるようにする
C++20対応は vs2019 からなので Appveyor のビルドイメージを変更する
@AppVeyorBot
Copy link

Build sakura 1.0.4451 failed (commit aa72867765 by @berryzplus)

C++コンパイラーのC++準拠レベルをC++20にして std::format を使えるようにする
@AppVeyorBot
Copy link

Build sakura 1.0.4452 failed (commit 867dc47e77 by @berryzplus)

@berryzplus
Copy link
Contributor Author

状況

初回ビルドは Appveyor のビルドイメージを vs2019 に変更しようとして、「cinst がない」のエラーでビルド失敗。
2回目のビルドは Appveyor のビルドイメージを戻して、「vs2017stdcpp20 に未対応」のエラーでビルド失敗。

今後の対応

Appveyor の問題を解決できるまで凍結。

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.

2 participants