Bug#584172: Disabling GetIDMapper inlining for building VISU

[Denis Barbier]

On 2010/6/2 Andre Espaze wrote:
> Hello Denis,
>
> On Wed, Jun 02, 2010 at 10:30:15AM +0200, Denis Barbier wrote:
>> On 2010/6/2 Andre Espaze wrote:
>> > It is finally not necessary to build the module with the -g option,
>> > I have enclosed a patch that disable the GetIDMapper function inlining
>> > when built with g++ and optimizations.
>>
>> I am no C++ expert, but this looks very similar to
>>    http://www.parashift.com/c++-faq-lite/templates.html#faq-35.13
>> and it is thus not clear whether this is really a g++ bug.
>> Here is a different fix, I tested this approach with Andre's
>> test.cpp, but not with salome.
> Thank you very much for the suggestion, it works but it seems to
> me that you forgot a 'template' keyword before the symbol export.

Oops, you are fully right, sorry.  A new patch is attached for reference.

> I have enclosed a patch tested on my machine, fell free to correct
> it if I was wrong.
[...]

I do not understand why you added #if tests, just adding 'template' at
the very beginning is enough.  It was not explicit in my previous
mail, but I tried to find an alternative solution because your's works
only with g++, and you will need those #if tests if you want to
forward it upstream.  OTOH mine is portable, and it could be adopted
more easily by upstream, I guess.  Within Debian, which patch is
checked in does not matter.

Denis
-------------- next part --------------
A non-text attachment was scrubbed...
Name: instantiation2.patch
Type: application/x-empty
Size: 672 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/debian-science-maintainers/attachments/20100602/16387f2b/attachment.bin>