r316447 - Fix spelling in comment, field is isMsStruct, not Strust
Keane, Erich via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 24 09:10:19 PDT 2017
Ah, thanks for letting me know! I’ll do that now.
From: thakis at google.com [mailto:thakis at google.com] On Behalf Of Nico Weber
Sent: Tuesday, October 24, 2017 8:59 AM
To: Keane, Erich <erich.keane at intel.com>
Cc: cfe-commits <cfe-commits at lists.llvm.org>
Subject: Re: r316447 - Fix spelling in comment, field is isMsStruct, not Strust
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 at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>> wrote:
Date: Tue Oct 24 08:34:59 2017
New Revision: 316447
Fix spelling in comment, field is isMsStruct, not Strust
--- 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 at lists.llvm.org<mailto:cfe-commits at lists.llvm.org>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the cfe-commits