[PATCH] misc-uninitialized-field
Alexander Kornienko
alexfh at google.com
Fri Jun 19 07:33:54 PDT 2015
The check would be interesting to some projects, but it's not generally accepted that all fields of structs/ classes need to be initialized in the constructor. They need to be initialized _before they are used_. Thus, it needs to be moved out of the "misc" module which is reserved for more generic checks that are fine for most projects (at least, for LLVM and for projects using Google C++ style).
Also, are you aware of any styles that require initialization of all class fields in the constructor? Would be nice to refer to them in the check description.
REPOSITORY
rL LLVM
http://reviews.llvm.org/D10553
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list