[Development] qHash / QHash changes

Olivier Goffart olivier at woboq.com
Mon Apr 16 00:08:43 CEST 2012


On Saturday 14 April 2012 20:06:53 Giuseppe D'Angelo wrote:
> So, after *too many* commits[1] -- QHash randomization was merged a
> few hours ago!
> 
> https://qt.gitorious.org/qt/qtbase/commit/c01eaa438200edc9a3bbcd8ae1e8ded058
> bea268
> 
> Thanks to all of the guys involved for the ideas, feedback, reviewing
> the patches... and staging them :-)
> 

Nice work!

I'm just not really happy with the fact that you removed the optimisation for 
int to avoid storing both the hash and the key.
You can't put it back in Qt 5.x because it is not binary compatible.
Did you really not find a way to fix it?  (i.e: not storing the hash for 
Primitive types.)



More information about the Development mailing list