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

static_assertのメッセージにあるtypoの修正 #826

Merged
merged 2 commits into from Dec 16, 2020

Conversation

ToruNiina
Copy link
Member

気づいたので直しましたが、この部分にはvalue == (true|false)を含まないようにしているページもあるようなので、単に消すだけでも良かったかもしれません。

@yumetodo
Copy link
Member

yumetodo commented Dec 9, 2020

むしろvalue == (true|false)と書いているページが有ることを今知りました・・・。

@ToruNiina
Copy link
Member Author

統一するという観点からも、trait::valuetrait::operator()など自体の説明をしているところ以外のコメントにあるvalue == (true|false)は全て削除したほうがシンプルでよいかもしれませんね。無いページもあるわけですし、そのほうが説明している内容に集中できるように思います。どうでしょう?

@yumetodo
Copy link
Member

yumetodo commented Dec 9, 2020

自分としてもそのほうがむしろわかりやすいと思います。

このまま @ToruNiina さんが修正されますか?

@ToruNiina
Copy link
Member Author

そうですね。このまま修正します。少々お待ちを。

@faithandbrave
Copy link
Member

ありがとうございます。これでマージしてしまいますね。

@faithandbrave faithandbrave merged commit a29c840 into cpprefjp:master Dec 16, 2020
@ToruNiina ToruNiina deleted the typo-comment branch December 16, 2020 06:56
@ToruNiina
Copy link
Member Author

ありがとうございます。

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.

None yet

3 participants