diff options
author | Alp Toker <alp@nuanti.com> | 2014-05-20 22:03:47 +0000 |
---|---|---|
committer | Alp Toker <alp@nuanti.com> | 2014-05-20 22:03:47 +0000 |
commit | 6883743fd8157e61a1f57759527d90c6d72107c6 (patch) | |
tree | 0c3377bf3df05eeb32e214ca0ef41dce9981b2b8 /clang/lib/Sema/SemaDecl.cpp | |
parent | fdafb94f019d10bf767fbea33c35b3fc1f4d3c83 (diff) | |
download | bcm5719-llvm-6883743fd8157e61a1f57759527d90c6d72107c6.tar.gz bcm5719-llvm-6883743fd8157e61a1f57759527d90c6d72107c6.zip |
Reduce string duplication
If we're so keen on saving a dynamic allocation to add the trailing space, we
might as well do it in style.
llvm-svn: 209247
Diffstat (limited to 'clang/lib/Sema/SemaDecl.cpp')
-rw-r--r-- | clang/lib/Sema/SemaDecl.cpp | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/clang/lib/Sema/SemaDecl.cpp b/clang/lib/Sema/SemaDecl.cpp index 44ca495cb15..d65ba117dc8 100644 --- a/clang/lib/Sema/SemaDecl.cpp +++ b/clang/lib/Sema/SemaDecl.cpp @@ -513,35 +513,30 @@ static bool isTagTypeWithMissingTag(Sema &SemaRef, LookupResult &Result, LookupResult R(SemaRef, Name, NameLoc, Sema::LookupTagName); SemaRef.LookupParsedName(R, S, &SS); if (TagDecl *Tag = R.getAsSingle<TagDecl>()) { - const char *TagName = 0; - const char *FixItTagName = 0; + StringRef FixItTagName; switch (Tag->getTagKind()) { case TTK_Class: - TagName = "class"; FixItTagName = "class "; break; case TTK_Enum: - TagName = "enum"; FixItTagName = "enum "; break; case TTK_Struct: - TagName = "struct"; FixItTagName = "struct "; break; case TTK_Interface: - TagName = "__interface"; FixItTagName = "__interface "; break; case TTK_Union: - TagName = "union"; FixItTagName = "union "; break; } + StringRef TagName = FixItTagName.drop_back(); SemaRef.Diag(NameLoc, diag::err_use_of_tag_name_without_tag) << Name << TagName << SemaRef.getLangOpts().CPlusPlus << FixItHint::CreateInsertion(NameLoc, FixItTagName); |