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