[PATCH] D25710: [Doc] Drop MSVC2013 support
Krzysztof Parzyszek via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 18 08:13:07 PDT 2016
kparzysz added inline comments.
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
> 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.
If MSVC was the only reason that braced initializers were disallowed then we should allow them now. Why keep that restriction after MSVC is upgraded?
More information about the llvm-commits