Multiple definitions

[Martin Magnusson]

I have a problem with multiple definitions that I can't quite straighten
out.

I have a templated class defined inside a namespace, and I want to
create a function in that namespace that works on a specific instance of
the templated class.

Like this:

// file Numerical.hpp:
namespace Numerical
{
template<int N>
class Vector { ... };

Vector<3> convert( Vector<3> v )
{ ... }

}

However, when doing it like that I get complaints from gcc, during
linking, about multiple definitions of Numerical::conv ert. If I write
convert as a member function instead, it all works fine. The whole
Numerical::Vect or class is defined in the hpp file. I'm pretty sure I
don't have any multiple includes, and that all of my header files have
the proper #ifndef guards.

What could be the problem?

[John Harrison]

"Martin Magnusson" <[e-mail]> wrote in message
news:1095412402 .4V9RMbK7zaKQ2z dOMce88Q@terane ws...

I have a problem with multiple definitions that I can't quite straighten
out.

I have a templated class defined inside a namespace, and I want to
create a function in that namespace that works on a specific instance of
the templated class.

Like this:

// file Numerical.hpp:
namespace Numerical
{
template<int N>
class Vector { ... };

Vector<3> convert( Vector<3> v )
{ ... }

}

However, when doing it like that I get complaints from gcc, during
linking, about multiple definitions of Numerical::conv ert. If I write
convert as a member function instead, it all works fine. The whole
Numerical::Vect or class is defined in the hpp file. I'm pretty sure I
don't have any multiple includes, and that all of my header files have
the proper #ifndef guards.

What could be the problem?

convert is not a template function so if you want to put it in a header file
you must declare it inline. Include guards are not the issue. If you include
the header file in more than one source file you are going to get multiple
definitions of any non-templated non-inline function.

Alternatively move convert to a source file, and just leave the prototype in
the header file. This is probably what you should do.

john

[Greg Comeau]

In article <2q************ *@uni-berlin.de>,
John Harrison <jo************ *@hotmail.com> wrote:

"Martin Magnusson" <[e-mail]> wrote in message
news:109541240 2.4V9RMbK7zaKQ2 zdOMce88Q@teran ews...

I have a problem with multiple definitions that I can't quite straighten
out.

I have a templated class defined inside a namespace, and I want to
create a function in that namespace that works on a specific instance of
the templated class.

Like this:

// file Numerical.hpp:
namespace Numerical
{
template<int N>
class Vector { ... };

Vector<3> convert( Vector<3> v )
{ ... }

}

However, when doing it like that I get complaints from gcc, during
linking, about multiple definitions of Numerical::conv ert. If I write
convert as a member function instead, it all works fine. The whole
Numerical::Vect or class is defined in the hpp file. I'm pretty sure I
don't have any multiple includes, and that all of my header files have
the proper #ifndef guards.

What could be the problem?

convert is not a template function so if you want to put it in a header file
you must declare it inline. Include guards are not the issue. If you include
the header file in more than one source file you are going to get multiple
definitions of any non-templated non-inline function.

Alternativel y move convert to a source file, and just leave the prototype in
the header file. This is probably what you should do.

Yup. I would suggest the OP get a copy of Stroustrup's
"The C++ Programming Language" (see http://www.comeaucomputing.com/booklist )
and see Chapter 9 regarding some throughts on program construction
and file organization.
--
Greg Comeau / Comeau C++ 4.3.3, for C++03 core language support
Comeau C/C++ ONLINE ==> http://www.comeaucomputing.com/tryitout
World Class Compilers: Breathtaking C++, Amazing C99, Fabulous C90.
Comeau C/C++ with Dinkumware's Libraries... Have you tried it?