I'd rather you put in a dispose method that explicitly disconnected the event handlers. That's the common pattern with things that register for events.

But it's probably not too bad.