VC++の規格準拠モードをONにする #1053
Merged
VC++の規格準拠モードをONにする #1053
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
VC++の「規格準拠(強化)モード」をオンにする。 詳細については以下VC++チームのブログを参照。 https://devblogs.microsoft.com/cppblog/msvc-conformance-improvements-in-visual-studio-2017-version-15-5/
ビルドを通すために規格非準拠はここで修正してしまう。 6⇒4errors
|
✅ Build sakura 1.0.2266 completed (commit e241dc7b1e by @berryzplus) |
beru
approved these changes
Sep 23, 2019
|
レビューありがとうございます。 |
HoppingTappy
pushed a commit
to HoppingTappy/sakura
that referenced
this pull request
Jun 16, 2020
…conformance_mode VC++の規格準拠モードをONにする
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR の目的
VC++のC++規格準拠モードをONにすることにより、
今後、コードベースに変なコードが混入するリスクを軽減します。
同時に、現状でC++規格に準拠できていないコードに対策を入れます。
カテゴリ
PR の背景
issue #872 「開発環境(Visual Studio)の機能をもっと活用するためにコード改善を行いたい」の残課題です。
残課題として残っていた理由は
/permissive-を利用するためには、やっかいな課題をいくつかクリアしないといけなかったからです。#1044 の対応で、
/permissive-を使うための前提条件「けっこう新しいWindows 10 SDKを使う」が解決できたので、投入を試みることにしました。で、やってみたらビルドエラーが出たので過去の検証で作った暫定対処コードをくっつけて、ビルド可能な状態にしてPRとしました。
PR のメリット
PR のデメリット (トレードオフとかあれば)
PR の影響範囲
関連チケット
#1044 プロジェクトのビルドにWindows10 SDKを使う
close #872 開発環境(Visual Studio)の機能をもっと活用するためにコード改善を行いたい
参考資料