[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