mehdi_amini added inline comments. ================ Comment at: include/clang/Basic/LangOptions.h:131 @@ +130,3 @@ + /// input is a header file (i.e. -x c-header). + bool IsHeaderFile = false; + ---------------- aaron.ballman wrote: > Should move the initializer to the constructor (not certain that MSVC 2013 > supports this construct, but it's also more consistent). Interestingly I prefer to have them on the fields themselves (when writing new struct/classes), but I agree that consistency takes precedence when adding a field to an existing class.
http://reviews.llvm.org/D15926 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits