<div dir="ltr">LGTM.<div><br></div><div>-Eli</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Aug 21, 2013 at 2:32 PM, Christian Wailes <span dir="ltr"><<a href="mailto:chriswailes@google.com" target="_blank">chriswailes@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi dblaikie, delesley, aaron.ballman,<br>
<br>
A warning was added to clang that duplicated, word for world, the text from another warning.  This warning was removed, and code was changed to use the existing warning.<br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1467" target="_blank">http://llvm-reviews.chandlerc.com/D1467</a><br>
<br>
Files:<br>
  include/clang/Basic/DiagnosticSemaKinds.td<br>
  lib/Sema/SemaDeclAttr.cpp<br>
<br>
Index: include/clang/Basic/DiagnosticSemaKinds.td<br>
===================================================================<br>
--- include/clang/Basic/DiagnosticSemaKinds.td<br>
+++ include/clang/Basic/DiagnosticSemaKinds.td<br>
@@ -2189,9 +2189,6 @@<br>
 def warn_use_of_temp_while_consumed : Warning<<br>
   "invocation of method '%0' on a temporary object while it is in the "<br>
   "'consumed' state">, InGroup<Consumed>, DefaultIgnore;<br>
-def warn_uniqueness_attribute_wrong_decl_type : Warning<<br>
-  "%0 attribute only applies to methods">,<br>
-  InGroup<Consumed>, DefaultIgnore;<br>
<br>
 // ConsumedStrict warnings<br>
 def warn_use_in_unknown_state : Warning<<br>
Index: lib/Sema/SemaDeclAttr.cpp<br>
===================================================================<br>
--- lib/Sema/SemaDeclAttr.cpp<br>
+++ lib/Sema/SemaDeclAttr.cpp<br>
@@ -1003,8 +1003,8 @@<br>
   if (!checkAttributeNumArgs(S, Attr, 0)) return;<br>
<br>
   if (!(isa<CXXMethodDecl>(D) || isa<CXXConstructorDecl>(D))) {<br>
-    S.Diag(Attr.getLoc(), diag::warn_uniqueness_attribute_wrong_decl_type) <<<br>
-      Attr.getName();<br>
+    S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type) <<<br>
+      Attr.getName() << ExpectedMethod;<br>
     return;<br>
   }<br>
<br>
@@ -1019,8 +1019,8 @@<br>
   if (!checkAttributeNumArgs(S, Attr, 0)) return;<br>
<br>
   if (!isa<CXXMethodDecl>(D)) {<br>
-    S.Diag(Attr.getLoc(), diag::warn_uniqueness_attribute_wrong_decl_type) <<<br>
-      Attr.getName();<br>
+    S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type) <<<br>
+      Attr.getName() << ExpectedMethod;<br>
     return;<br>
   }<br>
<br>
@@ -1035,8 +1035,8 @@<br>
   if (!checkAttributeNumArgs(S, Attr, 0)) return;<br>
<br>
   if (!isa<CXXMethodDecl>(D)) {<br>
-    S.Diag(Attr.getLoc(), diag::warn_uniqueness_attribute_wrong_decl_type) <<<br>
-      Attr.getName();<br>
+    S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type) <<<br>
+      Attr.getName() << ExpectedMethod;<br>
     return;<br>
   }<br>
<br>
@@ -1051,8 +1051,8 @@<br>
   if (!checkAttributeNumArgs(S, Attr, 0)) return;<br>
<br>
   if (!isa<CXXMethodDecl>(D)) {<br>
-    S.Diag(Attr.getLoc(), diag::warn_uniqueness_attribute_wrong_decl_type) <<<br>
-      Attr.getName();<br>
+    S.Diag(Attr.getLoc(), diag::warn_attribute_wrong_decl_type) <<<br>
+      Attr.getName() << ExpectedMethod;<br>
     return;<br>
   }<br>
<br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br></div>