Improve Variadic function templates article #2224
Conversation
Use sequence instead of tuple. Minor wording changes. Formatting tweaks.
|
Thanks for your pull request and interest in making D better, @ntrel! We are looking forward to reviewing it, and you should be hearing from a maintainer soon.
Please see CONTRIBUTING.md for more information. If you have addressed all reviews or aren't sure how to proceed, don't hesitate to ping us with a simple comment. Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. |
wilzbach
left a comment
There was a problem hiding this comment.
I think we just need to make the HEADERTOC generator happy.
| ) | ||
|
|
||
| $(H3 C++ Solutions) | ||
| $(H3 $(LNAME2 cpp-solutions, C++ Solutions)) |
There was a problem hiding this comment.
This needs to be H2 (and consequently all H2 need to be H3) as the HEADERTOC generation will fail otherwise
| ) | ||
|
|
||
| $(H4 The C++ Extension Solution) | ||
| $(H4 C++ Variadic Templates) |
There was a problem hiding this comment.
H3 + LNAME2 if you want this to appear on the TOC.
|
@wilzbach Fixed using H2, tests pass. I've not added anchors for the other headings though. |
Following on from #2204: