Boost C++ Libraries: Ticket #3501: Reusing concepts from different compile units yields multiple defined symbols

Gunter — Sun, 04 Oct 2009 19:27:36 GMT

(In [56578]) see #3501: Reusing concepts from different compile units yields multiple defined symbols

added BOOST_UBLAS_INLINE macros to requested locations

status changed

Gunter — Sun, 04 Oct 2009 19:44:20 GMT

status new → assigned

@Christoph: please check if the committed version fixes your problem. If not, please provide a small example to reproduce the problem.

anonymous — Mon, 05 Oct 2009 05:48:24 GMT

@Gunter Committed version fixes my problem. I did not test on a fresh trunk checkout. All I did was replacing concepts.hpp of 1.39.0 by current head revision. Thanks for your help.

status changed; resolution set

Gunter — Mon, 05 Oct 2009 22:48:31 GMT

status assigned → closed
resolution → fixed

(In [56614]) fix #3499 and fix #3501:

added inline to free helper functions
added missing concept check
fixed broken internal checks

status changed; resolution deleted

christoph.heindl@… — Fri, 09 Oct 2009 08:20:37 GMT

status closed → reopened
resolution fixed

Hi Gunter,

I fear we need to reopen the bug. In Debug the same linker error as reported occurs because BOOST_UBLAS_INLINE is not defined to inline at all.

Therefore I suggest attributing the free functions with inline explicitely, unless someone knows a better solution.

best regards, Christoph

anonymous — Fri, 09 Oct 2009 18:00:56 GMT

In config.hpp you can find the lines

#ifdef NDEBUG
#ifndef BOOST_UBLAS_INLINE
#define BOOST_UBLAS_INLINE inline
#endif
...

This means that inline is disabled in debug builds in order to be able to follow the execution path. Explicitely defining

#define BOOST_UBLAS_INLINE inline

before including any uBLAS header should solve the problem.

christoph.heindl@… — Fri, 09 Oct 2009 19:10:00 GMT

It would solve the problem indeed.

I wonder, however, if requiring the user to do the bugfixing for a third-party library in his code is the best way to handle this.

Gunter — Wed, 28 Oct 2009 21:07:48 GMT

(In [57202]) see #3501: Reusing concepts from different compile units yields multiple defined symbols (Incomplete)

removed inlines that were added with the last commit
added anonymous namespace to make functions local to compilation unit

Gunter — Wed, 28 Oct 2009 22:54:20 GMT

(In [57208]) see #3501: Reusing concepts from different compile units yields multiple defined symbols (Incomplete)

updated documentation

Gunter — Tue, 06 Apr 2010 20:15:13 GMT

(In [61114]) see #3501: moved release note entry to correct release

status changed; resolution set

Gunter — Tue, 06 Apr 2010 20:18:15 GMT

status reopened → closed
resolution → fixed

(In [61115]) fix #3501: merge changes 57202,57208,61114 into release

boost/numeric/ublas/detail/concepts.hpp libs/numeric/ublas/doc/release_notes.htm