Utiliser les gdkpixbuf

Version imprimable

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

Dans la fonction :

Code:

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:

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;
}

22/02/2010, 16h15
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 :)
22/02/2010, 16h47
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 ?
22/02/2010, 17h18
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.
22/02/2010, 18h00
gerald3d

Aie :cry: 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 :calim2:
22/02/2010, 18h03
gerald3d

Et voila!!!

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

Citation:

... 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 :piou: :mur:
23/02/2010, 10h47
liberforce

Sans compter que quand tu as du padding, il y en a sur toutes les lignes sauf la dernière...

Citation:

If you are doing memcpy() of raw pixbuf data, note that the last row in the pixbuf may not be as wide as the full rowstride, but rather just as wide as the pixel data needs to be. That is, it is unsafe to do memcpy (dest, pixels, rowstride * height) to copy a whole pixbuf. Use gdk_pixbuf_copy() instead, or compute the width in bytes of the last row as width * ((n_channels * bits_per_sample + 7) / 8).

Source: http://library.gnome.org/devel/gdk-p...dk-pixbuf.html
23/02/2010, 11h30
lowwa132

Citation:

Envoyé par gerald3d

Je souffre le martyre :piou: :mur:

Aller, courage :zoubi: