[PATCH] D64671: [clang-tidy] New check: misc-init-local-variables

Fri Sep 6 14:22:23 PDT 2019

Eugene.Zelenko added a comment.

In D64671#1661585 <https://reviews.llvm.org/D64671#1661585>, @jpakkane wrote:

> > It'll be reasonable to get IncludeStyle default from .clang-format.
>
> I looked at existing checks and they all do the same thing as this one. In fact I got the code for this by directly copypasting an existing check.

SInse this option may be also set in .clang-format, it's reasonable to re-use it instead of forcing user to duplicate it.

================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:71
+
+- New :doc:`cppcoreguidelines-init-variables
+  <clang-tidy/checks/cppcoreguidelines-init-variables>` check.
----------------
jpakkane wrote:
> Eugene.Zelenko wrote:
> > Please sort new checks list alphabetically.
> The list is already incorrectly alphabetized. Should I reorganize all the entries or just put mine in a "correct" spot (between "bugprone" and "linuxkernel")?
It'll not hurt to correct other mistakes too.

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

https://reviews.llvm.org/D64671