[libcxx-commits] [PATCH] D95251: Remove noexcept from basic_string::find and implementation functions.

[Casey Carter via Phabricator via libcxx-commits]

CaseyCarter added a comment.

In D95251#2516256 <https://reviews.llvm.org/D95251#2516256>, @zoecarver wrote:

> In the issue, I'll ask for clarification and propose either that all these find-like members are marked noexcept or none of them are. The thing that's really making me uneasy is that two of them are and the rest aren't.

It's probably a wide vs. narrow contract thing. The find overloads that take const char*, for example, have a precondition that the pointer is non-null. Under the so-called "Lakos rule" they would/could/should be "Throws: Nothing" but not noexcept.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95251/new/

https://reviews.llvm.org/D95251