Discussion :

[Astraya]

Salut,

Voila je voyage dans du code existant et il y a une chose que je vois souvent dans les codes type Win32, Emscripten et autre... C'est de rajouter un define video en début du fonction genre:

Code :

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

1
2
#define CALLBACK
void(CALLBACK*drawFunc)(struct_escontext*)

Quel est l'intérêt de faire ça? Je vois bien un intérêt documentaire, mais y a t-il un intérêt de compilation?

Merci

[koala01]

Salut,
à voir c qui suit ce define ( void(CALLBACK*drawFunc)(struct_escontext*) ), j'aurais tendance à penser que cela nous permet de prévoir un point de variation pour ce qui vient juste avant l'étoile dans la première paire de parenthèse.

Si CALLBACK est défini à une valeur vide, cela nous donne un callback proche de void(*drawFunc)(struct_escontext*) mais, si le besoin se fait sentir, il "suffit" de donner une valeur différente pour le transformer en quelque chose comme void(Renderer::*drawFuncà(struct_escontext *) le jour où l'on décide d'abandonner C+ au profit de C++.

Bien sur, ce n'est qu'un exemple parmi tant d'autre du genre de modification que ce symbole est susceptible de permettre "en une second"

[foetus]

J'ai sorti mon compilateur parce que je pensais que cela permettait de réinitialiser la macro
En fait, pas du tout: j'ai eu une erreur de redéfinition de macro

Je penche donc pour un truc sale : une macro qui était utilisée, mais après les aléas du code, elle a perdu sa valeur.
Après, peut-être que les dévs ont voulu garder la possibilité de remettre une valeur ... ou ils n'ont pas pris le temps de nettoyer le code.

Édit: CALLBACK fait penser aux x86 calling conventions, ce qui n'est pas déconnant pour une vieille API bas niveau comme le win32

Je pense éventuellement à une espèce d'uniformisation dans le code: on retrouve certaines macros dans [tous les]/ [une grosse majorité des] modules de compilation (fichier.cpp + les include)

Sinon une macro vide peut être utile et moi même je l'utilise dans ce cas (transposé en C):

Code :

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

1
2
3
4
5
#ifdef DEBUG
#define DLOG( X ) printf X;
#else
#define DLOG( X )
#endif

qui s'utilise ainsi:

Code :

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

1
2
    int i = 9; float f = 2.25f;
    DLOG( ("i: %d\nf: %f\n", i, f)  );

[renoo]

Je pense que dans l'idée ça sert à faire un truc comme ça :

Code :

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

1
2
3
4
5
#ifdef MON_VARIANT_SOUS_WINDOWS
#define CALLBACK __stdcall
#else
#define CALLBACK
#endif

Ça doit servir pour la convention d'appel (ordre de passage des paramètres sur la pile, gestion de celle ci...).

[Astraya]

Merci pour les retours, je vois très bien l'intérêt de pouvoir activité certaines fonction du code en fonction du type de compilation comme les assert en debug et pas en release ou les log également. Mais dans le code que je vois il n'y a que une définition vide... donc bon... je présume que c'est du code déchet qui n'a pas été enlevé...

J'avais bien pensé à ta solution koala mais dans ce cas la si la fonction callback devient membre, la fonction qui la setup est également différente car il faudra passer une instance en paramètre lors de l'enregistrement du callback...

Bref, je présume que le code n'est pas au top sachant que il ne défini que le define vide sans autres ifdef... Donc je ne vois qu'un intérêt documentaire..

Merci encore pour vos idées et retours.

[the Hound]

Je rejoins l'avis de renoo : dans ce genre de constructions, le define est souvent utilisé pour préciser la convention d'appel.

[Médinoc]

Il est possible que CALLBACK ait été défini ici pour compiler sous *n*x un code initialement prévu pour Windows, par exemple.