<https://webkit.org/b/120707> Make WebKit's localizable strings mechanism usable...
authormrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Sep 2013 00:49:56 +0000 (00:49 +0000)
committermrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 5 Sep 2013 00:49:56 +0000 (00:49 +0000)
WebKit's localizable strings mechanism is also used outside of WebKit so it needs to work
both with and without ARC.

Reviewed by Anders Carlsson.

* Misc/WebLocalizableStrings.h: Mark the bundle member as unretained. This matches how the
member is used within WebLocalizableStrings.mm.

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

Source/WebKit/mac/ChangeLog
Source/WebKit/mac/Misc/WebLocalizableStrings.h

index c993687..8481111 100644 (file)
@@ -1,3 +1,15 @@
+2013-09-04  Mark Rowe  <mrowe@apple.com>
+
+        <https://webkit.org/b/120707> Make WebKit's localizable strings mechanism usable under ARC
+
+        WebKit's localizable strings mechanism is also used outside of WebKit so it needs to work
+        both with and without ARC.
+
+        Reviewed by Anders Carlsson.
+
+        * Misc/WebLocalizableStrings.h: Mark the bundle member as unretained. This matches how the
+        member is used within WebLocalizableStrings.mm.
+
 2013-09-02  Darin Adler  <darin@apple.com>
 
         [Mac] No need for HardAutorelease, which is same as CFBridgingRelease
index 853b601..15b0204 100644 (file)
@@ -36,7 +36,7 @@ typedef struct NSBundle NSBundle;
 
 typedef struct {
     const char *identifier;
-    NSBundle *bundle;
+    __unsafe_unretained NSBundle *bundle;
 } WebLocalizableStringsBundle;
 
 #ifdef __cplusplus