Attachment 137390 Details for Bug 276682 – patch fix

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

[patch] patch fix

ShapeAntiAliasingFix.patch (text/plain), 2.54 KB, created by Marc Gobeil

on 2009-05-27 15:04:14 EDT

(hide)

Description:

Filename:

MIME Type:

Creator: Marc Gobeil

Created: 2009-05-27 15:04:14 EDT

Size: 2.54 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.draw2d
>Index: src/org/eclipse/draw2d/Shape.java
>===================================================================
>RCS file: /cvsroot/tools/org.eclipse.gef/plugins/org.eclipse.draw2d/src/org/eclipse/draw2d/Shape.java,v
>retrieving revision 1.12
>diff -u -r1.12 Shape.java
>--- src/org/eclipse/draw2d/Shape.java	3 Apr 2009 18:36:45 -0000	1.12
>+++ src/org/eclipse/draw2d/Shape.java	27 May 2009 19:04:15 -0000
>@@ -10,7 +10,6 @@
>  *******************************************************************************/
> package org.eclipse.draw2d;
> 
>-import org.eclipse.swt.SWT;
> import org.eclipse.swt.graphics.LineAttributes;
> 
> /**
>@@ -48,8 +47,8 @@
> 	private boolean outline;
> 	private boolean xorFill;
> 	private boolean xorOutline;
>-	private int antialias;
>-	private int alpha;
>+	private Integer antialias;
>+	private Integer alpha;
> 	
> 	/**
> 	 * Default constructor.
>@@ -63,8 +62,8 @@
> 		outline = true;
> 		xorFill = false;
> 		xorOutline = false;
>-		antialias = SWT.DEFAULT;
>-		alpha = 255;
>+		antialias = null;
>+		alpha = null;
> 
> 		// synchronize parameters
> 		lineWidth = (int)lineAttributes.width;
>@@ -91,8 +90,12 @@
> 	 * @see Figure#paintFigure(Graphics)
> 	 */
> 	public void paintFigure(Graphics graphics) {
>-		graphics.setAntialias(antialias);
>-		graphics.setAlpha(alpha);
>+		if(antialias != null) {
>+			graphics.setAntialias(antialias.intValue());
>+		}
>+		if(alpha != null) {
>+			graphics.setAlpha(alpha.intValue());
>+		}
> 		//graphics.rotate(rotation);
> 
> 		/* see bug #267397: paintFigure was historically not called, disabling
>@@ -238,14 +241,14 @@
> 	/**
> 	 * @since 3.5
> 	 */
>-	public int getAlpha() {
>+	public Integer getAlpha() {
> 		return alpha;
> 	}
> 
> 	/**
> 	 * @since 3.5
> 	 */
>-	public int getAntialias() {
>+	public Integer getAntialias() {
> 		return antialias;
> 	}
> 	
>@@ -366,24 +369,32 @@
> 	/**
> 	 * @since 3.5
> 	 */
>-	public void setAlpha(int value) {
>+	public void setAlpha(Integer value) {
> 		if(alpha != value) {
> 			alpha = value;
> 			repaint();
> 		}
> 	}
> 	
>+	public void setAlpha(int value) {
>+		setAlpha(Integer.valueOf(value));
>+	}
>+	
> 	/**
> 	 * @see org.eclipse.swt.graphics.GC#setAntialias(int)
> 	 * @param value
> 	 * @since 3.5
> 	 */
>-	public void setAntialias(int value) {
>+	public void setAntialias(Integer value) {
> 		if(antialias != value) {
> 			antialias = value;
> 			repaint();
> 		}
> 	}
>+	
>+	public void setAntialias(int value) {
>+		setAntialias(Integer.valueOf(value));
>+	}
> 
> 	/**
> 	 * Sets all line attributes at once.

Actions: View | Diff

Attachments on bug 276682: 136170 | 137349 | 137350 | 137390