Discussion :

[lowwa132]

http://gtk.developpez.com/faq/?page=gdkpixbuf


Dans la fonction :

Code :

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
gboolean gdkpixbuf_set_colors_by_coordinates(GdkPixbuf *pixbuf, gint x, gint y, guchar red, guchar green, guchar blue)
{
  guchar *pixel=NULL;
  gint channel=0;
  gint width=0;
 
  if (!pixbuf) return FALSE;
  if (x<0 || y<0) return FALSE;
  if (x>gdk_pixbuf_get_width(pixbuf)) return FALSE;
  if (y>gdk_pixbuf_get_height(pixbuf)) return FALSE;
 
  pixel=gdk_pixbuf_get_pixels(pixbuf);
  channel=gdk_pixbuf_get_n_channels(pixbuf);
  width=gdk_pixbuf_get_width(pixbuf);
 
  pixel[(x*channel)+(y*width*channel)]   = red;
  pixel[(x*channel)+(y*width*channel)+1] = green;
  pixel[(x*channel)+(y*width*channel)+2] = blue;
 
  return TRUE;
}

Récupérer width ne marche pas à tous les coups !

Je viens de le tester, et il y a un décalage à chaque ligne qui devient de plus en plus important à mesure que l'on se déplace vers le bas de la fenêtre.

J'ai corrigé le problème en modifiant la fonction en allant chercher le rowstride de l'image :

Code :

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
25
gboolean gdkpixbuf_set_colors_by_coordinates(GdkPixbuf *pixbuf, gint x, gint y, guchar red, guchar green, guchar blue)
{
    guchar *pixels=NULL, *p=NULL;
    gint n_channels=0;
    gint width=0, rowstride=0;
 
    if (!pixbuf) return FALSE;
    if (x<0 || y<0) return FALSE;
    if (x>gdk_pixbuf_get_width(pixbuf)) return FALSE;
    if (y>gdk_pixbuf_get_height(pixbuf)) return FALSE;
 
    pixels=gdk_pixbuf_get_pixels(pixbuf);
    n_channels = gdk_pixbuf_get_n_channels (pixbuf);
    rowstride = gdk_pixbuf_get_rowstride (pixbuf);
 
    p = pixels + y * rowstride + x * n_channels;
 
    printf("%d\n", n_channels);
 
    p[0] = red;
    p[1] = green;
    p[2] = blue;
 
    return TRUE;
}

[teuf13]

J'allais demander quelle était la question avant de réaliser que le but du post était de signalé un bug dans la doc présente sur le site Il faut effectivement utiliser la row-stride et pas la largeur (width) pour accéder au tableau de pixels. Il faudrait que qqu'un corrige ça

[gerald3d]

Sans contredire ce qui est écrit et malgré que ce soit moi l'auteur de la fonction originale je cherche à comprendre pourquoi il y aurait un décalage à chaque ligne.

le rowstride est égale au nombre de pixels sur une ligne multiplié par le nombre de canaux.

Donc écrire y*rowstride ou y*width*channel revient au même.

Pour moi, mais je peux me tromper, les deux fonctions se valent. Quelqu'un verrait-il la coquille ?

[teuf13]

Oui, le rowstride te permet d'ajouter du "padding" (des octets à ignorer) à la fin de chaque ligne. Donc les deux valeurs que tu cites ne sont pas forcément égales, d'où l'intérêt d'avoir un attribut "rowstride" pour savoir la largeur exacte d'une ligne.

[gerald3d]

Aie ca me fait mal ce que tu dis. Ca remet toute ma programmation en question. Il me semblait que le tableau de guchar contenu dans un GdkPixbuf était linéaire, sans ajout de quelconques octets à ignorer en fin de chaque ligne. Faut que je relise la doc de plus prêt moi

[gerald3d]

Et voila!!!

La description dans la doc l'explique très bien :

... Rows in the image are stored top to bottom, and in each row pixels are stored from left to right. There may be padding at the end of a row. The "rowstride" value of a pixbuf, as returned by gdk_pixbuf_get_rowstride(), indicates the number of bytes between rows.

Je souffre le martyre