[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.
Repository:
rCXX libc++
https://reviews.llvm.org/D54673
More information about the libcxx-commits
mailing list