Changeset 273940 in webkit

Timestamp:

Mar 4, 2021 8:17:40 PM (3 years ago)

Author:

achristensen@apple.com

Message:

[Cocoa] REGRESSION(r272752): fix some internal builds that use WTFString::WTFString(NSString *)
​https://bugs.webkit.org/show_bug.cgi?id=222610

Reviewed by Chris Dumez.

• inspector/scripts/codegen/objc_generator.py:

(ObjCGenerator.objc_protocol_import_expression_for_member):
(ObjCGenerator.objc_protocol_import_expression_for_parameter):
(ObjCGenerator.protocol_to_objc_expression_for_member):
(ObjCGenerator.payload_to_objc_expression_for_member):

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• inspector/scripts/codegen/objc_generator.py (modified) (3 diffs)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2021-03-04  Alex Christensen  <achristensen@webkit.org>
+
+        [Cocoa] REGRESSION(r272752): fix some internal builds that use WTFString::WTFString(NSString *)
+        https://bugs.webkit.org/show_bug.cgi?id=222610
+
+        Reviewed by Chris Dumez.
+
+        * inspector/scripts/codegen/objc_generator.py:
+        (ObjCGenerator.objc_protocol_import_expression_for_member):
+        (ObjCGenerator.objc_protocol_import_expression_for_parameter):
+        (ObjCGenerator.protocol_to_objc_expression_for_member):
+        (ObjCGenerator.payload_to_objc_expression_for_member):
+
 2021-03-04  Saam Barati  <sbarati@apple.com>
 

trunk/Source/JavaScriptCore/inspector/scripts/codegen/objc_generator.py

@@ -391 +391 @@
         if isinstance(member.type, EnumType):
             if member.type.is_anonymous:
-                return 'fromProtocolString<%s>(%s)' % (self.objc_enum_name_for_anonymous_enum_member(declaration, member), name)
-            return 'fromProtocolString<%s>(%s)' % (self.objc_enum_name_for_non_anonymous_enum(member.type), name)
+                return 'fromProtocolString<%s>((__bridge CFStringRef)%s)' % (self.objc_enum_name_for_anonymous_enum_member(declaration, member), name)
+            return 'fromProtocolString<%s>((__bridge CFStringRef)%s)' % (self.objc_enum_name_for_non_anonymous_enum(member.type), name)
         return self.objc_protocol_import_expression_for_variable(member.type, name)
 
@@ -443 +443 @@
             if isinstance(member.type, EnumType):
                 if member.type.is_anonymous:
-                    return 'fromProtocolString<%s>(%s).value()' % (self.objc_enum_name_for_anonymous_enum_member(declaration, member), sub_expression)
-                return 'fromProtocolString<%s>(%s).value()' % (self.objc_enum_name_for_non_anonymous_enum(member.type), sub_expression)
+                    return 'fromProtocolString<%s>((__bridge CFStringRef)%s).value()' % (self.objc_enum_name_for_anonymous_enum_member(declaration, member), sub_expression)
+                return 'fromProtocolString<%s>((__bridge CFStringRef)%s).value()' % (self.objc_enum_name_for_non_anonymous_enum(member.type), sub_expression)
             return sub_expression
         if category == ObjCTypeCategory.Object:
@@ -488 +488 @@
             sub_expression = 'payload[@"%s"]' % member.member_name
             if member.type.is_anonymous:
-                return 'fromProtocolString<%s>(%s)' % (self.objc_enum_name_for_anonymous_enum_member(declaration, member), sub_expression)
+                return 'fromProtocolString<%s>((__bridge CFStringRef)%s)' % (self.objc_enum_name_for_anonymous_enum_member(declaration, member), sub_expression)
             else:
-                return 'fromProtocolString<%s>(%s)' % (self.objc_enum_name_for_non_anonymous_enum(member.type), sub_expression)
+                return 'fromProtocolString<%s>((__bridge CFStringRef)%s)' % (self.objc_enum_name_for_non_anonymous_enum(member.type), sub_expression)
         if isinstance(_type, ObjectType):
             objc_class = self.objc_class_for_type(member.type)