Discussion :

[vdemeester]

Bonjour,

Je travail sur un code, qui n'est pas de moi, où je trouve beaucoup beaucoup de macros (sur plusieurs lignes). Outre le fait que c'est assez moche, et assez difficilement debugguable (enfin je trouve), il y a des petites subtilités. J'ai le code suivant :

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
#define m_OasisMsg_sizefct(type) \
static unsigned short MaxSizeofMsg() {return(sizeof(Oasis##type)                            \
                                        -sizeof(RadioMsgHeader::sizeofLength()));}; \
static unsigned short sizeofMsgWithoutData( void )                                  \
{                                                                                   \
    return(MaxSizeofMsg() - MaxSizeofDataPart());                                   \
}                                                                                   \
unsigned short sizeofMsg( void )                                                    \
{                                                                                   \
    //..
}
 
/*
 * qui est appelée comme ce qui suit, dans une classe par exemple.
 */
 
m_OasisMsg_BASIC_FUNCTIONS(StartConnexionReq,E_MNGT_START_CONNEXION_REQ
                              ,C_ACK_REQUESTED)

Je n'avais au départ par la moindre idée de la signification des ##. J'ai alors lu la documentation du préprocesseur, de GCC. J'ai lu que c'était utilisé pour faire de la stringification et de la concatenation, et tout s'est éclaircit. En gros, dans l'exemple on veut au final faire un sizeof de OasisStartConnexionReq, qui est une classe. Le seul soucis, c'est que gcc n'a pas l'air de le prendre comme ça, ou en tout cas je me récupère des erreurs du type error: init_Msg_header' undeclared (first use this function), fonction qui est normalement définit par cette macro. Je me pose donc 2 questions :

1. Le project a été codé au départ avec Visual Studio 6, qui, si je ne me trompe pas, ne supporte pas vraiment les standards puisqu'il est presque aussi vieux . Est-ce le comportement est le même pour le préprocesseur de VC++ 6 et gcc ?
2. Dans la documentation du préprocesseur, j'ai cru ne pas comprendre, des histoire d'espace. Est-ce réellement la bonne syntaxe ?

L'exemple donnée dans la documentation du préprocesseur de gcc marche bien (ce qui est normal ), et est le suivant.

Code :

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

1
2
3
4
5
6
7
#define COMMAND(NAME)  { #NAME, NAME ## _command }
struct command commands[] =
{
  COMMAND (quit),
  COMMAND (help),
  ...
};

Enfin, une autre utilisation est le cas suivant. Et là, c'est le drame pour ma compréhension.

Code :

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

1
2
3
4
5
#define m_OasisMsg_init_Bobo(code,ackFlag)            \
void init_Bobo(unsigned char AckFlag=##ackFlag){    \
 this->msg_header.init(ntohs(code)                          \
                          ,ntoh(sizeofMsg()),AckFlag);      \
}

Si quelqu'un a des pistes, ça pourrait m'être grandement utile . (en espérant de pas poster encore une fois de plus dans le mauvais forum)

[Invité]

eh, c'est pas parce qu'on dit beaucoup de choses contre microsoft et sa politique du monopole, que son compileur VC6 ne comprend pas le C et le C++ (avec le standard qui n'etait pas encore paru a l'époque)... ce genre de syntaxe est parfaitement implémentée et comprise par son préprocesseur...

[D[r]eadLock]

Pour le deuxieme exemple (avec ackFlag), il n'y a pas lieu de mettre ## puisqu'on ne va pas concatener '=' avec 'ackFlag' !!! (ca genere je crois des warning 'invalid token....').

Quant a ton pb de compil' (init_Msg_header), je ne vois pas pas ou c'est declarer dans le code que tu donne.

Pour debugger le code apres passage du pre-compilo, tu peux utiliser -E avec gcc

Code :

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

 gcc <tes options sauf -c et -o> -E toto.c > toto.post.c

[vdemeester]

eh, c'est pas parce qu'on dit beaucoup de choses contre microsoft et sa politique du monopole, que son compileur VC6 ne comprend pas le C et le C++ (avec le standard qui n'etait pas encore paru a l'époque)... ce genre de syntaxe est parfaitement implémentée et comprise par son préprocesseur...

Bah, c'était une question, sachant que le soucis que j'ai, je l'ai avec g++ et pas avec VC++ 6

Le init_Msg_header est déclaré dans le #define m_OasisMsg_sizefct(type) (je ne l'ai juste pas mis, car il y avait beaucoup trop de ligne). Il est alors fort probable que l'erreur soit due à autre chose, mais, j'avais envie de savoir/comprendre s'il n'y avait pas de subtiles différences de ce type entre g++ et vc++ 6.

Je vais tenter de me passer de cette syntaxe.

[vdemeester]

Si je prends cetet déclaration par exemple

Code :

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

#define mGLB_SetAttribut(attr) this->##attr = fRight.##attr;

Elle fonctionne sous Visual C++ 6, mais apparament pas avec g++. On se retrouve ici dans le cas énoncé par D[r]eadLock, et les ## sont inutiles, et il me sort de toute façon une erreur (c'est le même cas que le coup du = mais avec . et ->). Je commence à comprendre .

VC++ 6 a donc l'air d'être assez laxiste, no ?

[Alp]

Il l'est, du fait qu'il ne respecte pas un certain nombre de points de la norme.

Cependant, les dernières versions améliorent _nettement_ cela.