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

minmaxの注記に、ダングリングが発生するケースを追加 #492

Merged
merged 1 commit into from
Dec 10, 2017

Conversation

sumomoneko
Copy link
Member

qiitaにて、std::minmax()の落とし穴について投稿いたしました。
https://qiita.com/sumomoneko/items/58358eb8bcc8b70a481b

実際にminmax()を使う際に参照されることが多いであろうこのサイトにて、
注意喚起があって良いのではと思い、またyumetodoさんの後押しもあって
備考を追記しました。

下記議論を参照し、サンプルコードの拡張記法にも対応させています。
#481
最適化の具合により、assert()で問題が出たり出なかったりします。

@saki7 saki7 self-requested a review December 10, 2017 00:13
@saki7 saki7 added the bug label Dec 10, 2017
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.

@saki7 saki7 merged commit 3638143 into cpprefjp:master Dec 10, 2017
@saki7
Copy link
Contributor

saki7 commented Dec 10, 2017

@sumomoneko GitHub Organizationの招待を送らせていただきました。MemberになるとPull Requestのマージとレビューができるようになります。また、些細な変更などをpr出さずに直接コミットすることが出来るようになります。よろしければ、今後ともよろしくお願いします。

@sumomoneko
Copy link
Member Author

マージありがとうございます!またお誘い頂きありがとうございます。
個人的な立ち位置としては、規格の厳密さを追いかけるよりも、スキルセットもバラバラな現場でみんなそれなりにハッピーになれるようなお手伝いが出来たらなと考えています。
今後ともよろしくお願いします。

yumetodo added a commit that referenced this pull request Dec 16, 2017
#492@sumomoneko 氏によりこの文が加筆されたが、
その意図と検索可能性を考えればダングリングという語を消すのはまずい。

ref:
- 76c00ea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants