Skip to content

Convert "Const, Immutable and Shared" to a SECTION2#2044

Merged
dlang-bot merged 1 commit intodlang:masterfrom
wilzbach:fix-sections
Jan 2, 2018
Merged

Convert "Const, Immutable and Shared" to a SECTION2#2044
dlang-bot merged 1 commit intodlang:masterfrom
wilzbach:fix-sections

Conversation

@wilzbach
Copy link
Copy Markdown
Contributor

@wilzbach wilzbach commented Jan 2, 2018

@dlang-bot
Copy link
Copy Markdown
Contributor

Thanks for your pull request, @wilzbach!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

spec/class.dd Outdated
where $(I Identifier) is the name generated for the anonymous nested class.

$(SECTION3 $(LEGACY_LNAME2 ConstClass, const-class, Const, Immutable and Shared Classes),
$(SECTION2 $(LEGACY_LNAME2 ConstClass, const-class, Const, Immutable and Shared Classes),
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The doc is weird, only shows "Const". I think you need: $(SECTION2 $(LEGACY_LNAME2 ConstClass, const-class, $(ARGS Const, Immutable and Shared Classes)),

@wilzbach
Copy link
Copy Markdown
Contributor Author

wilzbach commented Jan 2, 2018

The doc is weird,

Yeah this is still a small issue with the TOC generator, but that's unrelated to this PR.
SECTION2 works like SECTION3:

image

Copy link
Copy Markdown
Member

@andralex andralex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

per comment

@andralex
Copy link
Copy Markdown
Member

andralex commented Jan 2, 2018

@wilzbach I figure but can you please do both improvements here? it's quick and easy.

@wilzbach
Copy link
Copy Markdown
Contributor Author

wilzbach commented Jan 2, 2018

@wilzbach I figure but can you please do both improvements here? it's quick and easy.

Done.

@dlang-bot dlang-bot merged commit b347f5f into dlang:master Jan 2, 2018
@wilzbach wilzbach deleted the fix-sections branch July 18, 2018 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants