gdk_pixmap_create_from_data () et Ubuntu-10.0.4

Version imprimable

02/10/2010, 10h46
pvincent

1 pièce(s) jointe(s)

gdk_pixmap_create_from_data () et Ubuntu-10.0.4

Bonjour,
J'utilise sans problème gdk_pixmap_create_from_data () avec NULL comme premier argument mais en donnant la profondeur (depth), comme indiqué dans la doc:
drawable : a GdkDrawable, used to determine default values for the new pixmap. Can be NULL, if the depth is given.

Ceci, avec ma Debian Lenny et sous Windows (GTK+-2.16). Mais hier, j'ai mis à jour Ubuntu-8.0.4 vers Ubuntu-10.0.4 (comme proposé par le site) et depuis, j'ai le message:

Gdk-CRITICAL **: _gdk_drawable_get_source_drawable: assertion `GDK_IS_DRAWABLE (drawable)' failed

J'ai cherché dans http://bugzilla.gnome.org et je n'ai rien trouvé à ce sujet.

S'agit-il d'un bogue de la version 20 de GTK ou de Ubuntu?

Je dois dire que je ne suis pas un utilisateur régulier de Ubuntu et que je ne sais pas trop comment procéder pour savoir si c'est un bogue connu, ou s'il ne l'est pas, et comment le signaler.
Je joins un petit programme pour illustrer le problème.

Merci de votre attention,
P. Vincent

En effet ton programme fonctionne tout de même mais le message d'erreur apparait. Je suis aussi sous Ubuntu 10.04. Apparemment il considère que le premier paramètre ne peut pas être NULL.

Ceci évite le message d'erreur mais bien sûr n'explique pas pourquoi :
Code:

1 2 3 4 5 6 7 pixmap = gdk_pixmap_create_from_data (window->window //UBUNTU-10.0.4 window->window ,(char*) grace_icon_bits , grace_icon_width , grace_icon_height , gg_depth //UBUNTU-10.0.4 -1 ,&black ,&white);

03/10/2010, 09h33
pvincent

Merci de ta réponse rapide.
J'ai contourné la difficulté en utilisant le GdkDrawable, mais je pense que c'est un bogue de GTK.
03/10/2010, 09h51
gerald3d

Possible. Ou alors une erreur dans la documentation. Le premier argument n'accepte pas la valeur NULL.

A suivre...

C'est un bug de GTK introduit dans la version 2.18. La valeur de "drawable" n'est pas testée.

Comparer gdk_pixmap_new:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
GdkPixmap *
gdk_pixmap_new (GdkDrawable *drawable,
                gint         width,
                gint         height,
                gint         depth)
{
  GdkDrawable *source_drawable;
 
  if (drawable)
    source_drawable = _gdk_drawable_get_source_drawable (drawable);
  else
    source_drawable = NULL;
  return _gdk_pixmap_new (source_drawable, width, height, depth);
}

et gdk_pixmap_create_from_data:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
GdkPixmap*
gdk_pixmap_create_from_data (GdkDrawable    *drawable,
                             const gchar    *data,
                             gint            width,
                             gint            height,
                             gint            depth,
                             const GdkColor *fg,
                             const GdkColor *bg)
{
  GdkDrawable *source_drawable;

  source_drawable = _gdk_drawable_get_source_drawable (drawable);
  return _gdk_pixmap_create_from_data (source_drawable,
                                       data, width, height,
                                       depth, fg,bg);
}

Il aurait fallu écrire:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
GdkPixmap*
gdk_pixmap_create_from_data (GdkDrawable    *drawable,
                             const gchar    *data,
                             gint            width,
                             gint            height,
                             gint            depth,
                             const GdkColor *fg,
                             const GdkColor *bg)
{
  GdkDrawable *source_drawable;

  if (drawable)
    source_drawable = _gdk_drawable_get_source_drawable (drawable);
  else
    source_drawable = NULL;
  return _gdk_pixmap_create_from_data (source_drawable,
                                       data, width, height,
                                       depth, fg,bg);
}

Un volontaire pour ouvrir un bug sur le bugzilla ? Au passage, le code est ouvert, on peut voir rapidement quand un bug a été introduit ;)

04/10/2010, 18h43
Gamall

Vive l'open source :ccool:
04/10/2010, 19h07
gerald3d

Je me suis lancé : Bug 631332.
05/10/2010, 00h46
liberforce

Merci ;). Si personne ne s'y colle, j'irai commiter le correctif, ça me permettra de dérouiller mon accès aux serveurs :mouarf:
05/10/2010, 08h51
gerald3d

Je n'ai pas voulu ajouter ta solution pour ne pas me l'approprier. Il faut savoir rendre à César ce qui appartient à César ;)
05/10/2010, 09h52
liberforce

Ouais, enfin sur un patch d'un centaine de lignes, je veux bien, mais pour 3 lignes faut p'têt pas déconner, je vais pas les breveter non plus :mrgreen:. Je peux d'ailleurs vous annoncer que le patch a déjà été commité et le bug résolu :ccool:. Ah, la magie du libre.