Discussion :

[Nono Sto]

Hello

Dans un header nommé documented.h j'ai ecris la definition suivante:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
VOID NTAPI HalReadWritePCISpace
(
  IN ULONG   BusNumber,
  IN ULONG   SlotNumber,
  IN ULONG   RegisterNumber,
  IN PVOID   Buffer,
  IN ULONG   Length,
  IN BOOLEAN WritePCISpace
)
{
    return;
}

Dès lors des que j'inclue dans un entête documented.h j'ai ce message et je comprend pas:

Code :

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

1
2
3
 
pci.obj : error LNK2005: "void __stdcall HalReadWritePCISpace(unsigned long,unsigned long,unsigned long,void *,unsigned long,unsigned char)" (?HalReadWritePCISpace@@YGXKKKPAXKE@Z) already defined in main.obj
Debug/test.exe : fatal error LNK1169: one or more multiply defined symbols found

Auriez vous une solution SVP?

Merci

[koala01]

Salut,

N'aurais tu pas, par le plus grand des hasards, inclus le fichier d'implémentation (*.cpp) qui contient cette fonction dans ton fichier main.cpp

N'as tu pas encore compris qu'il ne fallait JAMAIS inclure un fichier .cpp, qu'il n'y avait que les fichier d'en-tête qui devait être utilisés avec la directive #include

[dalfab]

Et il ne faut jamais définir de fonction dans un fichier header.
A chaque inclusion du header la fonction sera créée et recréée, ce qui est l'erreur que tu obtiens.

[Nono Sto]

Merci

J'ai inclu aucun .cpp nul part parcontre je n'ai fait dans documented.h qu'une définition. Du coup j'ai essayé de faire une déclaration dans le documented.h et à la compilation aucun souci:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
VOID NTAPI HalReadWritePCISpace
(
  IN ULONG   BusNumber,
  IN ULONG   SlotNumber,
  IN ULONG   RegisterNumber,
  IN PVOID   Buffer,
  IN ULONG   Length,
  IN BOOLEAN WritePCISpace
);

Par contre dès que je créer le documented.cpp et que j'inclue le documented.h j'ai un paquet d'erreur sans même avoir écrit une ligne:

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
 
 
Compiling...
undocumented.cpp
c:\...\documented.h(33) : error C2146: syntax error : missing ';' before identifier 'NTSTATUS'
c:\...\documented.h(33) : error C2501: 'NTSTATUS' : missing storage-class or type specifiers
c:\...\documented.h(43) : error C2146: syntax error : missing ';' before identifier 'SHORT'
c:\...\documented.h(43) : error C2146: syntax error : missing ';' before identifier 'CSHORT'
c:\...\documented.h(43) : error C2501: 'SHORT' : missing storage-class or type specifiers
c:\...\documented.h(43) : error C2501: 'CSHORT' : missing storage-class or type specifiers
c:\...\documented.h(47) : error C2143: syntax error : missing ';' before '*'
c:\...\documented.h(47) : error C2501: 'PSZ' : missing storage-class or type specifiers
c:\...\documented.h(48) : error C2143: syntax error : missing ';' before '*'
c:\...\documented.h(48) : error C2501: 'PCSZ' : missing storage-class or type specifiers
c:\...\documented.h(53) : error C2146: syntax error : missing ';' before identifier 'Length'
c:\...\documented.h(53) : error C2501: '_STRING::USHORT' : missing storage-class or type specifiers
c:\...\documented.h(53) : error C2501: '_STRING::Length' : missing storage-class or type specifiers
c:\...\documented.h(54) : error C2146: syntax error : missing ';' before identifier 'MaximumLength'
c:\...\documented.h(54) : error C2501: '_STRING::USHORT' : missing storage-class or type specifiers
c:\...\documented.h(54) : error C2501: '_STRING::MaximumLength' : missing storage-class or type specifiers
c:\...\documented.h(55) : error C2146: syntax error : missing ';' before identifier 'Buffer'
c:\...\documented.h(55) : error C2501: '_STRING::PCHAR' : missing storage-class or type specifiers
c:\...\documented.h(55) : error C2501: '_STRING::Buffer' : missing storage-class or type specifiers
c:\...\documented.h(67) : error C2146: syntax error : missing ';' before identifier 'Status'
c:\...\documented.h(67) : error C2501: '_IO_STATUS_BLOCK::__unnamed::NTSTATUS' : missing storage-class or type specifiers
c:\...\documented.h(67) : error C2501: '_IO_STATUS_BLOCK::__unnamed::Status' : missing storage-class or type specifiers
c:\...\documented.h(68) : error C2146: syntax error : missing ';' before identifier 'Pointer'
c:\...\documented.h(68) : error C2501: '_IO_STATUS_BLOCK::__unnamed::PVOID' : missing storage-class or type specifiers
c:\...\documented.h(68) : error C2501: '_IO_STATUS_BLOCK::__unnamed::Pointer' : missing storage-class or type specifiers
c:\...\documented.h(71) : error C2146: syntax error : missing ';' before identifier 'Information'
c:\...\documented.h(71) : error C2501: '_IO_STATUS_BLOCK::ULONG_PTR' : missing storage-class or type specifiers
c:\...\documented.h(71) : error C2501: '_IO_STATUS_BLOCK::Information' : missing storage-class or type specifiers
c:\...\documented.h(77) : error C2065: 'NTAPI' : undeclared identifier
c:\...\documented.h(77) : error C2065: 'PIO_APC_ROUTINE' : undeclared identifier
c:\...\documented.h(77) : error C2501: 'VOID' : missing storage-class or type specifiers
c:\...\documented.h(77) : fatal error C1903: unable to recover from previous error(s); stopping compilation

J'ai ces erreurs que dans ce cas là.

[dalfab]

Tu nous indiques un fichier de 10 lignes qui a des erreurs qui apparaissent de la ligne 33 à la ligne 77, ça ne pas facile du tout pour pouvoir t'aider!

Au feeling, je dirais qu'il manque des #include dans le fichier documented.h.

[Nono Sto]

En fait pour l'instant tant que je definis dans documented.h toute les fonctions pas de probleme mais si je fais sa dans un .cpp j'ai des erreur.

[koala01]

Tu nous indiques un fichier de 10 lignes qui a des erreurs qui apparaissent de la ligne 33 à la ligne 77, ça ne pas facile du tout pour pouvoir t'aider!

Au feeling, je dirais qu'il manque des #include dans le fichier documented.h.

En fait, il y a de très fortes chances pour que le compilateur ait en réalité "perdu les pédales" à la ligne 33, lorsqu'il n'a pas compris ce que représentait NTSTATUS (qui n'apparait d'ailleurs pas dans le code montré ), et que les autres erreurs ne soient que des "faux positifs" induits par cette "perte des pédales".

Mais, de fait, je dirais sensiblement la même chose au feeling

[koala01]

En fait pour l'instant tant que je definis dans documented.h toute les fonctions pas de probleme mais si je fais sa dans un .cpp j'ai des erreur.

Ben oui, mais il y a des règles pour définir les fonctions dans un fichier d'en-tête...

La principale d'entre-elles étant que tu dois t'assurer que l'éditeur de liens considérera qu'il est "normal" de se retrouver avec plusieurs implémentations (au niveau du code binaire généré) de la même fonction; par exemple, en la déclarant inline.

Et, de toutes manières, le fait de déporter la définition d'une fonction dans un fichier d'implémentation ne devrait pas avoir de répercussion sur l'analyse du fichier d'en-tête dans lequel elle se trouvait à l'origine.

"Dans le pire des cas", l'implémentation déportée sera dans un fichier qui ne sera pas compilé à cause d'un problème de définition du projet, ce qui se traduira par une erreur lors de l'édition de liens.

Si bien que, au vu des deux situations dont nous avons connaissance, à savoir:

1. l'implémentation d'une fonction dans le fichier d'en-tête qui compile mais provoque une erreur à l'édition de liens et
2. l'implémentation d'une fonction dans un fichier d'implémentation qui provoque une erreur de compilation lors de l'analyse du fichier d'en-tête

je suis désolé d'avoir à te le dire, mais il n'y a qu'une seule conclusion à tirer: tu as de toutes évidence fait "une connerie" en déportant la fonction qui posait problème, sans doute en supprimant l'inclusion d'un fichier d'en-tête que tu auras peut-être jugé (à tors) comme inutile

[Nono Sto]

Le problème est que j'ai du récupérer ce fameux documented.h et il est délicat à modifier. Du coup la meilleur solution est d'ajouter les définitions des quelques fonctions que je dois ajouter dans le documented.h. Alors quels conseil me donneriez vous SVP pour essayé de faire sa propre?

[Bousk]

Tu utilises quel compilateur ?
Est-il trop stupide pour redéfinir la fonction à chaque include plutôt que de l'inliner ?
Puisque tu implémentes la fonction dans un fichier header, pourquoi n'est-elle pas inline ?

[Médinoc]

Je sais qu'on dit parfois que lorsqu'on écrit une implémentation, il est de bon ton de commencer par le #include spécifique, mais dans le cas d'un dev Windows, je pense que commencer par <windows.h> (ou si l'on utilise des en-têtes précompilés, "stdafx.h") prend le pas sur cette règle de style.