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

Hubert Tong via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Sat Feb 12 08:22:55 PST 2022


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



More information about the libcxx-commits mailing list