Follow-up Comment #5, bug #25266 (project gnustep):
Fred,
This change...
"ASSIGN(_connections, [[coder decodeObjectForKey: @"NSConnections"]
mutableCopy]);
replacing it with
_connections = [[coder decodeObjectForKey: @"NSConnections"] mutableCopy];"
...is incorrect, I believe. I changed this to:
ASSIGN(_connections, (NSMutableArray *)
[coder decodeObjectForKey: @"NSConnections"])
and it seems to work fine.
In the dealloc the retain count for _connections is 2 only because the
NSEnumerator which was used to iterate over it in the awakeWithContext: method
has not yet been released and dealloc'd.
After this change the NSIBOutletConnectors are being freed. I have attached
a snapshop of Bean's memory panel.
I am still testing this. I committed all of the changes discused above, plus
the one you suggested in nibInstatiateWithInstantiator:.
I am going to continue to document the code and should be done with all of
that by tonight.
Thanks, GC
(file #17250)
_______________________________________________________
Additional Item Attachment:
File name: Bean_memory_panel.png Size:59 KB
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/bugs/?25266>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
_______________________________________________
Bug-gnustep mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/bug-gnustep
