macro de fonction

**ManusDei** · 20/04/2011, 09h23

Bonjour,

je reprend du code (qui compilait il y a 10 ans), et j'ai quelques problèmes avec les macros.

Code C :

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
#define addfield_parse_uint(x)\
uint##x##_t addfield_parse_uint##x(char *s){\
    uint##x##_t i;\
    char c;\
    int res; \
    if(s[0] != '0' || s[1] == '\0') \
    	res = sscanf(s,"%"SCNu##x"%c",&i,&c); \
    else if(s[1] == 'x' || s[1] == 'X') \
    	res = sscanf(s + 2,"%"SCNx##x"%c",&i,&c); \
    else \
    	res = sscanf(s + 1,"%"SCNo##x"%c",&i,&c); \
    if(res != 1)\
        gep_erreur(1,"Cannot see an uint"#x" in '%s`\n",s);\
    return i;\
}
addfield_parse_uint(8);
addfield_parse_uint(16);
addfield_parse_uint(32);
addfield_parse_uint(64);

Quand je veux compiler ça, j'obtiens ce message d'erreur :

Code :

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

1
2
3
D:\workspace\gliss\src\gep\gep_addfield.c: In function 'addfield_parse_uint8':
D:\workspace\gliss\src\gep\gep_addfield.c:16:1: error: expected ')' before 'SCNx8'
D:\workspace\gliss\src\gep\gep_addfield.c:16:1: error: expected ')' before 'SCNo8'

(j'ai changé le numéro de ligne pour que ça colle avec la balise code au-dessus, sympa la nouvelle balise)

Le problème c'est que je ne comprend pas grand chose aux macros.
Là je ne comprend pas vraiment où est le problème, est-ce que quelqu'un pourrait m'expliquer ? Je n'ai pas réussi à trouver d'explications suffisamment claires sur internet (je ne connais que les #define "de base" pour remplacer une chaine par une valeur).

Edit: Le compilateur ne renvoie d'erreur que pour la macro avec 8, celles avec 16, 32, 64 passent l'étape de la compilation.

Invité · 20/04/2011, 09h35

Bonjour,

Il existe un argument du compilateur pour obtenir la pré-compilation en sortie standard: -E.

Ça permettra de voir le résultat de la transformation des macros.

**pfeuh** · 20/04/2011, 09h53

Salut,

Ta macro semble définir des fonctions dont le type de retour est uint##x##_t avec ## remplacé par un des paramètres 8, 16, 32 ou 64. SI tu n'as pas fait de typedef pour ces 4 types, le compilo râle, c'est ce qui se produit chez moi. Peut-être n'as tu pas donné tout le code?

A+

Pfeuh

**ManusDei** · 20/04/2011, 10h16

J'ai rajouté l'option -E, je n'ai rien de plus en sortie, mais ça compile (je génère mes makefile avec CMake et compile avec MinGW sous Eclipse).
Bien sûr pas d'exe, pas de link.

Dans un fichier .h j'ai ça, donc la version avec 8 est définie pour u, mais SCNx8 et SCNo8 ne sont définis nulle part (et pourtant ça compile avec -E).

Code :

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

1
2
3
4
5
6
#ifndef SCNu8  
#define SCNu8 "hhu"
#endif
#ifndef SCNd8
#define SCNd8 "hhd"
#endif

Par inclusion, dans inttypes.h (fichier système que je n'ai pas défini), SCNu16, SCNu32 et SCNu64 sont définis.

En fait, il faut que je rajoute les définitions pour SCNx8 et SCNu8 qui manquent (et que j'enlève l'option -E)?

EDIT : SCNx8 et SCNu8 sont présents dans inttypes.h

**ManusDei** · 20/04/2011, 10h53

J'ai enlevé -E et rajouté -v à la compilation, mais j'ai pas l'impression que çà apporte grand chose.

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
[ 66%] Building C object src/gep/CMakeFiles/gep.dir/gep_addfield.c.obj
Using built-in specs.
COLLECT_GCC=C:\Win16App\MinGW\bin\gcc.exe
COLLECT_LTO_WRAPPER=c:/win16app/mingw/bin/../libexec/gcc/mingw32/4.5.2/lto-wrapper.exe
Target: mingw32
Configured with: ../gcc-4.5.2/configure --enable-languages=c,c++,ada,fortran,objc,obj-c++ --disable-sjlj-exceptions --with-dwarf2 --enable-shared --enable-libgomp --disable-win32-registry --enable-libstdcxx-debug --enable-version-specific-runtime-libs --disable-werror --build=mingw32 --prefix=/mingw
Thread model: win32
gcc version 4.5.2 (GCC) 
COLLECT_GCC_OPTIONS='-O3' '-DNDEBUG' '-v' '-ID:\workspace\gliss\src\common' '-o' 'CMakeFiles\gep.dir\gep_addfield.c.obj' '-c' '-mtune=i386' '-march=i386'
 c:/win16app/mingw/bin/../libexec/gcc/mingw32/4.5.2/cc1.exe -quiet -v -ID:\workspace\gliss\src\common -iprefix c:\win16app\mingw\bin\../lib/gcc/mingw32/4.5.2/ -DNDEBUG D:\workspace\gliss\src\gep\gep_addfield.c -quiet -dumpbase gep_addfield.c -mtune=i386 -march=i386 -auxbase-strip CMakeFiles\gep.dir\gep_addfield.c.obj -O3 -version -o c:\temp\ccPWWGRC.s
GNU C (GCC) version 4.5.2 (mingw32)
	compiled by GNU C version 4.5.2, GMP version 5.0.1, MPFR version 2.4.1, MPC version 0.8.1
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
ignoring nonexistent directory "c:\win16app\mingw\bin\../lib/gcc/mingw32/4.5.2/../../../../mingw32/include"
ignoring nonexistent directory "/mingw/include"
ignoring duplicate directory "c:/win16app/mingw/lib/gcc/../../include"
ignoring duplicate directory "c:/win16app/mingw/lib/gcc/../../lib/gcc/mingw32/4.5.2/include"
ignoring duplicate directory "c:/win16app/mingw/lib/gcc/../../lib/gcc/mingw32/4.5.2/include-fixed"
ignoring nonexistent directory "c:/win16app/mingw/lib/gcc/../../lib/gcc/mingw32/4.5.2/../../../../mingw32/include"
ignoring nonexistent directory "/mingw/include"
#include "..." search starts here:
#include <...> search starts here:
 D:\workspace\gliss\src\common
 c:\win16app\mingw\bin\../lib/gcc/mingw32/4.5.2/../../../../include
 c:\win16app\mingw\bin\../lib/gcc/mingw32/4.5.2/include
 c:\win16app\mingw\bin\../lib/gcc/mingw32/4.5.2/include-fixed
End of search list.
GNU C (GCC) version 4.5.2 (mingw32)
	compiled by GNU C version 4.5.2, GMP version 5.0.1, MPFR version 2.4.1, MPC version 0.8.1
GGC heuristics: --param ggc-min-expand=100 --param ggc-min-heapsize=131072
Compiler executable checksum: 48c0bd49e9a9404f24c600b71b435d06
D:\workspace\gliss\src\gep\gep_addfield.c: In function 'addfield_parse_uint8':
D:\workspace\gliss\src\gep\gep_addfield.c:129:1: error: expected ')' before 'SCNx8'
D:\workspace\gliss\src\gep\gep_addfield.c:129:1: error: expected ')' before 'SCNo8'
D:\workspace\gliss\src\gep\gep_addfield.c: In function 'addfield_parse_int8':
D:\workspace\gliss\src\gep\gep_addfield.c:149:1: error: expected ')' before 'SCNx8'
D:\workspace\gliss\src\gep\gep_addfield.c:149:1: error: expected ')' before 'SCNo8'
mingw32-make[2]: *** [src/gep/CMakeFiles/gep.dir/gep_addfield.c.obj] Error 1
mingw32-make[1]: *** [src/gep/CMakeFiles/gep.dir/all] Error 2
mingw32-make: *** [all] Error 2

Invité · 20/04/2011, 11h04

Ce que dit le man gcc :

-E Stop after the preprocessing stage; do not run the compiler proper. The output is in the form of preprocessed source code, which is sent to the standard output.

J'ai pas réussi à trouver d'info sur l'équivalent mingw, mais ça m'étonnerait que cette option ait été supprimé.

**ManusDei** · 20/04/2011, 11h18

Je suis allé voir dans le fichier .obj (manque d'habitude).
Je rajoute des retours à la ligne pour que ça soit un peu plus lisible.

Ca me semble être du code C tout ce qu'il y a de plus normal :s

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
uint8_t addfield_parse_uint8(char *s);
uint16_t addfield_parse_uint16(char *s);
uint32_t addfield_parse_uint32(char *s);
uint64_t addfield_parse_uint64(char *s);
 
 
uint8_t addfield_parse_uint8(char *s){ 
  uint8_t i; char c; int res; 
  if(s[0] != '0' || s[1] == '\0') 
  res = sscanf(s,"%""hhu""%c",&i,&c); 
  else if(s[1] == 'x' || s[1] == 'X') res = sscanf(s + 2,"%"SCNx8 "%c",&i,&c); 
  else res = sscanf(s + 1,"%"SCNo8 "%c",&i,&c); 
  if(res != 1) gep_erreur_full(1 , "D:\\workspace\\gliss\\src\\gep\\gep_addfield.c",129,"Cannot see an uint""8"" in '%s`\n",s); 
  return i;};
 
uint16_t addfield_parse_uint16(char *s){ uint16_t i; char c; int res; if(s[0] != '0' || s[1] == '\0') res = sscanf(s,"%""hu""%c",&i,&c); else if(s[1] == 'x' || s[1] == 'X') res = sscanf(s + 2,"%""hx""%c",&i,&c); else res = sscanf(s + 1,"%""ho""%c",&i,&c); if(res != 1) gep_erreur_full(1 , "D:\\workspace\\gliss\\src\\gep\\gep_addfield.c",130,"Cannot see an uint""16"" in '%s`\n",s); return i;};
uint32_t addfield_parse_uint32(char *s){ uint32_t i; char c; int res; if(s[0] != '0' || s[1] == '\0') res = sscanf(s,"%""u""%c",&i,&c); else if(s[1] == 'x' || s[1] == 'X') res = sscanf(s + 2,"%""x""%c",&i,&c); else res = sscanf(s + 1,"%""o""%c",&i,&c); if(res != 1) gep_erreur_full(1 , "D:\\workspace\\gliss\\src\\gep\\gep_addfield.c",131,"Cannot see an uint""32"" in '%s`\n",s); return i;};
uint64_t addfield_parse_uint64(char *s){ uint64_t i; char c; int res; if(s[0] != '0' || s[1] == '\0') res = sscanf(s,"%""I64u""%c",&i,&c); else if(s[1] == 'x' || s[1] == 'X') res = sscanf(s + 2,"%""I64x""%c",&i,&c); else res = sscanf(s + 1,"%""I64o""%c",&i,&c); if(res != 1) gep_erreur_full(1 , "D:\\workspace\\gliss\\src\\gep\\gep_addfield.c",132,"Cannot see an uint""64"" in '%s`\n",s); return i;};

**Elijha** · 20/04/2011, 11h38

Bonjour,

Il ne reconnait pas les macro SCNx8 et SCNo8 (ou bien elles n'existent pas dans l'include). A toi de les définir.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#ifndef SCNu8
    #define SCNu8 "hhu"
#endif
#ifndef SCNd8
    #define SCNd8 "hhd"
#endif
#ifndef SCNx8
    #define SCNx8 "hhx"
#endif
#ifndef SCNo8
    #define SCNo8 "hho"
#endif

Ta macro

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
#include <stdint.h>    // Pour uint8_t/uint16_t etc.
 
#define addfield_parse_uint(x)\
uint##x##_t addfield_parse_uint##x(char *s)             \
{                                                       \
    uint##x##_t i ;                                     \
    char c ;                                            \
    int res ;                                           \
    if(s[0]!='0' || s[1]=='\0')                         \
        res = sscanf(s,"%"SCNu##x"%c", &i, &c) ;        \
    else if(s[1]=='x' || s[1]=='X')                     \
    	res = sscanf(s + 2,"%"SCNx##x"%c", &i, &c) ;    \
    else                                                \
    	res = sscanf(s + 1,"%"SCNo##x"%c", &i, &c) ;    \
    if(res!=1)                                          \
        printf("Cannot see an uint"#x" in '%s'\n", s) ; \
    return i ;                                          \
}

nota: J'ai remplacer gep_erreur par un printf.

Ta déclaration

Code :

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

1
2
    // Fonction pour un octet
    addfield_parse_uint(8) ;

Il faut faire pareil pour 16, 32 et 64 biensûr.

**ManusDei** · 20/04/2011, 11h50

C'était bien une question de reconnaissance de macros.

Merci à vous 3, ça compile maintenant (yapuka tester).

macro de fonction

C

Discussions similaires

Partager

Partager