Comment 14 for bug 1058652

Revision history for this message
Victor Martinez (victored) wrote :

Thank you for testing Ngô!

That's an excellent explanation of the problem. You're right, the previous devs of Wingpanel were asumming that every indicator only had one entry (menuitem), missing the fact that libapplication.so had a bunch of them. Indeed, the hashmap you mention was unreferencing the previous menuitem when a new one was added.

I'm glad we're finally managing references properly and safely. This kind of bugs should never come back.