encore des warings

Bonjour, j'ai encore des warnings lors de la compilation que je n'arrive pas à enlever.
Voici mon premier warning :

Citation:

---

../source/mysource/species.c:274: attention : passage de l'argument n°2 de « NumbersAtoms » a une largeur différente en raison du prototype

---

et voici le code en question :

Code:

---

1 2	s->atoms->C=NumbersAtoms(s->name,'C');

---

Voici le prototype de ma fonction NumbersAtoms :

Code:

---

1 2	unsigned int NumbersAtoms(char *,char);

---

J'ai un second warning :

Citation:

---

../source/mysource/randompulling.c:52: attention : cast does not match function type

---

voici le code en question

Code:

---

1 2	double r=RAND;

---

où j'avais fait un

Code:

---

1 2	#define RAND ((double) rand())/((double) RAND_MAX)

---

Voici mes options de compilation (que j'ai chopées sur ce forum) :

Code:

---

1 2

CFLAGS=-Wall -O3 -Wchar-subscripts -Wcomment -Wformat=2 -Wimplicit-int -Werror-implicit-function-declaration -Wmain -Wparentheses -Wsequence-point -Wreturn-type -Wswitch -Wtrigraphs -Wunused -Wuninitialized -Wunknown-pragmas -Wfloat-equal -Wundef -Wshadow -Wpointer-arith -Wbad-function-cast -Wwrite-strings -Wconversion -Wsign-compare -Waggregate-return -Wstrict-prototypes -Wmissing-prototypes -Wmissing-declarations -Wmissing-noreturn -Wformat -Wmissing-format-attribute -Wno-deprecated-declarations -Wpacked -Wredundant-decls -Wnested-externs -Winline -Wlong-long

---

Pouvez-vous me passer un petit coup de main ?
Merci

Je pense que le premier warning est dû au fait que tu aies une fonction dont l'un des paramètres est d'un type d'une largeur inférieure à int.
Ça veut dire que s'il n'y avait pas le prototype, le char serait promu en int.

Pour le second warning, je ne vois pas.

Citation:

---

Envoyé par salseropom

Code:

---

1 2	unsigned int NumbersAtoms(char *,char);

---

Citation:

---

../source/mysource/species.c:274: attention : passage de l'argument n°2 de « NumbersAtoms » a une largeur différente en raison du prototype

---

---

Il te dit en gros que le type char sera converti en une autre taille. Je conseille d'utiliser diretement le type int. Un paramètre de type char, ça ne sert à rien et ça ne fait qu'ajouter du code de conversion supplémentaire et inutile.

Code:

---

1 2	unsigned int NumbersAtoms (char *, int);

---

Citation:

---

Code:

---

1 2	double r=RAND;

---

avec

Code:

---

1 2	#define RAND ((double) rand())/((double) RAND_MAX)

---

Citation:

---

../source/mysource/randompulling.c:52: attention : cast does not match function type

---

---

Il te dit en gros que rand() ne retourne pas un double et que tu castes en double. Je pense que le cast est inutile. Le type de r suffit à faire le cast automatiquement.

Code:

---

1 2	double r=RAND;

---

avec

Code:

---

1 2	#define RAND (rand())/((double) RAND_MAX)

---

OK, merci. Je n'ai plus mes warnings. J'avais fait

Code:

---

1 2	#define RAND ((double) rand())/((double) RAND_MAX)

---

pour avoir une division de double (et non de int). Mais il est vrai que le (double) RAND_MAX m'assure que j'effectue bien une division de double.

J'ai encore deux autres warnings qui m'intriguent énormément :

voici le premier

Citation:

---

../source/mysource/functions.c:233: attention : passage de l'argument n°1 de « calloc » a une largeur différente en raison du prototype

---

et voici le code

Code:

---

1 2	d->name=strdup(id->pdf);

---

où d->name et id->pdf sont des char *

et mon second warning est :

Citation:

---

../source/mysource/mechanism.c:468: attention : passage de l'argument n°2 de « fgets » a une largeur différente en raison du prototype

---

où le code est

Code:

---

1 2	while((fgets(line,sizeof line,file)!=NULL) && (strstr(line,"END")==NULL) )

---

où j'ai fait :

Code:

---

1 2	char line[1000]

---

Depuis que je compile avec toutes ces options, je pleure devant tous mes warning...

Merci encore.

Citation:

---

Envoyé par salseropom

Code:

---

1 2	d->name=strdup(id->pdf);

---

Citation:

---

../source/mysource/functions.c:233: attention : passage de l'argument n°1 de « calloc » a une largeur différente en raison du prototype

---

où d->name et id->pdf sont des char *

---

C'est plutôt étrange.

Déjà, il semble que strdup() soit une macro ou une fonction inline implémentée avec calloc(), ce qui est une drôle d'idée...
Ensuite, il doit y avoir un cast bizarre dans cette implémentation...

C'est quelle version de gcc ? Je vais essayer de reproduire le défaut chez moi...

Citation:

---

Code:

---

1 2	while((fgets(line,sizeof line,file)!=NULL) && (strstr(line,"END")==NULL) )

---

Citation:

---

../source/mysource/mechanism.c:468: attention : passage de l'argument n°2 de « fgets » a une largeur différente en raison du prototype

---

---

Ce warning est normal. Il montre un bug dans la conception de fgets(). Le type du 2 ème paramètre aurait du être size_t, or il est int, ce qui n'est pas adapté. a part une cast (int), je ne vois pas comment corriger ça...

Code:

---

1 2	while((fgets(line,(int)sizeof line,file)!=NULL) && (strstr(line,"END")==NULL) )

---

Nota : la conditiopn ne fonctionnera que si la dernière ligne est incomplète et contient "END". Dans le cas général, il faut penser à netoyer la ligne avant de la tester... (p.e. avec clean(), maintes fois cité...)

Citation:

---

Depuis que je compile avec toutes ces options, je pleure devant tous mes warning...

---

C'est normal, c'est fait pour une surveillance du code extrèmement sévère...

Ce ne sont pas des options de débutants... Je les avais signalées à titre indicatif, mais je ne recommande pas ça à des débutants...

Code:

---

-Wall -Wextra -Wwrite-strings -O2

---

est largement suffisant pour commencer.

Salut Emmanuel, merci de tes explications.
J'utilise la version gcc 3.3.5

Citation:

---

Ce ne sont pas des options de débutants... Je les avais signalées à titre indicatif, mais je ne recommande pas ça à des débutants...

---

J'espère avoir dépassé le niveau débutant... en sachant que je suis encore loin du niveau expert...

Citation:

---

C'est normal, c'est fait pour une surveillance du code extrèmement sévère...

---

Bon ça me rassure. Mis à part les warning dûs aux fgets, calloc et strncpy je n'en ai plus d'autres. J'espère que mon code est "bien écrit".

Citation:

---

Envoyé par salseropom

Salut Emmanuel, merci de tes explications.
J'utilise la version gcc 3.3.5



J'espère avoir dépassé le niveau débutant... en sachant que je suis encore loin du niveau expert...



Bon ça me rassure. Mis à part les warning dûs aux fgets, calloc et strncpy je n'en ai plus d'autres. J'espère que mon code est "bien écrit".

---

Il peut subsister des warnings (le moins possible, car ça pollue), du moment qu'on sait expliquer qu'ils ne signalent pas un danger.

oui, tout à fait.

Pour ma part, je conseillerais de garder -Wwrite-strings: C'est une bonne pratique à prendre...
Edit: Corrigé, parfait ;)

Citation:

---

Envoyé par Médinoc

Pour ma part, je conseillerais de garder -Wwrite-strings: C'est une bonne pratique à prendre...

---

+1

Je corrige.