macro préprocesseur code basé donne une erreur C2400

Question

#define CANCEL_COMMON_DIALOG_HOOK(name) \ 
void __declspec(naked) ##name##CancelCommonDialogHook(void) \ 
{ \ 
    __asm \ 
    { \ 
     add  esp, [k##name##CancelCommonDialogStackOffset] \ 
     jz  RESTORE \ 
     jmp  [k##name##CancelCommonDialogNewFileRetnAddr] \ 
    RESTORE: \ 
     pushad \ 
     call DoSavePluginCommonDialogHook \ 
     test eax, eax \ 
     jnz  REMOVE \ 
     popad \ 
     jmp  [k##name##CancelCommonDialogRestoreRetnAddr] \ 
    REMOVE: \ 
     popad \ 
     jmp  [k##name##CancelCommonDialogRemoveRetnAddr]  \ 
    } \ 
} 

En utilisant la macro ci-dessus provoque le compilateur pour lancer cette erreur:

error C2400: inline assembler syntax error in 'second operand'; found 'RESTORE'

Qu'ai-je fait mal?

EDIT:

void __declspec(naked) #name##CancelCommonDialogHook(void)    \ 
{                  \ 
    __asm add  esp, [k##name##CancelCommonDialogStackOffset]  \ 
    __asm jz  RESTORE            \ 
    __asm jmp  [k##name##CancelCommonDialogNewFileRetnAddr]   \ 
    RESTORE:                \ 
    __asm pushad              \ 
    __asm call DoSavePluginCommonDialogHook       \ 
    __asm test eax, eax            \ 
    __asm jnz  REMOVE            \ 
    __asm popad              \ 
    __asm jmp  [k##name##CancelCommonDialogRestoreRetnAddr]   \ 
    REMOVE:                \ 
    __asm popad              \ 
    __asm jmp  [k##name##CancelCommonDialogRemoveRetnAddr]   \ 
} 

Le code ci-dessus ne fonctionne pas non plus:

error C2447: '{' : missing function header (old-style formal list?) error C2014: preprocessor command must start as first nonwhite space

Answer 1

L'a corrigé en incluant le corps de la fonction dans une autre étendue.

Answer 2

Au moins la dernière fois que je l'ai essayé, vous ne pouviez pas créer des étiquettes à l'intérieur d'un bloc de montage en ligne dans VC++. L'utilisation d'étiquettes de type C fonctionne bien:

void __declspec(naked) ##name##CancelCommonDialogHook(void) \ 
{ \ 
    __asm \ 
    { \ 
     add  esp, [k##name##CancelCommonDialogStackOffset] \ 
     jz  RESTORE \ 
     jmp  [k##name##CancelCommonDialogNewFileRetnAddr] \ 
    }   \ 
    RESTORE: \ 
    _asm {  \ 
     pushad \ 
     call DoSavePluginCommonDialogHook \ 
     test eax, eax \ 
     jnz  REMOVE \ 
     popad \ 
     jmp  [k##name##CancelCommonDialogRestoreRetnAddr] \ 
    }   \ 
    REMOVE: \ 
    __asm { \ 
     popad \ 
     jmp  [k##name##CancelCommonDialogRemoveRetnAddr]  \ 
    } \ 
} 

Je n'ai pas écrit tout ensemble en ligne dans VC++ pour un certain temps, donc je ne peux pas garantir que cela fonctionnera, mais je suppose qu'il ya une chance assez juste .

Answer 3

J'ai été sans accès à l'ordinateur, donc j'espère que vous avez déjà résolu cela. Je pense que le problème est que l'utilisation de "\" pour terminer la ligne indique au préprocesseur C de fusionner la ligne suivante avec cette ligne. Voir le commentaire 3 sur this page, et l'épissage de ligne here. Cela fonctionne bien pour la plupart des instructions C, mais est plus problématique pour l'assemblage, car les nouvelles lignes sont la manière dont il déline les instructions.

Je peux penser à deux approches d'une solution. Le premier est de trouver quelque chose comme C ";" qui peut être utilisé comme un séparateur de statment dans l'assemblage, je ne sais pas si une telle chose existe. La deuxième approche consiste à tout emballer dans des instructions distinctes __asm. Prendre la deuxième approche que vous obtenez, ce qui suit:

void __declspec(naked) ##name##CancelCommonDialogHook(void)   
{                  
    __asm{add  esp, [k##name##CancelCommonDialogStackOffset]}  \ 
    __asm{jz  RESTORE}           \ 
    __asm{jmp  [k##name##CancelCommonDialogNewFileRetnAddr]}  \ 
    RESTORE:               \ 
    __asm{pushad}              \ 
    __asm{call DoSavePluginCommonDialogHook}      \ 
    __asm{test eax, eax}           \ 
    __asm{jnz  REMOVE}            \ 
    __asm{popad}              \ 
    __asm{jmp  [k##name##CancelCommonDialogRestoreRetnAddr]}  \ 
    REMOVE:               \ 
    __asm{popad}              \ 
    __asm{jmp  [k##name##CancelCommonDialogRemoveRetnAddr]}  \ 
} 

NOTE: Je l'ai laissé les étiquettes en dehors des déclarations asm parce que:

1. Je pense que vous pouvez
2. Je ne suis pas sûr les règles de cadrage pour les étiquettes définies dans __asm blocs

Answer 4

Juste une supposition sauvage: la macro développer tout le texte à une seule ligne, de se retrouver avec add esp, [k...] jz RESTORE jmp k.... Peut-être qu'il est utile de mettre un point-virgule à la fin de chaque instruction de l'assembleur.

Une indication pour cette hypothèse est que l'erreur se produit sur votre deuxième "ligne". Le premier est correct, mais le second sera fusionné au premier, c'est donc la première occasion pour le compilateur d'être confus. Si l'erreur avait été quelque part plus tard, ce ne serait probablement pas le cas.