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

Reply via email to