[Development] qHash / QHash changes
thiago.macieira at intel.com
Mon Apr 16 01:38:22 CEST 2012
On segunda-feira, 16 de abril de 2012 00.08.43, Olivier Goffart wrote:
> On Saturday 14 April 2012 20:06:53 Giuseppe D'Angelo wrote:
> > So, after *too many* commits -- QHash randomization was merged a
> > few hours ago!
> > https://qt.gitorious.org/qt/qtbase/commit/c01eaa438200edc9a3bbcd8ae1e8ded0
> > 58 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.
It was stored as the same field because hash and key were the same.
if you salt the hash, it's no longer the same. Therefore, they can't share a
> 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.)
Thiago Macieira - thiago.macieira (AT) intel.com
Software Architect - Intel Open Source Technology Center
Intel Sweden AB - Registration Number: 556189-6027
Knarrarnäsgatan 15, 164 40 Kista, Stockholm, Sweden
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 190 bytes
Desc: This is a digitally signed message part.
Url : http://lists.qt-project.org/pipermail/development/attachments/20120415/1ba5b743/attachment.bin
More information about the Development