RFR: 8013155: [pack200] improve performance of pack200
Kumar Srinivasan
kumar.x.srinivasan at oracle.com
Tue Apr 30 18:32:38 UTC 2013
Hi Alex,
Great job identifying this hot method. I realize the code is
complicated but optimized
for speed.
Couple of nits:
I don't think you need the parens
j = (nextsemi < nextangl ? nextsemi : nextangl);
Coding conventions nits:
missing spaces for operators.
for (int i = firstl+1, j; i > 0; i = sig.indexOf('L', j)+1) {
I take it all the regression tests have been run ? in
jdk/test/tools/pack200.
Thanks
Kumar
> Please review this fix which by fine-tuning of just one method gives
> up to 40% performance
> improvement on packing and repacking of jar files.
> Also removed unneeded local variable initialization from Code class -
> not for performance gain,
> just to make world a better place.
>
> http://cr.openjdk.java.net/~kizune/8013155/webrev.00
More information about the core-libs-dev
mailing list