On 11/25/2015 06:14 PM, Claes Redestad wrote:
> On 2015-11-25 16:06, Aleksey Shipilev wrote:
>> On 11/25/2015 05:44 PM, Claes Redestad wrote:
>> Nit: don't you want to inline BD.getTypeChar into BD.unparseSig to avoid
>> "c != 'L')" comparison? Or you want to keep it separate as the utility
>> method?
> 
> Inlining the isPrimitive check should be enough to avoid extra comparisons:
> 
> http://cr.openjdk.java.net/~redestad/8143926/webrev.02

Yes. Looks good!

-Aleksey


Reply via email to