Problème avec strlwr

**licorne** · 09/05/2006, 16h13

Bonjour,

J'ai le message :

makes pointer from integer without a cast

à la ligne

while((p = strstr(strlwr(pelt),strlwr(elt))) && p != NULL)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
FILE *fichier;
char *elt = NULL;
char *pelt = NULL;
 
elt = argv[2];
 
if ((fichier = fopen(chaine,"r")) == NULL) 
   exit(1);
 
  while (fgets(ligne,sizeof ligne,fichier) != NULL)
  {
   lgligne = strlen(ligne);
   pelt = ligne;
   /* Recherche de la chaine dans la ligne lue */
   while((p = strstr(strlwr(pelt),strlwr(elt))) && p != NULL)
   {
    nb++;
    p+=lgelt;
    pelt = p;
   }
  }

Pourtant pelt et elt sont tous bien des pointers.

Avez-vous une idée ?

**crocodilex** · 09/05/2006, 18h16

Ne connaissant pas la fonction strlwr() (je n'ai pas trouvé de man, même sur ma distrib linux), je dirais à première vu que strlwr() retourne un int alors que strstr() attends un char *

**gege2061** · 09/05/2006, 18h20

Envoyé par crocodilex

Ne connaissant pas la fonction strlwr(),

Il s'agit surement d'une fonction maison ? Avoir son prototype pourrais nous aider ainsi que la déclaration de p

**Emmanuel Delahaye** · 10/05/2006, 00h10

Envoyé par licorne

/* Recherche de la chaine dans la ligne lue */
while((p = strstr(strlwr(pelt),strlwr(elt))) && p != NULL)

Comportement indéfini

fonction strlwr() non standard
nom réservé

http://emmanuel-delahaye.developpez....htm#id_reserve

**licorne** · 10/05/2006, 11h45

La déclaration de p :

Code :

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

 char *p = NULL;

Lors de la compilation du prog j'ai obtenu les warnings suivants sur la ligne "while((p = strstr(strlwr(pelt),strlwr(elt))) && p != NULL)" :

implicit declaration of function 'strlwr'
passing arg 1 of 'strstr' makes pointer from integer without a cast
passing arg 2 of 'strstr' makes pointer from integer without a cast

D'autre part le compilateur est mingw et dans la string.h on trouve :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
#ifndef __STRICT_ANSI__
/*
 * Extra non-ANSI functions provided by the CRTDLL library
 */
...
_CRTIMP char* __cdecl _strlwr (char*);
...
#ifndef _NO_OLDNAMES
/*
 * Non-underscored versions of non-ANSI functions. They live in liboldnames.a
 * and provide a little extra portability. Also a few extra UNIX-isms like
 * strcasecmp.
 */
...
_CRTIMP char* __cdecl strlwr (char*);
...

J'ai quand-même lancé le prog puisqu'il s'agissait de warnings et ça marche. Cependant je me méfie des warnings qui indiquent un pb de compatibilité de format.
Ai-je raison de me méfier ?

Pouvez-vous m'indiquer également la signification de "implicit declaration of function 'strlwr'" ?

Merci pour votre aide précieuse.

**crocodilex** · 10/05/2006, 11h52

As-tu inclus string.h dans ton source ?

**licorne** · 10/05/2006, 11h59

Dans la string.h j'ai défini la fonction strlwr dans la section "/*
* Prototypes of the ANSI Standard C library string functions.
*/
" :

_CRTIMP char* __cdecl strlwr (char*);

Il reconnait (sans warning) et exécute correctement la fonction strlwr qui met en minuscule une chaine de caractères.

Pouvez-vous m'expliquer ou se trouve le code des fonctions déclarées dans la libraire "string.h" ?

**DaZumba** · 10/05/2006, 12h03

Le warning sur les arguments de strstr() signifient que le compilateur n'a pas le prototype de strlwr() et suppose qu'elle retourne un int.
Pourquoi n'a t-il pas le prototype de strlwr()?
- parce que tu n'inclus pas <string.h>
- parce que tu demandes une compilation en C ANSI strict (-ansi), ce qui desactives les extensions.

**crocodilex** · 10/05/2006, 12h11

Envoyé par licorne

Dans la string.h j'ai défini la fonction strlwr dans la section "/*
* Prototypes of the ANSI Standard C library string functions.
*/
" :

Il reconnait (sans warning) et exécute correctement la fonction strlwr qui met en minuscule une chaine de caractères.

Pouvez-vous m'expliquer ou se trouve le code des fonctions déclarées dans la libraire "string.h" ?

Qu'est-ce que tu es entrain de faire comme bricolage ?
"string.h" n'est pas une librairie, mais un fichier d'entête.
Expliques-nous ce que tu veux faire.

**licorne** · 10/05/2006, 13h13

Envoyé par crocodilex

Qu'est-ce que tu es entrain de faire comme bricolage ?
"string.h" n'est pas une librairie, mais un fichier d'entête.
Expliques-nous ce que tu veux faire.

J'ai donc déclaré l'entête strlwr dans le fichier d'entête string.h au même endroit que les autres entêtes "/*
* Prototypes of the ANSI Standard C library string functions.
*/
" avec la ligne suivante :

/*
* Prototypes of the ANSI Standard C library string functions.
*/
_CRTIMP void* __cdecl memchr (const void*, int, size_t) __MINGW_ATTRIB_PURE;
_CRTIMP int __cdecl memcmp (const void*, const void*, size_t) __MINGW_ATTRIB_PURE;
_CRTIMP void* __cdecl memcpy (void*, const void*, size_t);
_CRTIMP void* __cdecl memmove (void*, const void*, size_t);
_CRTIMP void* __cdecl memset (void*, int, size_t);
_CRTIMP char* __cdecl strcat (char*, const char*);
_CRTIMP char* __cdecl strchr (const char*, int) __MINGW_ATTRIB_PURE;
_CRTIMP int __cdecl strcmp (const char*, const char*) __MINGW_ATTRIB_PURE;
_CRTIMP int __cdecl strcoll (const char*, const char*); /* Compare using locale */
_CRTIMP char* __cdecl strcpy (char*, const char*);
_CRTIMP size_t __cdecl strcspn (const char*, const char*) __MINGW_ATTRIB_PURE;
_CRTIMP char* __cdecl strerror (int); /* NOTE: NOT an old name wrapper. */
_CRTIMP size_t __cdecl strlen (const char*) __MINGW_ATTRIB_PURE;
_CRTIMP char* __cdecl strncat (char*, const char*, size_t);
_CRTIMP int __cdecl strncmp (const char*, const char*, size_t) __MINGW_ATTRIB_PURE;
_CRTIMP char* __cdecl strncpy (char*, const char*, size_t);
_CRTIMP char* __cdecl strpbrk (const char*, const char*) __MINGW_ATTRIB_PURE;
_CRTIMP char* __cdecl strrchr (const char*, int) __MINGW_ATTRIB_PURE;
_CRTIMP size_t __cdecl strspn (const char*, const char*) __MINGW_ATTRIB_PURE;
_CRTIMP char* __cdecl strstr (const char*, const char*) __MINGW_ATTRIB_PURE;
_CRTIMP char* __cdecl strtok (char*, const char*);
_CRTIMP size_t __cdecl strxfrm (char*, const char*, size_t);
_CRTIMP char* __cdecl strlwr (char*);

Le résultat de la compilation montrait qu'il reconnaissait la fonction strlwr mais qu'apparament elle était mal déclarée dans le fichier string.h.
C'était juste un essai pour essayer de trouver une solution. Qu'en pensez-vous ?

**Neitsa** · 10/05/2006, 13h31

Bonjour,

J'ai donc déclaré l'entête strlwr dans le fichier d'entête string.h au même endroit que les autres entêtes

Houlala, il ne faudrait surtout pas toucher à ces fichiers !

Pourquoi ne pas utiliser un équivalent conforme au standard ?

En utilisant strlen()(<string.h>) et la fonction tolower() (<ctype.h>)dans une boucle, le problème serait reglé et conforme au standard.

La fonction tolower() ne travaille que sur un seul caractère, c'est pourquoi il faudra boucler sur chaque caractère de la chaîne.

**licorne** · 10/05/2006, 13h45

Envoyé par Neitsa

Bonjour,

Houlala, il ne faudrait surtout pas toucher à ces fichiers !

Pourquoi ne pas utiliser un équivalent conforme au standard ?

En utilisant strlen()(<string.h>) et la fonction tolower() (<ctype.h>)dans une boucle, le problème serait reglé et conforme au standard.

La fonction tolower() ne travaille que sur un seul caractère, c'est pourquoi il faudra boucler sur chaque caractère de la chaîne.

Ok, je convertis caractère par caractère.

Merci à tous.