This is fine, but a few years ago we agreed that we shouldn't repeat method names in doxygen. So an Even Better Fix would've been to just remove everything up to (and including) the ' - ' :-) Just FYI, no need to follow this up with the deletion.
On Tue, Oct 24, 2017 at 3:34 PM, Erich Keane via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: erichkeane > Date: Tue Oct 24 08:34:59 2017 > New Revision: 316447 > > URL: http://llvm.org/viewvc/llvm-project?rev=316447&view=rev > Log: > Fix spelling in comment, field is isMsStruct, not Strust > > Modified: > cfe/trunk/include/clang/AST/Decl.h > > Modified: cfe/trunk/include/clang/AST/Decl.h > URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/ > clang/AST/Decl.h?rev=316447&r1=316446&r2=316447&view=diff > ============================================================ > ================== > --- cfe/trunk/include/clang/AST/Decl.h (original) > +++ cfe/trunk/include/clang/AST/Decl.h Tue Oct 24 08:34:59 2017 > @@ -3540,7 +3540,7 @@ public: > return K >= firstRecord && K <= lastRecord; > } > > - /// isMsStrust - Get whether or not this is an ms_struct which can > + /// isMsStruct - Get whether or not this is an ms_struct which can > /// be turned on with an attribute, pragma, or -mms-bitfields > /// commandline option. > bool isMsStruct(const ASTContext &C) const; > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
