r191240 - ObjectiveC migrator. When migrating to property declaration

[Fariborz Jahanian]

Author: fjahanian
Date: Mon Sep 23 18:18:46 2013
New Revision: 191240

URL: http://llvm.org/viewvc/llvm-project?rev=191240&view=rev
Log:
ObjectiveC migrator. When migrating to property declaration
use Cocoa's naming convention for properties of ObjC
object type. // rdar://15045005

Modified:
    cfe/trunk/lib/ARCMigrate/ObjCMT.cpp
    cfe/trunk/test/ARCMT/objcmt-property.m.result

Modified: cfe/trunk/lib/ARCMigrate/ObjCMT.cpp
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/ARCMigrate/ObjCMT.cpp?rev=191240&r1=191239&r2=191240&view=diff
==============================================================================
--- cfe/trunk/lib/ARCMigrate/ObjCMT.cpp (original)
+++ cfe/trunk/lib/ARCMigrate/ObjCMT.cpp Mon Sep 23 18:18:46 2013
@@ -304,7 +304,9 @@ static bool rewriteToObjCProperty(const
   }
   PropertyString += " ";
   PropertyString += RT.getAsString(Context.getPrintingPolicy());
-  PropertyString += " ";
+  char LastChar = PropertyString[PropertyString.size()-1]        ;
+  if (LastChar != '*')
+    PropertyString += " ";
   if (LengthOfPrefix > 0) {
     // property name must strip off "is" and lower case the first character
     // after that; e.g. isContinuous will become continuous.

Modified: cfe/trunk/test/ARCMT/objcmt-property.m.result
URL: 
http://llvm.org/viewvc/llvm-project/cfe/trunk/test/ARCMT/objcmt-property.m.result?rev=191240&r1=191239&r2=191240&view=diff
==============================================================================
--- cfe/trunk/test/ARCMT/objcmt-property.m.result (original)
+++ cfe/trunk/test/ARCMT/objcmt-property.m.result Mon Sep 23 18:18:46 2013
@@ -22,21 +22,21 @@ typedef char BOOL;
   int ivarVal;
 }
 
-@property(nonatomic, weak) NSString * WeakProp;
+@property(nonatomic, weak) NSString *WeakProp;
 
-@property(nonatomic, retain) NSString * StrongProp;
+@property(nonatomic, retain) NSString *StrongProp;
 
 
 - (NSString *) UnavailProp  __attribute__((unavailable));
 - (void) setUnavailProp  : (NSString *)Val;
 
-@property(nonatomic, retain) NSString * UnavailProp1  
__attribute__((unavailable));
+@property(nonatomic, retain) NSString *UnavailProp1  
__attribute__((unavailable));
 
 
 - (NSString *) UnavailProp2;
 - (void) setUnavailProp2  : (NSString *)Val  __attribute__((unavailable));
 
-@property(nonatomic, copy) NSDictionary * undoAction;
+@property(nonatomic, copy) NSDictionary *undoAction;
 
 @end
 
@@ -56,10 +56,10 @@ typedef char BOOL;
 
 
 
-@property(nonatomic, retain) NSArray * names2;
-@property(nonatomic, retain) NSArray * names3;
-@property(nonatomic, retain) NSArray * names4;
-@property(nonatomic, retain) NSArray * names1;
+@property(nonatomic, retain) NSArray *names2;
+@property(nonatomic, retain) NSArray *names3;
+@property(nonatomic, retain) NSArray *names4;
+@property(nonatomic, retain) NSArray *names1;
 @end
 
 // Properties that contain the name "delegate" or "dataSource",
@@ -97,13 +97,13 @@ typedef char BOOL;
 @property(nonatomic, readonly) int Length;
 @property(nonatomic, readonly) id object;
 + (double) D;
-@property(nonatomic, readonly) void * JSObject 
WEBKIT_OBJC_METHOD_ANNOTATION(AVAILABLE_WEBKIT_VERSION_3_0_AND_LATER);
+@property(nonatomic, readonly) void *JSObject 
WEBKIT_OBJC_METHOD_ANNOTATION(AVAILABLE_WEBKIT_VERSION_3_0_AND_LATER);
 @property(nonatomic, getter=isIgnoringInteractionEvents, readonly) BOOL 
ignoringInteractionEvents;
 
-@property(nonatomic, getter=getStringValue, retain) NSString * stringValue;
+@property(nonatomic, getter=getStringValue, retain) NSString *stringValue;
 @property(nonatomic, getter=getCounterValue, readonly) BOOL counterValue;
 
-@property(nonatomic, getter=getns_dixtionary, readonly) NSDictionary * 
ns_dixtionary;
+@property(nonatomic, getter=getns_dixtionary, readonly) NSDictionary 
*ns_dixtionary;
 
 - (BOOL)is3bar; // watch out
 - (NSString *)get3foo; // watch out
@@ -187,7 +187,7 @@ DEPRECATED
 - (NSURL *)appStoreReceiptURL NS_AVAILABLE;
 - (void) setAppStoreReceiptURL : (NSURL *)object;
 
-@property(nonatomic, retain) NSURL * appStoreReceiptURLX NS_AVAILABLE;
+@property(nonatomic, retain) NSURL *appStoreReceiptURLX NS_AVAILABLE;
 
 
 // Do not infer a property.


_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits