rx.disposeBag may be using wrong address about nsobject-rx HOT 7 CLOSED

Yup, gotcha. That's tricky – I think you're right that the struct is getting copied and the address used by objc_get/setAssociatedObject. Hmm. We may have to remove support for non-reference-types. This was introduced in this pr. I think adding a : class constraint to the HasDisposeBag protocol should fix the issue (while removing the ability to use this on value types).

What do you think? Up for a pull request? Thanks again for opening the issue!

from nsobject-rx.

Right now I'm doing some more investigation, I'll update when I'm done but I think that's the issue.

from nsobject-rx.

Cool, thanks! Let us know what you find 👍

from nsobject-rx.

Ok, I did quite an investigation.

• I removed this pod completely and made sure there is no rx.disposeBag in my code
• Replaced rx.disposeBag by self.disposeBag

But still, I'm seeing that issue. Which means I can't blame this pod for memory leaking.

However, I think that need to be fixed. I'll try to put a PR soon.

from nsobject-rx.

Awesome, I look forward to it 👍

from nsobject-rx.

PR is up! Discuss and comment there.

Thanks

from nsobject-rx.

Closed by #48, and released to CocoaPods as 4.0.0. Thanks again!

from nsobject-rx.