Author: wlux
Date: Thu Oct 27 17:00:17 2016
New Revision: 40181

URL: http://svn.gna.org/viewcvs/gnustep?rev=40181&view=rev
Log:
Don't tell a lie. The retain count representation of gnustep-base and
libobjc2 differs, so we cannot use the ARC retain/release implementation
for now.

Modified:
    libs/base/trunk/ChangeLog
    libs/base/trunk/Source/NSObject.m

Modified: libs/base/trunk/ChangeLog
URL: 
http://svn.gna.org/viewcvs/gnustep/libs/base/trunk/ChangeLog?rev=40181&r1=40180&r2=40181&view=diff
==============================================================================
--- libs/base/trunk/ChangeLog   (original)
+++ libs/base/trunk/ChangeLog   Thu Oct 27 17:00:17 2016
@@ -1,3 +1,9 @@
+2016-10-27  Wolfgang Lux  <wolfgang....@gmail.com>
+
+       * Source/NSObject.m: Comment out _ARCCompliantRetainRelease for
+       now because the retain count representation in gnustep-base and
+       libobjc2 differs causing space leaks on 64-bit architectures.
+
 2016-10-24  Richard Frith-Macdonald <r...@gnu.org>
 
        * Source/NSData.m: Only atempt to read regular files into NSData,

Modified: libs/base/trunk/Source/NSObject.m
URL: 
http://svn.gna.org/viewcvs/gnustep/libs/base/trunk/Source/NSObject.m?rev=40181&r1=40180&r2=40181&view=diff
==============================================================================
--- libs/base/trunk/Source/NSObject.m   (original)
+++ libs/base/trunk/Source/NSObject.m   Thu Oct 27 17:00:17 2016
@@ -735,7 +735,17 @@
  * </p>
  */
 @implementation NSObject
+#if 0
+/* Commented out for now, since the representation of object retain
+ * counts in gnustep-base and libobjc2 currently is not compatible.
+ * While gnustep-base uses int32_t libobjc2 uses intptr_t, which makes
+ * a difference on 64-bit architectures.
+ */
+// FIXME Rewrite retain/release to use objc_retain/objc_release and
+// similarly object allocation to use class_createInstance when we
+// are using libobjc2.
 - (void)_ARCCompliantRetainRelease {}
+#endif
 
 + (void) _becomeMultiThreaded: (NSNotification *)aNotification
 {


_______________________________________________
Gnustep-cvs mailing list
Gnustep-cvs@gna.org
https://mail.gna.org/listinfo/gnustep-cvs

Reply via email to