Community
Participate
Working Groups
Created attachment 186347 [details] patch to remove unused variables related to bug 333678 ... EJBHelper in org.eclipse.jst.j2ee.ejb has an unused variable, executionMap, in loadChildren map. Recent changes in JDT compiler now correct detect this case, and remove it from the byte codes. Hence, that makes the bundle "different" even though version and qualifier are the same. It appears 'executionMap' was used in some code that is now commented out. If it is suspected that commented out code might be restored soon, this might be a case where it'd be more appropriate to re-tag the bundle, so its differences show up correct in versioning. However, if the unused variables are commented out for now, then same change could be made in maintenance stream too ... if it is easier to "leave the same" in both streams ... there is apparently no change from 'head' stream and 'maintenance' stream. The attached patch has the unused variables commented out, and the bundles manifest service field incremented by +1. (Should probably be +100, if committed only to head).
I've applied the patch and released for M6. I noticed the service field had been incremented to 402 already, but the 32 maintenance branch has 401, so 402 would be incorrect ... I think should have been +100 from maintenance. So, I changed to 502. Not sure why had been incremented to 402 ... if it was to "fix" this problem, or just to avoid being on versioning report. So, feel free to revert my fix of commenting out the unused variables, if that wasn't desired.
*** Bug 335075 has been marked as a duplicate of this bug. ***
