diff --git a/lib/Edit/RewriteObjCFoundationAPI.cpp b/lib/Edit/RewriteObjCFoundationAPI.cpp index 05abbed14c3d75ec03d18ad8ede0c94c416ae2ea..e257a0b6f1c0d42b7431e31f842fd68a0c8296ad 100644 --- a/lib/Edit/RewriteObjCFoundationAPI.cpp +++ b/lib/Edit/RewriteObjCFoundationAPI.cpp @@ -362,16 +362,16 @@ bool edit::rewriteToObjCProperty(const ObjCMethodDecl *Getter, const ParmVarDecl *argDecl = *Setter->param_begin(); QualType ArgType = argDecl->getType(); Qualifiers::ObjCLifetime propertyLifetime = ArgType.getObjCLifetime(); - if (propertyLifetime != Qualifiers::OCL_None) { - PropertyString += "("; - if (propertyLifetime == Qualifiers::OCL_Strong) - PropertyString += "strong"; - else if (propertyLifetime == Qualifiers::OCL_Weak) - PropertyString += "weak"; - else - PropertyString += "unsafe_unretained"; - PropertyString += ")"; + + if (ArgType->isObjCRetainableType() && + propertyLifetime == Qualifiers::OCL_Strong) { + PropertyString += "(copy)"; } + else if (propertyLifetime == Qualifiers::OCL_Weak) + PropertyString += "(weak)"; + else + PropertyString += "(unsafe_unretained)"; + QualType PropQT = Getter->getResultType(); PropertyString += " "; PropertyString += PropQT.getAsString(NS.getASTContext().getPrintingPolicy()); diff --git a/test/ARCMT/objcmt-property.m.result b/test/ARCMT/objcmt-property.m.result index 49162a4691465a0192f44c5aa01a91531b477999..11912d950a28cf7c36aa79e368ab992aa6b683ce 100644 --- a/test/ARCMT/objcmt-property.m.result +++ b/test/ARCMT/objcmt-property.m.result @@ -12,7 +12,7 @@ @property(weak) NSString *__weak WeakProp; -@property(strong) NSString * StrongProp; +@property(copy) NSString * StrongProp; - (NSString *) UnavailProp __attribute__((unavailable)); @@ -42,8 +42,8 @@ -@property(strong) NSArray * names2; -@property(strong) NSArray * names3; -@property(strong) NSArray *__strong names4; -@property(strong) NSArray * names1; +@property(copy) NSArray * names2; +@property(copy) NSArray * names3; +@property(copy) NSArray *__strong names4; +@property(copy) NSArray * names1; @end