xbmc/xbmc — comment on commit bfd5673a698d2f80f6d32e3b5efdf47737d3260f
File:  line 0
By topfs2:

Sorry for bringing this back up but what potential memory leak is fixed? Problem with the solution proposed here is that it adds at the very least 96bytes (on 32bit system) and most likely quite a bit more for each Variant in use. While this may be rather extreme to care I bring it up as CVariant was designed to be close to no overhead compared to using the explicit type. Before change the instance only ever used 64bits (if we assume it was no pointer), this made it quick to copy on the very basic types (bool, double, int) while obviously the more complex types needed larger copy. Now CVariant i(int) needs atleast 160bytes, which is quite much more :)