Re #10: Work around bug in Class::Observable observing instances

[ehuelsmann]

Description

Works around a bug in Class::Observable where observers are registered by memory address and not cleared when the value at that address is destroyed.

Relates to #10.

Type of change

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• My code follows the style guidelines of this project, please see the contribution guidelines.
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• New and existing unit tests pass locally with my changes

[coveralls]

Coverage increased (+0.02%) to 79.053% when pulling b757898 on ehuelsmann:issue-10 into d15d9ac on jonasbn:master.

[jonasbn]

Looks good, looking forward to seeing the results from the CPAN-testers. Hopefully the heisenbug is gone. Upstream changes might be forthcoming

[ap]

Given that #61 was merged, and so there is no longer a delete_all_observers method, this should be reverted.

It appears to work fine, but that’s only because the eval is swallowing the exception caused by the attempt to invoke a missing method. The exception even ends up not being observable at all, due to the fact that Perl >= 5.14 throws away changes to $@ that happen within a DESTROY.

Nevertheless – the method does run during destruction, it does set up an eval context, and it does try to find the delete_all_observers method, which is all completely pointless.

[ehuelsmann]

Yup. Thanks for pointing that out.