#852 False name collision between overloaded functions

[David Piepgrass]

Give SWIG 1.3.31 this:

void f();
namespace N {
    void f(int);
}

1>swig.i(60): Error: 'f' is multiply defined in the generated module.
1>swig.i(58): Error: Previous declaration of 'f'

It shouldn't happen since they have different arguments.

[Olly Betts]

Still present with trunk shortly before 2.0.0.

[Olly Betts]

Still present with current git master.

[Olly Betts]

Still present in git shortly after 4.2.0.

It also seems to fail even if %nspace N; is used with a target language with %nspace support.

[Olly Betts]

I noticed the problem reported here is actually documented in the manual:

https://www.swig.org/Doc4.2/SWIGPlus.html#SWIGPlus_namespaces

Because namespaces are flattened, it is possible for symbols defined in different namespaces to generate a name conflict in the target language. For example:

namespace A {
  void foo(int);
}
namespace B {
  void foo(double);
}

When this conflict occurs, you will get an error message that resembles this:

example.i:26. Error. 'foo' is multiply defined in the generated target language module.
example.i:23. Previous declaration of 'foo'

However this still seems like a bug to me (it'd be nicer to handle this as an overload, like would happen if the namespaces just weren't there). It especially seems wrong that the collision still happens if %nspace is used.