RFR: 8013155: [pack200] improve performance of pack200 (original) (raw)
Alexander Zuev alexander.zuev at oracle.com
Thu May 2 13:30:54 UTC 2013
- Previous message: hg: jdk8/tl/jdk: 8013225: Refresh jdk's private ASM to the latest.
- Next message: RFR: 8013155: [pack200] improve performance of pack200
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Kumar,
On 4/30/13 22:32, Kumar Srinivasan wrote:
Couple of nits: I don't think you need the parens
j = (nextsemi < nextangl ? nextsemi : nextangl);_ Here i tend to agree with John - the condition being a superposition of two really closely named variables might look confusing and since the world supply of parentheses is not limited i will humbly ask for permission to leave one extra pair of them there. _Coding conventions nits:_ _missing spaces for operators._ _for (int i = firstl+1, j; i > 0; i = sig.indexOf('L', j)+1) { Fixed. I take it all the regression tests have been run ? in jdk/test/tools/pack200. Sure - since it is not platform-specific code i took liberty to perform testing on Mac OS and Linux only - all the pack200 regression tests are passed.
The updated webrev can be seen here: http://cr.openjdk.java.net/~kizune/8013155/webrev.01
With best regards, Alex
- Previous message: hg: jdk8/tl/jdk: 8013225: Refresh jdk's private ASM to the latest.
- Next message: RFR: 8013155: [pack200] improve performance of pack200
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]