one header file defines a struct and the other uses it.

[lucky6969b]

// mesh.h
struct D3DXMESHCONTAINER_DERIVED
{
};


//  allocmeshhierarchy.h
#include "mesh.h"
void GenerateSkinnedMesh (LPDIRECT3DDEVICE9 pDevice, D3DXMESHCONTAINER_DERIVED* pMeshContainer);

[translated]
error C2011: D3DXFRAME_DERIVED repeated definition of d: \ users \ documents \ visual studio 2010 \ projects \ perfectsim v5.0 \ perfectsim v5.0 \ mesh \ mesh.h the 11 1 PerfectSim V5.0

What is correct linkage method?
Thanks
Jack

[VictorN]

error C2011: D3DXFRAME_DERIVED repeated definition of d: \ users \ documents \ visual studio 2010 \ projects \ perfectsim v5.0 \ perfectsim v5.0 \ mesh \ mesh.h the 11 1 PerfectSim V5.0

What is correct linkage method?
Thanks
Jack

What does linkage have to do with this compiler error C2011?
And, BTW, I don't see any D3DXFRAME_DERIVED type in the code snippets you have posted.

[Paul McKenzie]

What is correct linkage method?

Where are your include guards for that header? That is a compiler error, and the reason is more than likely that you do not have include guards to prevent the definition being encountered two (or more times) per compilation unit.

Regards,

Paul McKenzie

[lucky6969b]

Sorry, I have solved it myself

First
AllocMeshHierarchy.h
#include "mesh.h"

Second
AllocMeshHierarchy.cpp
#include "AllocMeshHierarchy.h"

third
mesh.h
struct D3DXMESHCONTAINER_DERIVED //etc
Thanks both of you
Jack

[Paul McKenzie]

Sorry, I have solved it myself

That is no solution. All you're doing is delaying the inevitable, and that is two or more inclusions of the header file in a single compilation unit.

The actual solution is to use include guards, as cilu and myself have pointed out. With a more complex program, you would go crazy trying to figure out what header goes first, second, third, etc. without include guards. You may not even have a scenario where you can guarantee that the header is seen only once by the compiler, so how are you going to solve that issue without using include guards?

Regards,

Paul McKenzie

[cilu]

You don't have include guards in your header, which means each symbol defined there will be created once in each translation unit (cpp file) that includes the header. You need either this:

#pragma once

struct D3DXMESHCONTAINER_DERIVED
{
};

or

#ifndef _THEHEADER_
#define _THEHEADER_

struct D3DXMESHCONTAINER_DERIVED
{
};

#endif // end of header