[libcxx-commits] [PATCH] D119441: [libc++] Fix locale name construction

[Hubert Tong via Phabricator via libcxx-commits]

hubert.reinterpretcast added inline comments.


================
Comment at: libcxx/src/locale.cpp:558
 
+string locale::__imp::build_name(string other, string one, locale::category c) {
+  if (c == locale::all)
----------------
ldionne wrote:
> The Standard says for e.g. `locale( const locale& other, const locale& one, category cat );`:
> 
> > Constructs a copy of other except for all the facets identified by the cat argument, which are copied from one. If both other and one have names, then the resulting locale also has a name.
> 
> Here, it seems that if `other` doesn't have a name but `one` does, the resulting locale will have a name. Is that intended?
> 
Sounds like the specification issue referred to by https://cplusplus.github.io/LWG/lwg-active.html#2295.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119441