[PATCH] D54673: Fix some `-Wshadow` diagnostics emitted by Clang trunk

Louis Dionne via Phabricator reviews at reviews.llvm.org
Mon Nov 19 08:11:04 PST 2018

ldionne added a comment.

I'm not sure these warnings normally hit users because they should include us with `-isystem`, right?

Comment at: include/charconv:549
         __first, __last, __value,
-        [](const char* __p, const char* __last, _Tp& __value,
-           int __base) -> from_chars_result {
+        [](const char* __p, const char* __lastx, _Tp& __valuex,
+           int __basex) -> from_chars_result {
In this case, would it be less intrusive to rename the parameter from `__first` to `__first_it` or something like that? I think the parameters are used less often here.

  rCXX libc++


More information about the libcxx-commits mailing list