Update cpp_interface.dd for recent developments #1154
Conversation
| ) | ||
|
|
||
| $(TROW | ||
| $(ARGS $(B core.stdc.stddef.wchar_t)), |
There was a problem hiding this comment.
This line seems to be causing some error in the LaTex generation.
There was a problem hiding this comment.
It's because of the underscore I think. Use $(UNDERSCORE) instead.
There was a problem hiding this comment.
Or use $(D ...), as is more appropriate here because you're referring to an identifier.
There was a problem hiding this comment.
Yeah, that makes sense. I changed it to use $(D ...) and the test passes now.
JakobOvrum
force-pushed
the
update_cpp_interface
branch
from
8f5fc5b to
dfa1482
Compare
|
I filed the linking issue: https://issues.dlang.org/show_bug.cgi?id=15372 |
JakobOvrum
force-pushed
the
update_cpp_interface
branch
from
dfa1482 to
4810674
Compare
|
|
So this is blocked by 15372? If so it would be useful if someone added the "Blocked" tag to this. |
|
Well, I could just remove function templates from the article. I just can't claim it works when it doesn't. |
|
I'd say add a note about bug 15372 and then merge this, as this shows how C++ inter-op should work baring bugs and that's how we document everything else. |
|
@JakobOvrum rebase please |
JakobOvrum
force-pushed
the
update_cpp_interface
branch
from
4810674 to
d871767
Compare
|
Is the |
Seems so. |
JakobOvrum
force-pushed
the
update_cpp_interface
branch
from
d871767 to
4a805d4
Compare
Fix issue 10987
JakobOvrum
force-pushed
the
update_cpp_interface
branch
from
4a805d4 to
04427d2
Compare
|
Is there a way to see the page somewhere ? |
|
I just merged 1194, if the test re-runs here, ping me and I'll merge this too. |
|
@adamdruppe The test has passed now. |
Update cpp_interface.dd for recent developments
|
Thanks for this. When will this actually go live on the website? |
|
The next time I, Martin or Andrei syncs it. Would you like me to sync it now? |
That would be nice. Thanks. People ask about C++ support from time to time on the forums, and I'd like to finally be able to just say, "go read the spec". |
|
Thanks. Any idea why "COM interface" at the top of the page is a dead link? |
|
It seems to be using the wrong macro. But http://dlang.org/COM.html is just a redirect to a wiki page with one line in it. The link should probably just be removed. |
I made a PR: #1249 |
Depends on: issue 15372. The template example doesn't link.
The first commit contains only the tabs to spaces changes, so it's probably easier to review using the diff for the second commit.