Move to C++17!!!

[sksat]

Overview

SSIA

Issue

• Related issues

Details

Update CMake CXX_STANDARD to 17

Validation results

Link to tests or validation results.

Scope of influence

• All code will be compile with C++17.

Supplement

Write additional comments if you need.

Note

• If there are related Projects, tie them together.
• Assignees should be set if possible.
• Reviewers should be set if possible.
• Set priority label if possible.

[sksat]

std::byteみたいなブロッカーがあるような気がしていたんですが，よく考えたら僕がusing namespace stdを消しまくっていたので衝突しなくなっていて普通にC++17でビルドできましたね

[200km]

こちら遅くなってすみません。この更新以降はユーザー側もC++17に移行してもらうようにするということで良いでしょうか。
このタイミングでバージョンを更新したりしますか？

[sksat]

今後C++17の機能をどんどん使っていきたい気持ちがあるのでユーザ側も移行すべきですね．バージョンも上げた方がよさそうです．

[200km]

ユーザー側の影響あるので、マージは少しお待ちいただけると嬉しいです。

[sksat]

ですね．あとはバージョン更新どうやっていくかとかを決めておきたいですね(semver導入なども)．

[200km]

バージョンも上げた方がよさそうです

これですが、バージョン番号をどうするかという点の整理も合わせて議論したいのですが、C++17のものはどういうバージョン番号割り当てましょうか？

[sksat]

そこなんですよね〜．少なくともsemver(的)なバージョニングでやるのはいいとしても，S2E coreの後方互換性ってどう考えたらいいもんかという． > バージョン番号

[200km]

後方互換性は今までもあまり考えてきておらず、後方互換がなくなる大きな更新のときにtag切って古いものを参照しやすくするくらいの運用でした。
これを気にしっかり方針決めても良いと思います。

[sksat]

というわけで方針決める場: #53

[200km]

最新状態に保ちたいユーザー部の方でも修正+動作確認したのでapproveします。

[sksat]

ありがとうございます．ではマージします(developですし)．