Gestion des paramètres callback : effacement des données

Version imprimable

Voir 40 message(s) de cette discussion en une page

18/09/2015, 21h18
troumad

Gestion des paramètres callback : effacement des données

Bonjour

J'ai un problème avec l'effacement de la structure passée en callback.

Voici sa déclaration :

Code:

retour=(FenetrePerso *)malloc(sizeof(FenetrePerso));

Son passage en paramètre :

Code:

gtk_builder_connect_signals(retour->resume->builder,retour);

Je ferme la fenêtre :

Code:

gtk_window_close(GTK_WINDOW(retour->resume->window));

Puis j'aimerai effacer la structure :

Code:

g_free(f_p); /* le f_p est le paramètre retour récupéré par la fonction qui efface */

Si je ne commente pas la ligne avec le g_free, mon programme plante régulièrement (mais pas toujours) lors de la fermeture des fenêtres. Si je teste avec le déboguer de CodeBllock (sous linux) il m'indique que ça plante au lancement de gtk :

Code:

gtk_main();

Pourquoi ? Un erreur cachée dans mon programme ?
18/09/2015, 22h13
gerald3d

Peux-tu nous montrer la "fonction qui efface " ?

La structure :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
typedef struct
{
    signed short caract[9];
    signed short race;
    signed short * classe;
    signed short * niveau;
    unsigned long * XP;
    signed short version;
    signed short sexe;
    unsigned short age;
    gchar * nom;
    gchar * joueur;
    gchar * nom_fichier;
    signed short * pdv; /* pouvoir passer par des nombres négatifs dans les calculs et marquer par 0 ou -1 la fin de la liste */
    signed short tab_voleur[FEUILLE_LAR_VOL];
    signed short * competence;
    signed short * niv_competence;
    signed short * arme;
    signed short * niv_arme;
    signed short * mag_arme;
    signed short alignement;
    gchar * cheveux;
    gchar * peau;
    gchar * yeux;
    signed short taille;
    signed short poids;
    signed short * bouclier; /* pointe sur NULL si pas de bouclier, sinon pointe sur un entier : mag du bouclier */
    signed short * bague;    /* pointe sur NULL si pas de bague   , sinon pointe sur un entier : mag de la bague */
    signed short armure1;
    signed short mag_armure1;
    signed short armure2;
    signed short mag_armure2;
    gchar * commentaire;
    gchar * pantheon; /* on ne peut mettre un entier car on peut rajouter des items en cours d'utilisation */
    gchar * dieu;
    signed short ** livre_sorts;
    signed short ** sort_su; /* négatif si pas dans le livre de voyage, 0 si pas du tout, 1 pas regardé, 2 compris, 3 pas compris : voir # define */
    signed short * psi;
    signed short * niv_psi;
    signed short * natif_psi;
 
} perso;
 
struct _FenetrePerso
{
    struct widgets * resume;    /* gestion de la fenêtre résumée                */
    struct widgets * modif;     /* gestion de la fenêtre de modifications       */
    perso perso;
    char ** uri;    /* NULL pour un perso, liste des persos pour un groupe      */
};                  /* tableau de pointeurs vers les url, NULL pour le dernier  */
 
 
typedef struct _FenetrePerso Fenetr

Code pour effacer qui ne fait pas planter :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
void libere_fenetre_perso(FenetrePerso * f_p)
{
    unsigned short i;
 
    if (f_p->uri!=NULL)
    {
        for (i=0;f_p->uri[i]!=NULL;i++)
        {
            g_free(f_p->uri[i]);
        }
        g_free(f_p->uri);
        f_p->uri=NULL;
    }
    else
    {
    }
    libere_perso(&(f_p->perso));
    /*g_free(f_p);*/
}
 
void libere_perso(perso * pers)
{
    unsigned short i;
    g_free(pers->classe);
    g_free(pers->niveau);
    g_free(pers->XP);
    g_free(pers->nom);
    g_free(pers->joueur);
    g_free(pers->nom_fichier);
    g_free(pers->pdv);
    g_free(pers->competence);
    g_free(pers->niv_competence);
    g_free(pers->arme);
    g_free(pers->niv_arme);
    g_free(pers->mag_arme);
    g_free(pers->cheveux);
    g_free(pers->peau);
    g_free(pers->yeux);
    g_free(pers->bouclier);
    g_free(pers->bague);
    g_free(pers->commentaire);
    g_free(pers->pantheon);
    g_free(pers->dieu);
    if (pers->livre_sorts!=NULL)
    {
        for (i=0;i<9;i++)
        {
            g_free(pers->livre_sorts[i]);
        }
        g_free(pers->livre_sorts);
        pers->livre_sorts=NULL;
    }
    else
    {
    }
    if (pers->sort_su!=NULL)
    {
        for (i=0;i<9;i++)
        {
            g_free(pers->sort_su[i]);
        }
        g_free(pers->sort_su);
        pers->sort_su=NULL;
    }
    else
    {
    }
    g_free(pers->psi); /* pas enore géré */
    g_free(pers->niv_psi);
    g_free(pers->natif_psi);
}
 
 
 
void init_perso(perso * pers)
{
	pers->nom_fichier=NULL;
	pers->classe=NULL;
	pers->niveau=NULL;
	pers->XP=NULL;
	pers->pdv=NULL;
	pers->joueur=NULL;
	pers->age=0;
	pers->version=1;
	pers->sexe=0;
	pers->nom=g_strdup(""); /* ne pas avoir NULL pour faire la différence avec un perso qui arrive d'un fichier à lire */
	pers->competence=NULL;
	pers->niv_competence=NULL;
	pers->arme=NULL;
	pers->niv_arme=NULL;
	pers->mag_arme=NULL;
	pers->alignement=4;
	pers->race=0;
	pers->cheveux=NULL;
	pers->peau=NULL;
	pers->yeux=NULL;
	pers->poids=0;
	pers->taille=0;
	pers->bouclier=NULL;
	pers->bague=NULL;
	pers->armure1=0;
	pers->mag_armure1=0;
	pers->armure2=0;
	pers->mag_armure2=0;
	pers->commentaire=NULL;
	pers->pantheon=NULL;
	pers->dieu=NULL;
	pers->livre_sorts=NULL;
	pers->sort_su=NULL;
	pers->psi=NULL;
	pers->niv_psi=NULL;
	pers->natif_psi=NULL;
}

19/09/2015, 09h36
gerald3d

Après une première lecture rapide je n'aime pas trop le fait de mettre à NULL un pointeur transmis en paramètre (f_p dans void libere_fenetre_perso(FenetrePerso * f_p)).

As-tu tenté d'utiliser Gdb sur ton application pour voir un peu les warnings ?
19/09/2015, 12h00
troumad
Il me semble que j'ai modifié le code juste avant que tu ne postes. J'ai même enlevé la mise à NULL du pointeur qui était inutile. Par contre, je l'ai placé juste après l'appel de la fonction.
Code:

1 2 libere_fenetre_perso(retour); retour=NULL;
Je crois que tu m'as déjà parlé de Gdb... Que n'est pas le gain en décibel ;) j'avoue que ce nom me rappelle quelque chose, mais que je ne sais pas du tout comment on s'en sert. Je n'ai aucun warning à ce niveau lors de la compilation, pourtant, je demande de les avoir tous. Mon seul warning est incompréhensible. Il me dit que j'initialise une variable pour rien alors qu'elle est utilisée par la suite dans la fonction :(
19/09/2015, 14h28
gerald3d

je ne comprends pas pourquoi tu mets à NULL retour. Ce pointeur pointe toujours sur un espace alloué. Tu vas avoir une fuite mémoire ici.

Pour Gdb tu peux l'utiliser tout simplement. Tu ajoutes -g lors de la compilation. Puis tu lances ton exécutable avec Gdb : gdb mon_programme.
Tu accèdes alors à un prompt. Tu lances depuis ce prompt "run". Ton programme se lance. Une fois qu'il a planté tu reviens au prompt. Tu peux alors taper "where" pour savoir où le programme a planté. Il te donnera la ligne exacte dans ton code source. Ensuite tu tapes toujours dans le prompt "continue" ce qui va terminer ton programme qui pour l'instant est bloqué. Enfin tu tapes "quit" pour sortir de Gdb.

Il existe bien d'autres possibilités mais cette première utilisation te donne l'essentiel : la ligne qui pose problème.

J'ai la même info qu'avec de débogueur de code::blocks. Je mets tout de même toutes les infos, même s'il me semble que les premières sont inutiles.

Code:

1
2
3
4
5
6
7
8
9
10
Program received signal SIGSEGV, Segmentation fault.
0x00007ffff5b2a6c0 in g_source_callback_ref () from /lib64/libglib-2.0.so.0
Missing separate debuginfos, use: dnf debuginfo-install gvfs-1.25.91-2.mga6.x86_64 lib64asyncns0-0.8-8.mga5.x86_64 lib64atk-bridge2.0_0-2.17.90-1.mga6.x86_64 lib64atk1.0_0-2.16.0-2.mga6.x86_64 lib64atspi0-2.17.90-2.mga6.x86_64 lib64bzip2_1-1.0.6-8.mga6.x86_64 lib64cairo2-1.14.2-1.mga6.x86_64 lib64canberra-gtk3_0-0.30-7.mga5.x86_64 lib64canberra0-0.30-7.mga5.x86_64 lib64croco0.6_3-0.6.8-7.mga6.x86_64 lib64datrie1-0.2.8-3.mga5.x86_64 lib64dbus1_3-1.8.16-2.mga6.x86_64 lib64drm2-2.4.65-1.mga6.x86_64 lib64epoxy0-1.3.1-1.mga6.x86_64 lib64expat1-2.1.0-11.mga6.x86_64 lib64ffi6-3.2.1-4.mga6.x86_64 lib64flac8-1.3.1-3.mga6.x86_64 lib64fontconfig1-2.11.94-1.mga6.x86_64 lib64freetype6-2.6-2.mga6.x86_64 lib64gbm1-10.6.7-1.mga6.tainted.x86_64 lib64gcrypt20-1.6.4-1.mga6.x86_64 lib64gdk_pixbuf2.0_0-2.31.7-1.mga6.x86_64 lib64gio2.0_0-2.45.8-1.mga6.x86_64 lib64glapi0-10.6.7-1.mga6.tainted.x86_64 lib64glib2.0_0-2.45.8-1.mga6.x86_64 lib64gpg-error0-1.20-1.mga6.x86_64 lib64graphite2_3-1.2.4-3.mga5.x86_64 lib64gtk+3_0-3.17.9-1.mga6.x86_64 lib64harfbuzz0-0.9.41-2.mga6.x86_64 lib64json2-0.12-3.mga5.x86_64 lib64ltdl7-2.4.2-14.mga6.x86_64 lib64lzma5-5.2.1-2.mga6.x86_64 lib64mesaegl1-10.6.7-1.mga6.tainted.x86_64 lib64mesagl1-10.6.7-1.mga6.tainted.x86_64 lib64ogg0-1.3.2-3.mga5.x86_64 lib64pango1.0_0-1.37.5-1.mga6.x86_64 lib64pcre1-8.37-5.mga6.x86_64 lib64pixman1_0-0.33.2-2.mga6.x86_64 lib64png16_16-1.6.18-1.mga6.x86_64 lib64pulseaudio0-6.0-2.mga6.x86_64 lib64pulsecommon6.0-6.0-2.mga6.x86_64 lib64rsvg2_2-2.40.10-1.mga6.x86_64 lib64sndfile1-1.0.25-9.mga5.x86_64 lib64systemd0-225-3.mga6.x86_64 lib64tdb1-1.3.6-2.mga6.x86_64 lib64thai0-0.1.21-3.mga5.x86_64 lib64vorbis0-1.3.5-1.mga5.x86_64 lib64vorbisenc2-1.3.5-1.mga5.x86_64 lib64vorbisfile3-1.3.5-1.mga5.x86_64 lib64wayland-client0-1.8.1-2.mga6.x86_64 lib64wayland-cursor0-1.8.1-2.mga6.x86_64 lib64wayland-egl1-10.6.7-1.mga6.tainted.x86_64 lib64wayland-server0-1.8.1-2.mga6.x86_64 lib64wrap0-7.6-46.mga5.x86_64 lib64x11-xcb1-1.6.3-1.mga6.x86_64 lib64x11_6-1.6.3-1.mga6.x86_64 lib64xau6-1.0.8-6.mga6.x86_64 lib64xcb-dri2_0-1.11-5.mga6.x86_64 lib64xcb-dri3_0-1.11-5.mga6.x86_64 lib64xcb-glx0-1.11-5.mga6.x86_64 lib64xcb-present_0-1.11-5.mga6.x86_64 lib64xcb-render0-1.11-5.mga6.x86_64 lib64xcb-shm0-1.11-5.mga6.x86_64 lib64xcb-sync1-1.11-5.mga6.x86_64 lib64xcb-xfixes0-1.11-5.mga6.x86_64 lib64xcb1-1.11-5.mga6.x86_64 lib64xcomposite1-0.4.4-7.mga5.x86_64 lib64xcursor1-1.1.14-5.mga5.x86_64 lib64xdamage1-1.1.4-7.mga5.x86_64 lib64xdmcp6-1.1.2-2.mga6.x86_64 lib64xext6-1.3.3-3.mga5.x86_64 lib64xfixes3-5.0.1-5.mga5.x86_64 lib64xi6-1.7.4-3.mga5.x86_64 lib64xinerama1-1.1.3-5.mga5.x86_64 lib64xkbcommon0-0.5.0-2.mga6.x86_64 lib64xml2_2-2.9.2-3.mga6.x86_64 lib64xrandr2-1.5.0-1.mga6.x86_64 lib64xrender1-0.9.9-1.mga6.x86_64 lib64xshmfence1-1.2-1.mga6.x86_64 lib64xxf86vm1-1.1.4-1.mga6.x86_64 lib64zip2-0.11.2-4.mga5.x86_64 lib64zlib1-1.2.8-8.mga6.x86_64 libgcc1-5.2.1-0.20150811.6.mga6.x86_64 libstdc++6-5.2.1-0.20150811.6.mga6.x86_64
(gdb) where
#0  0x00007ffff5b2a6c0 in g_source_callback_ref () at /lib64/libglib-2.0.so.0
#1  0x00007ffff5b2df38 in g_main_context_dispatch () at /lib64/libglib-2.0.so.0
#2  0x00007ffff5b2e318 in g_main_context_iterate.isra () at /lib64/libglib-2.0.so.0
#3  0x00007ffff5b2e632 in g_main_loop_run () at /lib64/libglib-2.0.so.0
#4  0x00007ffff76a10b5 in gtk_main () at /lib64/libgtk-3.so.0
#5  0x0000000000419d71 in main (argc=1, argv=0x7fffffffda88) at /home/troumad/Documents/glade/test_menu1/main.c:55

19/09/2015, 22h02
gerald3d

Hum, effectivement ce n'est pas très parlant. Il me faudrait le code complet pour pouvoir débogguer correctement.

Tu es sûr ? Regarde :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 [troumad@localhost test_menu1]$ wc -l *.c 126 chaine.c 1153 dieux.c 334 envp.c 484 gestion_fenetre_perso.c 54 gestion_glade.c 786 groupe.c 2030 lecture_fichier.c 422 lire_odc.c 67 main.c 997 menu.c 51 messages.c 297 modif_perso.c 936 ouverture.c 399 sorts.c 286 style.c 363 tableaux.c 607 tirage.c 227 utile.c 766 xml.c 135 zip.c 10520 total
Parce que, je veux bien de te passer, je commente au fur et à mesure. mais comme depuis que je suis dessus, il y a eu plusieurs périodes de plusieurs mois de relâche, parfois, moi-même, je ne sais plus trop ce que c'est...

20/09/2015, 09h24
gerald3d

Envoies-moi un MP si tu veux bien me passer tes codes sources.
20/09/2015, 09h47
troumad

Tu peux le trouver là : http://troumad.org/OOo/perso4_1.zip (la ligne qui fait planter est la ligne 389 de gestion_fenetre_perso.c . Là, elle est commentée pour que le programme tourne et que je puisse travailler la suite.

La gestion des fenêtres qui posent problème se trouve dans gestion_fenetre_perso.c avec des appel à ouverture.c et à groupe.c .
Pour arriver facilement au problème, je te conseille d'ouvrir le fichier samuel.grp que tu trouveras ici : http://troumad.org/OOo/samuel.zip . Ce fichier .grp (groupe) ouvrira les autres fichiers .pers (personnage du groupe). Et c'est à la fermeture des personnages que ça plante. Mais, pas toujours au même moment.

Pour info, c'est la nouvelle version de http://troumad.org/Mes_regles/perso.php que j'essaie de mettre au point avec glade (un peu) et moins d'obsolescence (uniquement celles générées par glade). Les fichiers que je passe (grp et pers) sont entièrement compatibles avec l'ancienne version car la nouvelle ne peut pas encore générer de ".pers" .

Bon courage...

J'ai pris le temps de télécharger ton code. Après ouverture de l'archive, une première compilation donne ceci :

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
26
27
28
29
30
groupe.c:207:12: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
     char * nom, * pt_n,  * tmp;
            ^
In file included from groupe.c:21:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
groupe.c: In function ‘enregitre_perso_sous’:
groupe.c:730:61: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void enregitre_perso_sous(GtkButton *button, FenetrePerso * perso)
                                                             ^
In file included from groupe.c:21:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
groupe.c: In function ‘enregitre_perso’:
groupe.c:735:56: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void enregitre_perso(GtkButton *button, FenetrePerso * perso)
                                                        ^
In file included from groupe.c:21:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
groupe.c: In function ‘nouveau_membre_groupe’:
groupe.c:740:73: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 signed short nouveau_membre_groupe(FenetrePerso * fenetre_perso, char * nom,unsigned short 
                                                                         ^
In file included from groupe.c:21:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];

Il faudrait déjà commencer par régler ce problème. Déclarer des variables de même nom posent de sérieux problèmes.

J'ai relancé une compilation avec l'option -Wextra ajoutée à ton Makefile. Voila ce que ca donne :

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
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
-*- mode: compilation; default-directory: "~/Projects/troomad/" -*-
Compilation started at Sun Sep 20 13:56:57
 
make -k 
gcc -o lecture_fichier.o -c lecture_fichier.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
lecture_fichier.c: In function ‘classe_entier’:
lecture_fichier.c:1916:37: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 unsigned short classe_entier(char * nom)
                                     ^
In file included from lecture_fichier.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
lecture_fichier.c: In function ‘race_entier’:
lecture_fichier.c:1940:35: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 unsigned short race_entier(char * nom)
                                   ^
In file included from lecture_fichier.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
lecture_fichier.c: In function ‘competence_entier’:
lecture_fichier.c:1963:41: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 unsigned short competence_entier(char * nom)
                                         ^
In file included from lecture_fichier.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
lecture_fichier.c: In function ‘arme_entier’:
lecture_fichier.c:1986:35: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 unsigned short arme_entier(char * nom)
                                   ^
In file included from lecture_fichier.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
lecture_fichier.c: In function ‘armure_entier’:
lecture_fichier.c:2009:37: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 unsigned short armure_entier(char * nom)
                                     ^
In file included from lecture_fichier.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gcc -o gestion_glade.o -c gestion_glade.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gestion_glade.c: In function ‘ouverture_glade’:
gestion_glade.c:8:29: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 void ouverture_glade(char * nom, unsigned short connect)
                             ^
In file included from gestion_glade.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gestion_glade.c: In function ‘ouverture_glade_retour’:
gestion_glade.c:19:48: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 struct widgets * ouverture_glade_retour(char * nom, unsigned short connect)
                                                ^
In file included from gestion_glade.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gestion_glade.c: In function ‘ferme_glade’:
gestion_glade.c:51:29: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void ferme_glade(GtkWidget *ChildWidget , struct widgets * widgets)
                             ^
gcc -o chaine.o -c chaine.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
chaine.c: In function ‘simplifie_uri’:
chaine.c:99:28: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 char * simplifie_uri(char *nom)
                            ^
In file included from chaine.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gcc -o xml.o -c xml.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gcc -o envp.o -c envp.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
envp.c: In function ‘envp_init’:
envp.c:72:27: warning: unused parameter ‘param’ [-Wunused-parameter]
 char * envp_init(gpointer param)
                           ^
gcc -o sorts.o -c sorts.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
sorts.c: In function ‘lire_struct_sort’:
sorts.c:89:25: warning: variable ‘l’ set but not used [-Wunused-but-set-variable]
  unsigned short i=0,k=0,l=0;
                         ^
sorts.c: In function ‘sort_entier’:
sorts.c:374:33: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 signed short sort_entier(char * nom,signed short niveau,signed short version)
                                 ^
In file included from sorts.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gcc -o lire_odc.o -c lire_odc.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gcc -o groupe.o -c groupe.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
groupe.c: In function ‘enregistre_groupe’:
groupe.c:85:32: warning: comparison is always true due to limited range of data type [-Wtype-limits]
     for(i=strlen(chemin_gr)-1;i>=0 && chemin_gr[i]!=SEPARATEUR;i--);
                                ^
groupe.c:86:10: warning: comparison is always true due to limited range of data type [-Wtype-limits]
     if (i>=0) /* on ne retient que le chemin, le nom est au bout */
          ^
groupe.c:96:33: warning: comparison is always true due to limited range of data type [-Wtype-limits]
         for(j=strlen(chaine)-1;j>=0 && chaine[j]!=SEPARATEUR;j--);
                                 ^
groupe.c:104:14: warning: comparison is always true due to limited range of data type [-Wtype-limits]
         if (j>=0) /* on ne retient que le chemin */
              ^
groupe.c:36:35: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void enregistre_groupe(GtkWidget *ChildWidget , FenetrePerso * fenetre)
                                   ^
groupe.c: In function ‘enregistre_groupe_sous’:
groupe.c:186:40: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void enregistre_groupe_sous(GtkWidget *ChildWidget, FenetrePerso * fenetre)
                                        ^
groupe.c: In function ‘nouveau_nom_groupe’:
groupe.c:207:12: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
     char * nom, * pt_n,  * tmp;
            ^
In file included from groupe.c:21:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
groupe.c:205:37: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void  nouveau_nom_groupe(GtkWidget *ChildWidget, struct callback_nv_nom * c_b)
                                     ^
groupe.c: In function ‘ajouter_perso_groupe’:
groupe.c:245:38: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void ajouter_perso_groupe(GtkWidget *ChildWidget, FenetrePerso * fenetre)
                                      ^
groupe.c: In function ‘ferme_nv_nom’:
groupe.c:298:30: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void ferme_nv_nom(GtkWidget *ChildWidget , GtkWidget * fenetre)
                              ^
groupe.c: In function ‘callback_suppr_perso_groupe’:
groupe.c:707:45: warning: unused parameter ‘button’ [-Wunused-parameter]
 void callback_suppr_perso_groupe(GtkButton *button, perso_groupe * p_gr)
                                             ^
groupe.c: In function ‘enregitre_perso_sous’:
groupe.c:730:61: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void enregitre_perso_sous(GtkButton *button, FenetrePerso * perso)
                                                             ^
In file included from groupe.c:21:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
groupe.c:730:38: warning: unused parameter ‘button’ [-Wunused-parameter]
 void enregitre_perso_sous(GtkButton *button, FenetrePerso * perso)
                                      ^
groupe.c:730:61: warning: unused parameter ‘perso’ [-Wunused-parameter]
 void enregitre_perso_sous(GtkButton *button, FenetrePerso * perso)
                                                             ^
groupe.c: In function ‘enregitre_perso’:
groupe.c:735:56: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void enregitre_perso(GtkButton *button, FenetrePerso * perso)
                                                        ^
In file included from groupe.c:21:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
groupe.c:735:33: warning: unused parameter ‘button’ [-Wunused-parameter]
 void enregitre_perso(GtkButton *button, FenetrePerso * perso)
                                 ^
groupe.c:735:56: warning: unused parameter ‘perso’ [-Wunused-parameter]
 void enregitre_perso(GtkButton *button, FenetrePerso * perso)
                                                        ^
groupe.c: In function ‘nouveau_membre_groupe’:
groupe.c:740:73: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 signed short nouveau_membre_groupe(FenetrePerso * fenetre_perso, char * nom,unsigned short 
                                                                         ^
In file included from groupe.c:21:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gcc -o zip.o -c zip.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gcc -o tirage.o -c tirage.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
tirage.c: In function ‘applique_menu’:
tirage.c:43:20: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
     FenetrePerso * perso;
                    ^
In file included from tirage.c:24:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
tirage.c:41:31: warning: unused parameter ‘wid_appel’ [-Wunused-parameter]
 void applique_menu(GtkWidget *wid_appel, struct widgets * widgets)
                               ^
tirage.c: In function ‘init_caract_voleur’:
tirage.c:471:40: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void init_caract_voleur(FenetrePerso * perso)
                                        ^
In file included from tirage.c:24:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
gcc -o ouverture.o -c ouverture.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
ouverture.c: In function ‘ouverture’:
ouverture.c:28:27: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void ouverture(GtkWidget *ChildWidget, struct widgets * _widgets)
                           ^
ouverture.c:28:57: warning: unused parameter ‘_widgets’ [-Wunused-parameter]
 void ouverture(GtkWidget *ChildWidget, struct widgets * _widgets)
                                                         ^
ouverture.c: In function ‘info_fichier’:
ouverture.c:35:12: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
     gchar *nom,ch[LONG];
            ^
In file included from ouverture.c:22:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
ouverture.c: In function ‘ouvre_perso’:
ouverture.c:115:35: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 FenetrePerso * ouvre_perso(char * nom)
                                   ^
In file included from ouverture.c:22:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
ouverture.c:119:20: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
     FenetrePerso * perso=NULL;
                    ^
In file included from ouverture.c:22:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
ouverture.c:651:39: warning: comparison is always true due to limited range of data type [-Wtype-limits]
                                 if (nv>=0 && nv<9 && perso->perso.livre_sorts[nv]!=NULL)
                                       ^
ouverture.c: At top level:
ouverture.c:903:7: warning: no previous declaration for ‘ouverture_liste’ [-Wmissing-declarations]
 void  ouverture_liste(GtkWidget *ChildWidget , struct widgets * widgets)
       ^
ouverture.c:918:6: warning: no previous declaration for ‘ouvre_recent’ [-Wmissing-declarations]
 void ouvre_recent (GtkRecentChooser *chooser, gpointer          user_data)
      ^
ouverture.c: In function ‘ouvre_recent’:
ouverture.c:918:65: warning: unused parameter ‘user_data’ [-Wunused-parameter]
 void ouvre_recent (GtkRecentChooser *chooser, gpointer          user_data)
                                                                 ^
ouverture.c: In function ‘callback_ouvre_perso’:
ouverture.c:930:38: warning: unused parameter ‘button’ [-Wunused-parameter]
 void callback_ouvre_perso(GtkButton *button, char * ch)
                                      ^
gcc -o menu.o -c menu.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
menu.c: In function ‘complete_menu’:
menu.c:112:34: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void complete_menu (GtkMenuItem *menuitem, struct widgets *widgets)
                                  ^
menu.c: In function ‘quite_choix’:
menu.c:428:29: warning: unused parameter ‘widget’ [-Wunused-parameter]
 void quite_choix(GtkWidget *widget, struct widgets * widgets)
                             ^
menu.c: At top level:
menu.c:674:6: warning: no previous declaration for ‘change_choix_sexe’ [-Wmissing-declarations]
 void change_choix_sexe(GtkRadioButton *widget, struct widgets * widgets)
      ^
menu.c: In function ‘Change_Tirage’:
menu.c:840:34: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void Change_Tirage (GtkMenuItem *menuitem, struct widgets * widgets)
                                  ^
menu.c:840:61: warning: unused parameter ‘widgets’ [-Wunused-parameter]
 void Change_Tirage (GtkMenuItem *menuitem, struct widgets * widgets)
                                                             ^
menu.c: In function ‘Change_Sexe’:
menu.c:850:32: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void Change_Sexe (GtkMenuItem *menuitem, struct widgets * widgets)
                                ^
menu.c:850:59: warning: unused parameter ‘widgets’ [-Wunused-parameter]
 void Change_Sexe (GtkMenuItem *menuitem, struct widgets * widgets)
                                                           ^
menu.c: In function ‘Change_Pdv’:
menu.c:855:31: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void Change_Pdv (GtkMenuItem *menuitem, struct widgets * widgets)
                               ^
menu.c:855:58: warning: unused parameter ‘widgets’ [-Wunused-parameter]
 void Change_Pdv (GtkMenuItem *menuitem, struct widgets * widgets)
                                                          ^
menu.c: At top level:
menu.c:975:6: warning: no previous declaration for ‘Change_Niv’ [-Wmissing-declarations]
 void Change_Niv(GtkMenuItem *menuitem, struct widgets * widgets)
      ^
menu.c: In function ‘Change_Niv’:
menu.c:975:30: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void Change_Niv(GtkMenuItem *menuitem, struct widgets * widgets)
                              ^
gcc -o main.o -c main.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
main.c: In function ‘cb_quit’:
main.c:62:30: warning: unused parameter ‘widget’ [-Wunused-parameter]
 gboolean cb_quit (GtkWidget *widget, GdkEvent  *event, struct widgets *widgets)
                              ^
main.c:62:49: warning: unused parameter ‘event’ [-Wunused-parameter]
 gboolean cb_quit (GtkWidget *widget, GdkEvent  *event, struct widgets *widgets)
                                                 ^
main.c:62:72: warning: unused parameter ‘widgets’ [-Wunused-parameter]
 gboolean cb_quit (GtkWidget *widget, GdkEvent  *event, struct widgets *widgets)
                                                                        ^
gcc -o classe.o -c classe.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gcc -o tableaux.o -c tableaux.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gcc -o utile.o -c utile.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
utile.c: In function ‘gtk_get_widget_by_name’:
utile.c:129:64: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
  GtkWidget * gtk_get_widget_by_name(GtkContainer * pere,char * nom)
                                                                ^
In file included from utile.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gcc -o messages.o -c messages.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
messages.c: In function ‘GPL’:
messages.c:3:24: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void GPL (GtkMenuItem *menuitem, struct widgets * _widgets)
                        ^
messages.c:3:51: warning: unused parameter ‘_widgets’ [-Wunused-parameter]
 void GPL (GtkMenuItem *menuitem, struct widgets * _widgets)
                                                   ^
messages.c: In function ‘Explication’:
messages.c:8:33: warning: unused parameter ‘menuitem’ [-Wunused-parameter]
 void Explication  (GtkMenuItem *menuitem, struct widgets * _widgets)
                                 ^
messages.c:8:60: warning: unused parameter ‘_widgets’ [-Wunused-parameter]
 void Explication  (GtkMenuItem *menuitem, struct widgets * _widgets)
                                                            ^
gcc -o dieux.o -c dieux.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
dieux.c: In function ‘Modif_dieu’:
dieux.c:390:29: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void Modif_dieu (GtkWidget *ChildWidget, struct widgets * _widgets)
                             ^
dieux.c:390:59: warning: unused parameter ‘_widgets’ [-Wunused-parameter]
 void Modif_dieu (GtkWidget *ChildWidget, struct widgets * _widgets)
                                                           ^
dieux.c: In function ‘Appli_dieu’:
dieux.c:525:12: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
     char * nom=NULL;
            ^
In file included from dieux.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
dieux.c: In function ‘Modif_Sphere’:
dieux.c:659:30: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void Modif_Sphere(GtkWidget *ChildWidget, struct widgets *widgets)
                              ^
dieux.c: At top level:
dieux.c:768:6: warning: no previous declaration for ‘finir_dieu’ [-Wmissing-declarations]
 void finir_dieu(GtkWidget *ChildWidget, struct widgets *widgets)
      ^
dieux.c: In function ‘finir_dieu’:
dieux.c:768:28: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void finir_dieu(GtkWidget *ChildWidget, struct widgets *widgets)
                            ^
dieux.c: At top level:
dieux.c:774:6: warning: no previous declaration for ‘annuler_sphere’ [-Wmissing-declarations]
 void annuler_sphere(GtkWidget *ChildWidget, struct widgets *widgets)
      ^
dieux.c: In function ‘aucune’:
dieux.c:798:24: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void aucune(GtkWidget *ChildWidget, struct widgets *widgets)
                        ^
dieux.c: In function ‘Majeure’:
dieux.c:803:25: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void Majeure(GtkWidget *ChildWidget, struct widgets *widgets)
                         ^
dieux.c: In function ‘mineure’:
dieux.c:808:25: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void mineure(GtkWidget *ChildWidget, struct widgets *widgets)
                         ^
dieux.c: In function ‘pantheon_entier’:
dieux.c:1041:37: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 signed short pantheon_entier(char * nom)
                                     ^
In file included from dieux.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
dieux.c: In function ‘dieu_entier’:
dieux.c:1065:33: warning: declaration of ‘nom’ shadows a global declaration [-Wshadow]
 signed short dieu_entier(char * nom,signed short panth)
                                 ^
In file included from dieux.c:1:0:
include.h:412:22: warning: shadowed declaration is here [-Wshadow]
 extern const gchar * nom[];
                      ^
gcc -o modif_perso.o -c modif_perso.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
modif_perso.c: In function ‘modif_perso’:
modif_perso.c:10:52: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void modif_perso(GtkWidget * appel, FenetrePerso * perso)
                                                    ^
In file included from modif_perso.c:1:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
modif_perso.c:10:30: warning: unused parameter ‘appel’ [-Wunused-parameter]
 void modif_perso(GtkWidget * appel, FenetrePerso * perso)
                              ^
modif_perso.c: At top level:
modif_perso.c:195:6: warning: no previous declaration for ‘ferme_modif’ [-Wmissing-declarations]
 void ferme_modif(GtkWidget *widget, FenetrePerso * perso)
      ^
modif_perso.c: In function ‘ferme_modif’:
modif_perso.c:195:52: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void ferme_modif(GtkWidget *widget, FenetrePerso * perso)
                                                    ^
In file included from modif_perso.c:1:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
modif_perso.c:195:29: warning: unused parameter ‘widget’ [-Wunused-parameter]
 void ferme_modif(GtkWidget *widget, FenetrePerso * perso)
                             ^
modif_perso.c: At top level:
modif_perso.c:282:6: warning: no previous declaration for ‘ferme_modif_perso’ [-Wmissing-declarations]
 void ferme_modif_perso(GtkWidget *wid, FenetrePerso * perso)
      ^
modif_perso.c: In function ‘ferme_modif_perso’:
modif_perso.c:282:55: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void ferme_modif_perso(GtkWidget *wid, FenetrePerso * perso)
                                                       ^
In file included from modif_perso.c:1:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
gcc -o gestion_fenetre_perso.o -c gestion_fenetre_perso.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
gestion_fenetre_perso.c: In function ‘ferme_fenetre_perso’:
gestion_fenetre_perso.c:184:57: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void ferme_fenetre_perso(GtkWidget *wid, FenetrePerso * perso)
                                                         ^
In file included from gestion_fenetre_perso.c:1:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
gestion_fenetre_perso.c: In function ‘destroy_fenetre_perso’:
gestion_fenetre_perso.c:219:59: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void destroy_fenetre_perso(GtkWidget *wid, FenetrePerso * perso)
                                                           ^
In file included from gestion_fenetre_perso.c:1:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
gestion_fenetre_perso.c: In function ‘affiche_resume’:
gestion_fenetre_perso.c:226:36: warning: declaration of ‘perso’ shadows a global declaration [-Wshadow]
 void affiche_resume(FenetrePerso * perso)
                                    ^
In file included from gestion_fenetre_perso.c:1:0:
include.h:377:3: warning: shadowed declaration is here [-Wshadow]
 } perso;
   ^
gestion_fenetre_perso.c: In function ‘nouveau’:
gestion_fenetre_perso.c:367:25: warning: unused parameter ‘wid’ [-Wunused-parameter]
 void nouveau(GtkWidget *wid, struct widgets * param)
                         ^
gestion_fenetre_perso.c:367:47: warning: unused parameter ‘param’ [-Wunused-parameter]
 void nouveau(GtkWidget *wid, struct widgets * param)
                                               ^
gcc -o style.o -c style.c -Wshadow  -Winit-self  -Wredundant-decls  -Wcast-align  -Wundef  -Wfloat-equal  -Winline  -Wunreachable-code  -Wmissing-declarations  -Wmissing-include-dirs  -Wswitch-enum  -Wswitch-default  -Wmain  -Wall  `pkg-config gtk+-3.0 --cflags` -export-dynamic -Wextra
style.c:3:10: warning: redundant redeclaration of ‘style1’ [-Wredundant-decls]
 gboolean style1(GtkWidget *ChildWidget, cairo_t *cr);
          ^
In file included from style.c:1:0:
include.h:623:10: note: previous declaration of ‘style1’ was here
 gboolean style1(GtkWidget *ChildWidget, cairo_t *cr);
          ^
style.c:4:10: warning: redundant redeclaration of ‘style2’ [-Wredundant-decls]
 gboolean style2(GtkWidget *ChildWidget, cairo_t *cr);
          ^
In file included from style.c:1:0:
include.h:624:10: note: previous declaration of ‘style2’ was here
 gboolean style2(GtkWidget *ChildWidget, cairo_t *cr);
          ^
style.c:5:10: warning: redundant redeclaration of ‘style3’ [-Wredundant-decls]
 gboolean style3(GtkWidget *ChildWidget, cairo_t *cr);
          ^
In file included from style.c:1:0:
include.h:625:10: note: previous declaration of ‘style3’ was here
 gboolean style3(GtkWidget *ChildWidget, cairo_t *cr);
          ^
style.c:6:10: warning: redundant redeclaration of ‘style4’ [-Wredundant-decls]
 gboolean style4(GtkWidget *ChildWidget, cairo_t *cr);
          ^
In file included from style.c:1:0:
include.h:626:10: note: previous declaration of ‘style4’ was here
 gboolean style4(GtkWidget *ChildWidget, cairo_t *cr);
          ^
style.c:7:10: warning: redundant redeclaration of ‘style5’ [-Wredundant-decls]
 gboolean style5(GtkWidget *ChildWidget, cairo_t *cr);
          ^
In file included from style.c:1:0:
include.h:627:10: note: previous declaration of ‘style5’ was here
 gboolean style5(GtkWidget *ChildWidget, cairo_t *cr);
          ^
style.c:8:10: warning: redundant redeclaration of ‘style_race’ [-Wredundant-decls]
 gboolean style_race(GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets);
          ^
In file included from style.c:1:0:
include.h:628:10: note: previous declaration of ‘style_race’ was here
 gboolean style_race(GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets);
          ^
style.c:9:10: warning: redundant redeclaration of ‘style_classe’ [-Wredundant-decls]
 gboolean style_classe(GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets);
          ^
In file included from style.c:1:0:
include.h:629:10: note: previous declaration of ‘style_classe’ was here
 gboolean style_classe(GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets);
          ^
style.c:10:6: warning: redundant redeclaration of ‘bord’ [-Wredundant-decls]
 void bord (GtkWidget *ChildWidget, cairo_t *cr);
      ^
In file included from style.c:1:0:
include.h:622:6: note: previous declaration of ‘bord’ was here
 void bord (GtkWidget *ChildWidget, cairo_t *cr);
      ^
style.c:11:6: warning: redundant redeclaration of ‘style’ [-Wredundant-decls]
 void style (GtkWidget *ChildWidget, cairo_t *cr,double rouge, double vert, double bleu);
      ^
In file included from style.c:1:0:
include.h:621:6: note: previous declaration of ‘style’ was here
 void style (GtkWidget *ChildWidget, cairo_t *cr,double rouge, double vert, double bleu);
      ^
style.c: In function ‘style’:
style.c:272:24: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void style (GtkWidget *ChildWidget, cairo_t *cr,double rouge, double vert, double bleu)
                        ^
style.c: In function ‘classe_active’:
style.c:278:32: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void classe_active (GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets)
                                ^
style.c:278:54: warning: unused parameter ‘cr’ [-Wunused-parameter]
 void classe_active (GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets)
                                                      ^
style.c:278:73: warning: unused parameter ‘widgets’ [-Wunused-parameter]
 void classe_active (GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets)
                                                                         ^
style.c: In function ‘race_active’:
style.c:283:30: warning: unused parameter ‘ChildWidget’ [-Wunused-parameter]
 void race_active (GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets)
                              ^
style.c:283:52: warning: unused parameter ‘cr’ [-Wunused-parameter]
 void race_active (GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets)
                                                    ^
style.c:283:71: warning: unused parameter ‘widgets’ [-Wunused-parameter]
 void race_active (GtkWidget *ChildWidget, cairo_t *cr,struct widgets *widgets)
                                                                       ^
gcc -o test lecture_fichier.o gestion_glade.o chaine.o xml.o envp.o sorts.o lire_odc.o groupe.o zip.o tirage.o ouverture.o menu.o main.o classe.o tableaux.o utile.o messages.o dieux.o modif_perso.o gestion_fenetre_perso.o style.o `pkg-config gtk+-3.0 --libs` -export-dynamic  -lzip
 
Compilation finished at Sun Sep 20 13:57:47

Il n'y a plus qu'à relever les manches pour régler tous ces warnings avant d'aller plus loin.

[edit] Pendant que j'y suis j'ai lancé plusieurs fois l'application. Elle fait aussi un segdefault aléatoire au démarrage. Il y a de vraiement des problèmes de gestion des pointeurs.

Dernière observation : Fermer l'application par la croix de la fenêtre ne fonctionne pas. Tu détruis la fenêtre mais ca n'arrête pas la boucle Gtk+.
[/Edit]

Je continue mes investigations. Voila les erreurs trouvés par cppcheck (tu peux utiliser cet outil en ligne de commande ou couplé avec Code::Block)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
Checking troomad/dieux.c...
[troomad/dieux.c:155]: (error) Common realloc mistake: 'pantheons' nulled but not freed upon failure
[troomad/dieux.c:543]: (error) Common realloc mistake: 'pantheons' nulled but not freed upon failure
[troomad/dieux.c:643]: (error) Common realloc mistake: 'pantheons' nulled but not freed upon failure
[troomad/dieux.c:352]: (error) Uninitialized variable: pan
 
[troomad/gestion_fenetre_perso.c:98]: (error) Common realloc mistake: 'liste_fenetre' nulled but not freed upon failure
[troomad/gestion_fenetre_perso.c:115]: (error) Common realloc mistake: 'liste_fenetre' nulled but not freed upon failure
[troomad/gestion_fenetre_perso.c:160]: (error) Common realloc mistake: 'liste_fenetre' nulled but not freed upon failure
 
[troomad/menu.c:282]: (error) Common realloc mistake: 'nb_classe' nulled but not freed upon failure
[troomad/menu.c:335]: (error) Common realloc mistake: 'widgets' nulled but not freed upon failure

Je n'affiche que les erreurs. Il y a tout un tas de warnings, mais pour l'instant je t'en fait grâce ;).

20/09/2015, 16h30
troumad

Bon, j'ai corrigé les problème de nom global/local. Le même lien permet de télécharger une version corrigée de ce problème, mais en travail sur la suite.
Par contre, je ne comprends pas les problèmes vu lors des realloc. À moins que les realloc soient devenus déconseillés, pour moi, ils sont bons.

Par contre, j'ai encore et depuis très longtemps un warning que me laisse perplexe :

Code:

/home/troumad/Documents/glade/test_menu1/sorts.c|89|attention : variable ‘l’ set but not used [-Wunused-but-set-variable]|

Cette variable l est utilisée par la suite ! Si j'enlève cette déclaration, j'ai une erreur lors de la compilation, plus un warning :

Code:

/home/troumad/Documents/glade/test_menu1/sorts.c|208|erreur: ‘l’ undeclared (first use in this function)|
20/09/2015, 16h37
gerald3d

Ce type de warning arrive lorsque tu utilises une variable dans une paire de {} alors qu'elle est déclarée à l'extérieur et qu'elle n'est pas utilisée en dehors. Tu pourrais peut être faire la déclaration de l dans le pavé conditionnel où elle est utilisée par exemple.
20/09/2015, 18h51
troumad

Ceci ne me convient pas : ce n'est pas une bonne méthode de programmation compatible avec les vieilles normes du C que je suis parfois obligé d'utiliser.
22/09/2015, 19h35
gerald3d
Citation:

Envoyé par troumad

Ceci ne me convient pas : ce n'est pas une bonne méthode de programmation compatible avec les vieilles normes du C que je suis parfois obligé d'utiliser.

Je ne comprends pas bien ta remarque. Si on a par exemple
Code:

1 2 3 4 5 6 7 8 9 void foo (int b) { int a; if (b==2) { a=b; } }
a n'a pas besoin d'être initialisé en début de fonction. Il suffit de la déclarer dans les accolades de la condition. Je ne vois pas en quoi on sort des standards du C. De plus la déclarer dans la condition permet une meilleure gestion de l'espace mémoire utilisé dans la pile.

Bon en même temps je suis loin de maîtriser toutes les normes... ;)
22/09/2015, 19h48
troumad

J'ai l'impression que les normes MISRA ( https://fr.wikipedia.org/wiki/MISRA_C ) acceptent de mettre les initialisations dans les accolades. Je vais essayer de le faire même si je ne l'ai jamais fait depuis 1990...

Je suis en train de reprendre ton projet en utilisant les autotools. Je suis presque arrivé à mes fins.

Lors de mes premières compilations j'obtiens en tout premier warning :

Code:

1
2
lecture_fichier.c:891:5: warning: ‘retour’ may be used uninitialized in this function [-Wmaybe-uninitialized]
     return retour;

Je remets ici la fonction incriminée :

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
26
27
28
29
struct psi_add2 * lecture_case_psi2(char * coordonnees,GMarkupDomNode * ooo)
{
    GMarkupDomNode * info;
    unsigned short i;
    struct psi_add2 * retour;
 
 
    info=donne_case_A1(coordonnees,ooo);
    if (info==NULL)
    {
        retour = NULL;
    }
    else if (info->nb_texte==0)
    {
        retour = NULL; /* pas de texte */
    }
    else
    {
        for (i=0;i<nb_psi;i++)
        {
            if (psi[i].add2.nom!=NULL && compare_sans_casse(psi[i].add2.nom,info->texte->texte)==0)
            {
                retour = &(psi[i].add2);
                i=nb_psi;
            }
        }
    }
    return retour;
}

Le warning provient du fait que le pointeur retour n'est pas initialisé au départ. Si on le place à NULL on peut revoir l'agencement de la fonction comme suit :

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
26
27
28
29
struct psi_add2 * lecture_case_psi2(char * coordonnees,GMarkupDomNode * ooo)
{
    GMarkupDomNode * info;
    unsigned short i;
    struct psi_add2 * retour = NULL;


    info=donne_case_A1(coordonnees,ooo);
    if (info==NULL) return NULL;
    {
        retour = NULL;
    }
    else if (info->nb_texte==0) return NULL;
    {
        retour = NULL; /* pas de texte */
    }
    else
    {
        for (i=0;i<nb_psi;i++)
        {
            if (psi[i].add2.nom!=NULL && compare_sans_casse(psi[i].add2.nom,info->texte->texte)==0)
            {
                retour = &(psi[i].add2);
                i=nb_psi;
            }
        }
    }
    return retour;
}

Ce premier code modifié donne en plus clair ceci.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
struct psi_add2 * lecture_case_psi2(char * coordonnees,GMarkupDomNode * ooo)
{
    GMarkupDomNode * info;
    unsigned short i;
    struct psi_add2 * retour NULL;
 
 
    info=donne_case_A1(coordonnees,ooo);
    if (info==NULL) return NULL;
 
    if (info->nb_texte==0) return NULL;
 
    for (i=0;i<nb_psi;i++)
    {
       if (psi[i].add2.nom!=NULL && compare_sans_casse(psi[i].add2.nom,info->texte->texte)==0)
       {
           retour = &(psi[i].add2);
           i=nb_psi;
       }
    }
  }
  return retour;
}

Puisque tu désires sortir de la boucle lorsque tu as trouvé une correspondance on peut encore réduire le code et cette fois-ci se passer carrément du pointeur retour :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
struct psi_add2 * lecture_case_psi2(char * coordonnees,GMarkupDomNode * ooo)
{
    GMarkupDomNode * info;
    unsigned short i;
 
    info=donne_case_A1(coordonnees,ooo);
    if (info==NULL || info->nb_texte==0) return NULL;
 
    for (i=0;i<nb_psi;i++)
    {
       if (psi[i].add2.nom!=NULL && compare_sans_casse(psi[i].add2.nom,info->texte->texte)==0)
        return &(psi[i].add2);
    }
 
  return NULL;
}

Sur ce il est tard je pars pour un repos récupérateur :D.

23/09/2015, 07h05
troumad

Je crois que les normes impliquent un seul return par fonction. Et on se rend bien compte que c'est important de programmer de cette façon si en modifiant une fonction, il faut à la fin libérer la mémoire.

Voir 40 message(s) de cette discussion en une page