[8u] RFR: 8203190: SessionId.hashCode generates too many collisions
Severin Gehwolf
sgehwolf at redhat.com
Fri May 17 11:37:21 UTC 2019
On Thu, 2019-05-16 at 19:10 +0100, Andrew John Hughes wrote:
>
> Change looks good.
Thanks for the review.
> Is there a reason the tests aren't included in the webrev? I think it
> would be better to have them checked in, even if they can't be run
> automatically.
The reason was that it's not a good test to be run automatically. It
would have to have some heuristic which it uses as "passed" and "fail".
Checking in the code anyway has a tendency for it to bitrot. If you
really feel strongly about it, I can add it. FWIW, the reference to the
test isn't going away so it'll be available either way.
> They will need copyright headers and I'd correct the spelling of
> 'collision' too :-)
:) Af for the typo: Well spotted.
Thanks,
Severin
More information about the security-dev
mailing list