[PATCH] D25710: [Doc] Drop MSVC2013 support

Piotr Padlewski via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 02:01:47 PDT 2016


Prazek accepted this revision.
Prazek added a reviewer: Prazek.
Prazek added a comment.

yey!



================
Comment at: docs/CodingStandards.rst:136
     uninitialized. Non-scalar members generally have appropriate default
-    constructors, and MSVC 2013 has problems when braced initializer lists are
-    involved.
+    constructors.
 
----------------
jlebar wrote:
> mehdi_amini wrote:
> > jlebar wrote:
> > > Can we add a comment or something to the effect of "should we reconsider this now that msvc 2013 is gone?"
> > Maybe we should consider it now?
> For my part, I have no problem with braced initializers inside a class.  They can be helpful.  But where possible I prefer using an equals sign, `int foo_member = 42`, because that's just easier to understand.
> 
> I dunno if that's contentious, though.
I also prefer = everywhere.


https://reviews.llvm.org/D25710





More information about the llvm-commits mailing list