Anastasia added inline comments. ================ Comment at: lib/Parse/ParseDecl.cpp:3519 @@ +3518,3 @@ + / 100); + const char *VerSpec = (VerMajor + std::string (".") + VerMinor).c_str(); + Diag(Tok, DiagID) << VerSpec << PrevSpec << isStorageClass; ---------------- rivanvx wrote: > Anastasia wrote: > > I think it will be nicer to use string (not char*) here too. > Other Spec are const char*, so I did it for consistency. But I don't care > either way. Yes, it will be shorter. StringRef should work too I think.
Thanks! http://reviews.llvm.org/D19780 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits