besoin d'aide sur des warnings

Bonjour, j'ai le code suivant
Code:

1 2 3 4 5 6 char seed[50]; /* ici, dans une sous-fonction je copie dans seed une string */ /* initialisation of the pseudo-random numbers generator seed */ unsigned seed_=(!strcmp(seed,"TIME")) ? (unsigned) time(NULL) : strtol(seed,&seed,10);
j'ai le warning suivant :

Citation:

../Sources/Mysources/main.c:454: warning: passing arg 2 of `strtol' from incompatible pointer type

pourtant j'ai bien mis un
Code:

1 2 &seed
autre warning : dès que je fais un strdup() j'ai le warning suivant

Citation:

../Sources/Mysources/main.c:520: warning: passing arg 1 of `calloc' with different width due to prototype

je sais que dans un strdup il y a un malloc (ou plutôt un calloc vu le warning), mais comment puis-je l'enlever ?

Voici mes options de compil (chopées sur la page d'Emmanuel Delahaye)

Citation:

ED_PARANOIA=-O2 -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 \
-Wunreachable-code

merci beaucoup.

Un tableau de char est implicitement convertible en char*, mais n'est pas un vrai char*. Tu ne peux donc pas passer son adresse à une fonction qui demande un pointeur vers un char*.
Tu dois créer une seconde variable, de type char* celle-ci, et passer son adresse à strtol().
Ou passer NULL à la place, si tu ne l'utilises pas.

Citation:

Envoyé par Médinoc

Un tableau de char est implicitement convertible en char*, mais n'est pas un vrai char*. Tu ne peux donc pas passer son adresse à une fonction qui demande un pointeur vers un char*.
Tu dois créer une seconde variable, de type char* celle-ci, et passer son adresse à strtol().
Ou passer NULL à la place, si tu ne l'utilises pas.

OK, merci, c'est réglé. Et pour le second warning, quelqu'un aurait-il une idée ?

Vu que je ne vois pas l'appel, ce n'est pas évident.
Et regarde si strdup() n'est pas une macro ou une fonction inline...

Voici qq passages lorsque j'utilise strdup()

Code:

1
2
3
4
 
char * p=strstr(line,"COMPEGR ");
p+=8; /* 8 = strlen("COMPEGR ") */
id->egr->comp=strdup(p);

ou bien

Code:

1
2
 
NameFileOutput[N+5]=strdup("Turbulence.txt");

OK, ça a l'air correct.
Épluche un peu les en-têtes...

Citation:
Envoyé par salseropom
Code:

1 2 3 4 5 6 char seed[50]; /* ici, dans une sous-fonction je copie dans seed une string */ /* initialisation of the pseudo-random numbers generator seed */ unsigned seed_=(!strcmp(seed,"TIME")) ? (unsigned) time(NULL) : strtol(seed,&seed,10);
Normal. le 2ème paramètre de strtol() attends l'adresse d'un pointeur sur char et non l'adresse d'un tableau de 50 char.

Il faut donc lui fournir un pointeur sur char :

Code:

char *pend;

et lui passer son adresse :

Code:

strtol (seed, &pend, 10);

en admettant que tu l'utilises par la suite. Si c'est pour ne pas l'utiliser, passer NULL directement :

Code:

strtol (seed, NULL, 10);

Rien de tout cel ne tombe du ciel, mais est expliqué dans le doc de strtol()...

Re,
man strdup() me donne
Code:

1 2 char * strdup (const char *s);
j'avoue que je ne vois pas trop où est le problème. Mais le problème est dans l'appel de calloc() par strdup(). A croire que cette fonction est "mal appelée"...

Citation:

Envoyé par salseropom

Voici qq passages lorsque j'utilise strdup()

strdup() n'est pas standard. Tu as inclus ce qu'il faut pour ton compilateur ? (en principe <string.h>)

Vérifie qu'il n'y a pas une macro 'strdup' parasite définie dans un de tes headers d'application ou 'maison'...

Citation:

Envoyé par Emmanuel Delahaye

strdup() n'est pas standard. Tu as inclus ce qu'il faut pour ton compilateur (en principe <string.h>)

Vérifie qu'il n'y a pas une macro 'strdup' parasite définie dans un de tes headers d'application ou 'maison'...

oui

dans /usr/include/string.h j'ai

Citation:

#if defined __USE_SVID || defined __USE_BSD || defined __USE_XOPEN_EXTENDED
/* Duplicate S, returning an identical malloc'd string. */
extern char *strdup (__const char *__s)
__THROW __attribute_malloc__ __nonnull ((1));
#endif

et dans /opt/dietlibc/include/string.h j'ai

Citation:

char *strdup(const char *s) __THROW __attribute_malloc__ ;

j'ai fait un
Code:

1 2 find / -name string.h

Apparemment, c'est plutôt du côté d'une macro ou fonction inline qu'il faut chercher.

Fais un grep avec "strdup" sur tous les fichiers d'en-tête standard + les fichiers des bibliothèques utilisées ou de ton programme, etc.

Voire même, passe directement ton code au préprocesseur (cpp) et fais un grep strdup dessus...

Citation:
Envoyé par salseropom
Voici qq passages lorsque j'utilise strdup()
Code:

1 2 3 4 char * p=strstr(line,"COMPEGR "); p+=8; /* 8 = strlen("COMPEGR ") */ id->egr->comp=strdup(p);
ou bien
Code:

1 2 NameFileOutput[N+5]=strdup("Turbulence.txt");
Dans le 2ième passage, ton tableau NameFileOutput est un tableau de char * ??
(char **Name.. suivi d'un alloc, ou bien char *Name...[50], mais PAS char *Name.. suivi d'un alloc, ou char Name..[50] ? )

Parce que si c'est un tableau de char, tu veux stocker un char* dans un char...

Citation:

Envoyé par souviron34

Dans le 2ième passage, ton tableau NameFileOutput est un tableau de char * ??
(char **Name.. suivi d'un alloc, ou bien char *Name...[50], mais PAS char *Name.. suivi d'un alloc, ou char Name..[50] ? )

Parce que si c'est un tableau de char, tu veux stocker un char* dans un char...

salut,
j'ai fait ceci
Code:

1 2 char * NameFileOutput[nbfile];
et ensuite je fais des
Code:

1 2 NameFileOutput[i]=strdup("blabla");
puis je fais (à la fin)
Code:

1 2 free(NameFileOutput[i]), NameFileOutput[i]=NULL; (pour 0 <= i < nbfile)