- User Since
- Jun 8 2015, 2:10 PM (158 w, 1 d)
Wed, May 23
Mon, May 21
The warning is off by default.
We typically do not add off-by-default warnings because experience has shown the rarely get enabled in practice. Can you explain a bit more about why this one is off by default?
Right. I believe this is going to be used in practice, the reason I'm adding it involves some user demand for such warning. Such quoted include use in frameworks happen often and we would like a smooth transition to happen here (e.g. do not initially affect -Werror users). If it proves worth it, we can migrate to on by default in the future. It wouldn't be a problem if we have it on by default on open source and disable by default downstream, but I rather be consistent.
See also PR22165.
May 8 2018
I think will be good idea to store data in JSON format too.
May 6 2018
May 3 2018
Will be good idea to add documentation and item in Release Notes.
I don't see anything wrong with this change, but will be good idea to wait for code owner approval.
May 2 2018
May 1 2018
Apr 30 2018
Apr 22 2018
Apr 20 2018
Apr 19 2018
Apr 18 2018
Apr 17 2018
Apr 16 2018
Please also take a look on 26817 for another idea for .data().
Apr 15 2018
See also D45444.
Apr 12 2018
There is Clang-tidy's readability-implicit-bool-conversion check.
Apr 10 2018
How about doing same for objc-property-declaration?
Apr 9 2018
Will be good idea to add HICPP alias.
It'll be good idea to have option to apply this check for pointer/references only, or include built-in types/enums.
Apr 8 2018
Apr 7 2018
Apr 6 2018
If this is Apple guideline, check name should reflect this. I think will be good idea to have general check for Apple naming conventions instead of separate checks for specific situations like objc-ivar-declaration and objc-property-declaration.
Apr 4 2018
I found /usr/lib/libpfm.so.3.10.0
Looks like CMake is not checking for external dependencies properly. I got build error on RHEL 6 because of missing perfmon/perf_event.h. Most likely perfmon version is too old on RHEL6.
Mar 31 2018
Mar 29 2018
Will be good idea to clarify where TEMP_FAILURE_RETRY come from.
Mar 28 2018
Mar 27 2018
Please mention new diagnostics in documentation and Release Notes.
Mar 26 2018
Please mention new tool in Release Notes and use :doc: to refer to its manual.
I think it's fine to commit trivial fixes without going via review.
Mar 23 2018
Please also mention new warning in Release Notes and documentation.
This duplicates Clang-tidy misc-unused-using-decls. If Clang will provide same or better functionality, it should be removed.
Mar 22 2018
Mar 20 2018
Mar 19 2018
Mar 17 2018
Mar 16 2018
Mar 15 2018
Please run Clang-format and Clang-tidy modernize.
Mar 14 2018
Mar 13 2018
Mar 11 2018
Mar 10 2018
Mar 9 2018
Mar 6 2018
Please mention new module in Release Notes. I think new modules should be before new checks there.
Please run Clang-fromat. If I'm not mistaken, llvm-c/Initialization.h should be after InstCombineInternal.h.
Mar 5 2018
Mar 2 2018
Mar 1 2018
By the word, may be similar check for std::string::rfind() and std::string::ends_with() (does abseil have analog) should be added too?
Feb 28 2018
Feb 27 2018
std::basic_string::starts_with() was suggested for C++20. May be will be good idea to generalize code to create absl and modernize checks?
Please add new module in docs/clang-tidy/index.rst and mention it in release notes.