Gerd,
Have the recent changes fixed the PGN instability issue ?
Joost wrote:
> Yet what surprised me is that flag values > 3 really
> could be
> meaningless. Note that they are not taken into account in
> the switch
> statement to follow. Hence I wonder what is going on in the
> hash
> machinery itself. Including why the closing tte_ScoreFlag()
> operation in
> ProbeHash() masks the hash flag with 0x07.
> I mean, it might as well be that this mask value should
> actually be
> equal to 0x03.... This would render a meaningful value and
> explain the
> buffer size of 4 uints at the same time!
I had the same first impression , but Gerd hasn't changed this
line : engine.cpp:
inline scoreFlagT tte_ScoreFlag (transTableEntryT * tte)
{ return (tte->flags & 7); }
Isn't it meant to be
{ return (tte->flags & 3); }
Alex, i think your recent cvs commit *comments* are erroneous.
The engine.cpp changes are not fixes to scidlet,
but Scid's built-in engine, which is different.
Gerd seems to be responsible for some quite detailed
technical programming. Perhaps he should have CVS write
permissions ?? That way he can fix all his bugs easily enough ;>
Steve
------------------------------------------------------------------------------
Learn how Oracle Real Application Clusters (RAC) One Node allows customers
to consolidate database storage, standardize their database environment, and,
should the need arise, upgrade to a full multi-node Oracle RAC database
without downtime or disruption
http://p.sf.net/sfu/oracle-sfdevnl
_______________________________________________
Scid-users mailing list
Scid-users@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/scid-users
