-
-
Notifications
You must be signed in to change notification settings - Fork 383
Add new macro REF, a variadic and more general XREF #1184
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
83057e6 to
bdf2328
Compare
|
Pushed a DDox implementation but I have no clue how to test it. |
New REF macro introduced in dlang/dlang.org/pull/1184
|
Pushed dlang/phobos/pull/3911 as an example of how it replaces existing macros. |
IIRC the situation is something along the lines of - some of the parameter macros (don't remember which exactly, one of |
dlang.org.ddoc
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See @adamdruppe 's recent PR for simple macros that prefix with dots, underscores etc.
9f9189b to
baa4851
Compare
|
Moved the new |
|
OK, can confirm that the DDox override works. |
baa4851 to
3aea507
Compare
|
Removed duplicate |
3aea507 to
9bd155f
Compare
|
Alright, now edit: Actually, most existing module links in Phobos do not use |
9bd155f to
eca7778
Compare
|
|
Add new macro REF, a variadic and more general XREF
|
LGTM :) |
New REF macro introduced in dlang/dlang.org/pull/1184
|
Thanks! |
Usage is like:
Added to dlang.org.ddoc because that's where XREF is defined. Can be used to link to symbols rooted in
std,core,etcor any other root package.Implementation is a bit messy. I don't know quite why, but using TAIL instead of those helper macros didn't work. Commas showed up in the output.
Will implement for DDox output ASAP.