Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 346518

Summary: weaving hook support creates an extra byte[] copy each class load when it is not necessary
Product: [Eclipse Project] Equinox Reporter: Thomas Watson <tjwatson>
Component: FrameworkAssignee: Thomas Watson <tjwatson>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3    
Version: 3.7   
Target Milestone: 3.7.1   
Hardware: PC   
OS: Mac OS X - Carbon (unsup.)   
Whiteboard:
Bug Depends on: 348770    
Bug Blocks:    
Attachments:
Description Flags
patch none

Description Thomas Watson CLA 2011-05-19 14:10:33 EDT
In org.eclipse.osgi.internal.baseadaptor.weaving.WeavingHookConfigurator.processClass(String, byte[], ClasspathEntry, BundleEntry, ClasspathManager) there is a call to org.eclipse.osgi.internal.baseadaptor.weaving.WovenClassImpl.setHooksComplete() which I think is unnecessary and results in an extra byte[] copy for each class load.
Comment 1 Thomas Watson CLA 2011-05-19 17:24:24 EDT
Created attachment 196172 [details]
patch
Comment 2 Thomas Watson CLA 2011-06-30 12:13:29 EDT
Patch released for 3.7.1.