[libcxx-commits] [PATCH] D126621: [libc++] Fix typo in comment at __optional_storage_base

[Will Hawkins via Phabricator via libcxx-commits]

hawkinsw added inline comments.


================
Comment at: libcxx/include/optional:396
 
-// optional<T&> is currently required ill-formed, however it may to be in the
+// optional<T&> is currently required to be ill-formed, however it may be in the
 // future. For this reason it has already been implemented to ensure we can
----------------
jloser wrote:
> philnik wrote:
> > This sentence still doesn't make any sense. I think there is a "not" missing somewhere.
> I'd be OK with something like this instead - what do you think?
> 
> 
> 
> > // optional<T&> is currently required to be ill-formed. However, it may be allowed in the
> > // future. For this reason, it has already been implemented to ensure we can make the change in an ABI-compatible manner.
> 
> 
Yes, of course!! I completely boofed. I was so excited to be able to fit that all within 80 characters that I never thought to reread! I am *so* sorry!

Also, thank you for being as persnickety as I am -- there is definitely a dash needed between ABI and compatible!!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126621