[Cocoa] Modernize some internal initializers to use instancetype instead of id
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Jun 2017 03:55:43 +0000 (03:55 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Jun 2017 03:55:43 +0000 (03:55 +0000)
https://bugs.webkit.org/show_bug.cgi?id=173112

Patch by Joseph Pecoraro <pecoraro@apple.com> on 2017-06-15
Reviewed by Wenson Hsieh.

* API/JSContextInternal.h:
* API/JSWrapperMap.h:
* API/JSWrapperMap.mm:
(-[JSObjCClassInfo initForClass:]):
(-[JSWrapperMap initWithGlobalContextRef:]):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@218379 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/JavaScriptCore/API/JSContextInternal.h
Source/JavaScriptCore/API/JSWrapperMap.h
Source/JavaScriptCore/API/JSWrapperMap.mm
Source/JavaScriptCore/ChangeLog

index 5308fbb..b37cfa4 100644 (file)
@@ -59,7 +59,7 @@ private:
 
 @interface JSContext(Internal)
 
-- (id)initWithGlobalContextRef:(JSGlobalContextRef)context;
+- (instancetype)initWithGlobalContextRef:(JSGlobalContextRef)context;
 
 - (void)notifyException:(JSValueRef)exception;
 - (JSValue *)valueFromNotifyException:(JSValueRef)exception;
index 046cc42..6c18c64 100644 (file)
@@ -31,7 +31,7 @@
 
 @interface JSWrapperMap : NSObject
 
-- (id)initWithGlobalContextRef:(JSGlobalContextRef)context;
+- (instancetype)initWithGlobalContextRef:(JSGlobalContextRef)context;
 
 - (JSValue *)jsWrapperForObject:(id)object inContext:(JSContext *)context;
 
index 98aa698..8cce369 100644 (file)
@@ -371,7 +371,7 @@ static void copyPrototypeProperties(JSContext *context, Class objcClass, Protoco
     JSC::Weak<JSC::JSObject> m_constructor;
 }
 
-- (id)initForClass:(Class)cls;
+- (instancetype)initForClass:(Class)cls;
 - (JSC::JSObject *)wrapperForObject:(id)object inContext:(JSContext *)context;
 - (JSC::JSObject *)constructorInContext:(JSContext *)context;
 - (JSC::JSObject *)prototypeInContext:(JSContext *)context;
@@ -380,7 +380,7 @@ static void copyPrototypeProperties(JSContext *context, Class objcClass, Protoco
 
 @implementation JSObjCClassInfo
 
-- (id)initForClass:(Class)cls
+- (instancetype)initForClass:(Class)cls
 {
     self = [super init];
     if (!self)
@@ -552,7 +552,7 @@ typedef std::pair<JSC::JSObject*, JSC::JSObject*> ConstructorPrototypePair;
     NSMapTable *m_cachedObjCWrappers;
 }
 
-- (id)initWithGlobalContextRef:(JSGlobalContextRef)context
+- (instancetype)initWithGlobalContextRef:(JSGlobalContextRef)context
 {
     self = [super init];
     if (!self)
index d26ced0..27a67eb 100644 (file)
@@ -1,3 +1,16 @@
+2017-06-15  Joseph Pecoraro  <pecoraro@apple.com>
+
+        [Cocoa] Modernize some internal initializers to use instancetype instead of id
+        https://bugs.webkit.org/show_bug.cgi?id=173112
+
+        Reviewed by Wenson Hsieh.
+
+        * API/JSContextInternal.h:
+        * API/JSWrapperMap.h:
+        * API/JSWrapperMap.mm:
+        (-[JSObjCClassInfo initForClass:]):
+        (-[JSWrapperMap initWithGlobalContextRef:]):
+
 2017-06-15  Matt Baker  <mattbaker@apple.com>
 
         Web Inspector: Instrument 2D/WebGL canvas contexts in the backend