Workaround Issue 17081 - Bodies in extern cpp functions in D files are not linked

[Darredevil]

https://issues.dlang.org/show_bug.cgi?id=17081

[kinke]

Why do you implement these functions? Isn't adding the const qualifier to name() enough to fix the linking error? As a side note, you don't check whether _arg for before() is null, but claim that the function is nothrow...

[Darredevil]

@kinke adding const was not enough to fix the problem, see the bug tracker link for example. Implementing them was the only way to fix the problem.
As for not checking for null, it's because I translated the exact source code from the GNU stdcpp, however you are correct, the nothrow will be removed.

[dnadlinger]

From the bug tracker:

I propose actually implementing all non-virtual functions from druntime/src/core/stdcpp in order to solve this.

Before rushing in a fix, it is important to figure out what is going wrong here. Linking against non-virtual C++ function is supposed to work (unless, of course, they are static/inline on the C++ side). Comparing the mangled name would be a first step.

[andralex]

Before rushing in a fix, it is important to figure out what is going wrong here.

This is a fine workaround. We'll end up implementing a few functions on the C++ side for inlining purposes anyway.

[andralex]

spaces around :, cc @wilzbach

[andralex]

no space before ( in function call (i.e. use math-like notation), cc @wilzbach

[andralex]

BTW this shall not be considered a fix, so I'll change the title.

[Darredevil]

@andralex Fixed spacing, waiting final tests for merge

[andralex]

Auto-merge toggled on

[andralex]

@Darredevil great, thx. Next step would be probably to actually see the problem with https://issues.dlang.org/show_bug.cgi?id=17081.

Also please push this again, looks like the autotester failed for no cause. cc @MartinNowak @braddr

[kinke]

@andralex: Related & trivial, but sitting there for more than a month already and mentioning more C++ interop issues: #1709