[trivial] Tweak std.meta docs#6146
Conversation
Replace uses of 'list' with sequence. S[n] is not the nth element (zero based). Add links, formatting.
ntrel
requested review from
MetaLang,
PetarKirov and
dnadlinger
as code owners
|
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. |
std/meta.d
Outdated
| * such as TL[$(I n)] which gets the $(I n)th type from the | ||
| * alias sequence. TL[$(I lwr) .. $(I upr)] returns a new type | ||
| * list that is a slice of the old one. | ||
| * Some operations on alias sequences are built into the language - |
There was a problem hiding this comment.
The comma was correct. - is not a valid way to make a sub-clause of a sentence.
|
|
||
| /** | ||
| * Returns the `AliasSeq` TList with the types sorted so that the most | ||
| * Returns an `AliasSeq` with the elements of TList sorted so that the most |
There was a problem hiding this comment.
Since you're modifying this line anyway, might as well change
`AliasSeq`
to
$(LREF AliasSeq)
Same throughout.
MetaLang
left a comment
MetaLang
left a comment
There was a problem hiding this comment.
Other than the change to the comma pointed out by @JackStouffer, LGTM.
|
I restored the comma, I've not made the LREF change. I changed a link title to match dlang/dlang.org#2204. |
Any particular reason? |
|
@JackStouffer Just that I didn't want to get into another change unless it added significant value. Merge please? ;-) |
4 participants
S[n]is not the nth element (due to zero based indexing).