[PATCH] D44539: [Sema][Objective-C] Add check to warn when property of objc type has assign attribute

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon May 21 00:52:08 PDT 2018


rjmccall added a comment.

This was approved by the Objective-C language group as a default-off warning.



================
Comment at: include/clang/Basic/DiagnosticSemaKinds.td:1018
+def warn_objc_property_assign_on_object : Warning<
+  "'assign' attribute must not be of object type, use 'unsafe_unretained' instead">,
+  InGroup<ObjCPropertyAssignOnObjectType>, DefaultIgnore;
----------------
"must" is rather strong for a warning.  Maybe something more like "'assign' attribute on property of object type could be 'unsafe_unretained'"?


Repository:
  rC Clang

https://reviews.llvm.org/D44539





More information about the cfe-commits mailing list