Discussion :

[salseropom]

Bonjour, j'ai le code suivant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

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

pourtant j'ai bien mis un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
&seed

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

../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)

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.

[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.

[salseropom]

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 ?

[Médinoc]

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...

[salseropom]

Voici qq passages lorsque j'utilise strdup()

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

ou bien

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Médinoc]

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

[Emmanuel Delahaye]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

   char *pend;

et lui passer son adresse :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

   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 :

Sélectionner tout - Visualiser dans une fenêtre à part

   strtol (seed, NULL, 10);

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

[salseropom]

Re,
man strdup() me donne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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"...

[Emmanuel Delahaye]

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'...

[salseropom]

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

#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

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

j'ai fait un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
find / -name string.h

[Médinoc]

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...

[souviron34]

Voici qq passages lorsque j'utilise strdup()

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

ou bien

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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...

[salseropom]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
char * NameFileOutput[nbfile];

et ensuite je fais des

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
NameFileOutput[i]=strdup("blabla");

puis je fais (à la fin)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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