☞ | ![]() ![]() |
65 |
Before: 91 lines (not counting unnecessary, noisy logging statements)
After: 64 lines (30% reduction)
Test suite still passes
Except for the one test that was broken to begin with
Several serious bugs fixed:
Garbage collection
Identity theft
Observers no longer called more than once
delete_observer works now
Module can be subclassed
Performance:
I don't know.
But I bet you don't either.
☞ | ![]() |
☞ |