Code review request [JDK 9] 8042449 Issue for negative byte major record version

Xuelei Fan xuelei.fan at oracle.com
Tue May 6 13:43:43 UTC 2014


On 5/6/2014 9:39 PM, Florian Weimer wrote:
> On 05/06/2014 03:37 PM, Xuelei Fan wrote:
>> On 5/6/2014 9:31 PM, Florian Weimer wrote:
>>> On 05/06/2014 02:00 PM, Xuelei Fan wrote:
>>>
>>>> Storing both int version and major/minor byte versions is a little bit
>>>> redundancy.  The update is significant.  I will focus on the signed
>>>> byte
>>>> issue in this fix.
>>>
>>> Yes, I get that.  I've verified that you've covered all the version
>>> comparisons.
> 
>> Thanks for the code review.  Do you have a OpenJDK author account?
> 
> I'm not an official reviewer, I'm afraid.
> 
I will your name in a "also reviewed by" section.  I still need an
official reviewer.

Thanks,
Xuelei




More information about the security-dev mailing list