Currently SLObject's repository property has an attribute weak but it doesn't look correct -- otherwise we carefully retain a reference to the repository object, it can get niled out and server method invocation starts to fail (and I saw a few occasions that happens).
I thought that it was kept as weak to avoid any cyclic reference, but I don't see any such possibilities. I think it should have been strong.
Currently
SLObject
'srepository
property has an attributeweak
but it doesn't look correct -- otherwise we carefully retain a reference to therepository
object, it can getnil
ed out and server method invocation starts to fail (and I saw a few occasions that happens).I thought that it was kept as
weak
to avoid any cyclic reference, but I don't see any such possibilities. I think it should have beenstrong
.