Community
Participate
Working Groups
Build Identifier: 1.6.10 I believe it would be interesting, according to Andy's suggestions, to: 1) Use LDC instead of classForName if Java 1.5 enabled. 2) Collapse calls to makeSJP avoiding calling to makeMethodSig when the flag is enabled. 3) Create a new makeSJP method not including exceptions when exceptions are not returned by the woven methods. These modifications may improve LTW time, define time and reduce woven class size. Find attached the patchs for the modifications including unitary tests. Thanks and regards. Reproducible: Always
Created attachment 177263 [details] Proposed patch
changes are all in. Slightly reworked them: - in bcel-builder I changed it to only use the wide form of LDC if the constant pool index needs it (>255). Uses addClass rather than lookupClass. - option handling slightly changed to fit in better with what we already use for X options. Changed name to -Xset:targetRuntime1_6_10 - new test class had to be referenced from the existing suite to ensure it is pulled in when running everything - javadoc on the new runtime factory methods. - slight refactoring in lazyclassgen, but basically what you had.
Andy, after performing our internal backward compatibility tests for 1.4 JVMs we realized there is a minor bug in the changes already committed at CVS (also in the proposed patch) that prevents working it in the right way when Weaving under AJ12 mode and 1.6.10 enhancements on I am attaching the patch to fix it. As well as the patch for the tests to test it and avoid regression issues.
Created attachment 177403 [details] This is the patch to fix the bug
Created attachment 177404 [details] This is the patch to add the proper test.
changes are in, thanks.
