Re: [libstdc++] Assertion in optional

2017-05-15 Thread Jonathan Wakely
On 14/05/17 09:19 +0200, Marc Glisse wrote: Hello, this patch adds 2 simple __glibcxx_assert in optional that match the precondition in the comment above. I am not sure if there was a reason the author wrote that comment instead of the assertion, but constexpr use still seems to work. Yes,

[libstdc++] Assertion in optional

2017-05-14 Thread Marc Glisse
Hello, this patch adds 2 simple __glibcxx_assert in optional that match the precondition in the comment above. I am not sure if there was a reason the author wrote that comment instead of the assertion, but constexpr use still seems to work. I hesitated about having the assertion in operator