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

Bug 278670

Summary: Issue with patch to fix Anti-aliasing does not work
Product: [Tools] GEF Reporter: Anthony Hunter <ahunter.eclipse>
Component: GEF-Legacy GEF (MVC)Assignee: Marc Gobeil <mgobeil>
Status: RESOLVED WONTFIX QA Contact:
Severity: normal    
Priority: P3    
Version: 3.5   
Target Milestone: 3.5.0 (Galileo) RC4   
Hardware: PC   
OS: Windows XP   
Whiteboard:

Description Anthony Hunter CLA 2009-06-01 16:13:00 EDT 
I used the patch in GMF Bug 276682 to fix GEF Bug 273541 .

Since Integer.valueOf(value) does not exist in Java 1.4, I fixed the patch to use new Integer(value).

Marc reports that using "new Integer()" skips an internal cache of Integer objects from -128 to 127 which covers most of the values assigned to setAlpha() and all the values for setAntialias(), and would avoid a lot of garbage when they're called repeatedly.

So Marc would like to provide an updated fix.
Comment 1 Marc Gobeil CLA 2009-06-03 15:05:35 EDT 
Integer.valueOf(value) and its associated cache don't seem to exist in Java 1.4, so can't make this improvement till GEF goes to Java 5.