Code review comment for lp:~jpakkane/unity-scopes-api/visibility

Revision history for this message
Michi Henning (michihenning) wrote :

I'm feeling uneasy about this because it's more complicated than what we had previously. None of these destructors are even vaguely performance critical, and saving a dozen entries in the symbol table hardly seems worth it. On balance, I think I'd prefer to leave things as they are. The whole symbol munging stuff is complicated enough already…

« Back to merge proposal