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

cpp example に修正 #488

Merged
merged 1 commit into from
Dec 5, 2017
Merged

cpp example に修正 #488

merged 1 commit into from
Dec 5, 2017

Conversation

e-kwsm
Copy link
Contributor

@e-kwsm e-kwsm commented Dec 4, 2017

* #480
* lang/cpp11/reference_collapsing.md: コンパイラを変更できなければ意味がない
@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

ありがとうございます。これは基本的には #480 の変更漏れへの対応という認識で良いでしょうか? 内容は一応全部見ましたが、OKだと思います。今は、 #487 で進行中の議論があるので、あちらで方針を明確にしてからこのPRに対応します。

@saki7
Copy link
Contributor

saki7 commented Dec 4, 2017

これたぶん fixes cpprefjp/kunai#69 だと思う

Copy link
Contributor

@saki7 saki7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#487 (comment) 左記コメントで説明した背景と合致しているので問題なし、非常に有用なfollow-upです。ありがとうございます。

@saki7 saki7 merged commit 96adf50 into cpprefjp:master Dec 5, 2017
@e-kwsm
Copy link
Contributor Author

e-kwsm commented Dec 5, 2017

これは基本的には #480 の変更漏れへの対応という認識で良いでしょうか?

はい.example を付けてないのは

実行可能でもあまりうまみがないため

そもそもこのままだとコンパイルが通らない

@saki7
Copy link
Contributor

saki7 commented Dec 5, 2017

すみません、自分で質問形式でコメントしていたのを忘れてマージしてしまいました 🙏

補足ありがとうございます。例として挙がっている記事では、

  • コードが十分短い
  • コンパイルが通ることが自明
  • コンパイルが通ることや通らないことが主題でない

以上の点があるので、これはこのままでも特に問題ないと思います。

もし、やっぱり付けたいということであれば、付けて大丈夫です。その場合はPRを投げないで直接コミットして良いです。

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