Discussion :

[pseudocode]

Les images TYPE_BYTE_GRAY sont gérées incorrectement par Java (les valeurs originales des pixels ne sont pas conservées): Bug ID: 5051418

Le workaround proposé sur le lien ci-dessus permet de corriger ce problème en créant une nouvelle image de type TYPE_BYTE_INDEXED. Malheureusement, ce format d'image provoque des effets de bords curieux dans MillieGui. En particulier les BorderExtenders décalent l'image (probleme avec la fonction "g2d.drawImage").

Je propose, lors du chargement, de convertir automatiquement les images TYPE_BYTE_GRAY en TYPE_RGB en utilisant le workaround:

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
BufferedImage bytegrayToRgb(BufferedImage input) {
	// Workaround bug ID: 5051418
	if(input.getType() == BufferedImage.TYPE_BYTE_GRAY) {
 
		// convert to TYPE_BYTE_INDEXED to have the correct values
		byte cmap[] = new byte[256];
		for (int i = 0; i < 256; i++) cmap[i] = (byte)i;
		IndexColorModel gray = new IndexColorModel(8, cmap.length, cmap, cmap, cmap);
		BufferedImage bg = new BufferedImage(input.getWidth(), input.getHeight(), BufferedImage.TYPE_BYTE_INDEXED, gray);
		Graphics2D g2d = bg.createGraphics(); 
		g2d.drawImage(input, AffineTransform.getTranslateInstance(0, 0), null);
		input.flush();
		g2d.dispose();
 
		// then convert to TYPE_RGB
		BufferedImage out = new BufferedImage(input.getWidth(), input.getHeight(), ColorSpace.TYPE_RGB);
		g2d = out.createGraphics();
		g2d.drawImage(bg, 0, 0, null);
		bg.flush();
		g2d.dispose();
		return out;
	}
	return input;
}

[millie]

En fait, quand je charge une image du disque, j'appelle :

MilleCommonUtils#convertIntoValidBufferedImage car il y avait des "bugs" avec les canaux alpha.

Donc a priori, ce serait là de le mettre ?

Code java :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
static public BufferedImage convertIntoValidBufferedImage(BufferedImage in) {
		//bugfix temporaire
		if(in.getType() ==0) {
			BufferedImage temp;
			if(in.getColorModel().hasAlpha())
				temp = new BufferedImage(in.getWidth(), in.getHeight(), BufferedImage.TYPE_INT_ARGB);
			else
				temp = new BufferedImage(in.getWidth(), in.getHeight(), BufferedImage.TYPE_INT_RGB);
			Graphics2D g = temp.createGraphics();
			g.drawImage(in, 0,0,null);
			g.dispose();
			return temp;
		}
		return in;
	}

[pseudocode]

En fait, quand je charge une image du disque, j'appelle :

MilleCommonUtils#convertIntoValidBufferedImage car il y avait des "bugs" avec les canaux alpha.

Donc a priori, ce serait là de le mettre ?

Oui surement. Je n'ai pas cherché dans ton source comment se faisait le chargement mais je te fais confiance.

[millie]

Bon, je l'ai mis là.

On va dire que c'est bon. Mes tests de non régression ont dit qu'il n'y avait pas de soucis