[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
----------------
hawkinsw wrote:
> 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!!
And, for the , as well (after the opening clause). You and I are obviously of one mind!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126621