review request for 7011998: diamond conversion for jgss and pkcs11

Stuart Marks stuart.marks at oracle.com
Fri Jan 14 00:15:14 UTC 2011


Yes, the byte codes are identical. I compiled with -g:none before and after the 
changes and the classfiles are all identical. (Even though the bytecodes are 
identical, the classfiles would differ because of changed line number 
information, which is disabled with -g:none.)

So, I assume this means that sunpkcs11.jar doesn't need to be updated, and that 
I can push this changeset without further changes?

s'marks

On 1/12/11 7:06 PM, Valerie (Yu-Ching) Peng wrote:
>
> The changes look good to me.
> BTW, I recall seeing in one of your earlier email that the byte code is the
> same w/ the usage of this diamond operator. Is this so?
> If not, then we need to update the sunpkcs11.jar also.
> Thanks,
> Valerie
>
> On 01/12/11 05:30 PM, Stuart Marks wrote:
>> Hi Valerie,
>>
>> You're up next for diamond conversion. :-)
>>
>> These should be pretty straightforward. Almost all changes are variable
>> initializations. There's one return statement, one use of diamond in a
>> ternary operator (a ? b : c), and one whitespace fixup.
>>
>> Webrev is here:
>>
>> http://cr.openjdk.java.net/~smarks/reviews/7011998/webrev.0/
>>
>> Thanks!
>>
>> s'marks
>



More information about the security-dev mailing list