Skip to content
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

Improve documentation of std.file.SpanMode (issue 8680) #5569

Merged
merged 1 commit into from
Jul 7, 2017

Conversation

CyberShadow
Copy link
Member

@CyberShadow CyberShadow commented Jul 7, 2017

This clarifies the terminology used for describing the iteration order of dirEntries, as the current terminology has been found misleading.

This clarifies the terminology used for describing the iteration order
of dirEntries, as the current terminology has been found misleading.
@dlang-bot
Copy link
Contributor

Thanks for your pull request, @CyberShadow!

Bugzilla references

Auto-close Bugzilla Description
8680 SpanMode.breadth is incorrectly named and implementation fails in Linux

Copy link
Member

@quickfur quickfur left a comment

Choose a reason for hiding this comment

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

LGTM

@CyberShadow
Copy link
Member Author

Sorry, I missed an overload: #5571

mode = Whether the directory's sub-directories should be
iterated in depth-first port-order ($(LREF depth)),
depth-first pre-order ($(LREF breadth)), or not at all
($(LREF shallow)).
Copy link
Member

Choose a reason for hiding this comment

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

Ddox doesn't seem to understand this: https://dlang.org/library-prerelease/std/file/dir_entries.html
I guess SpanMode.mode would work properly in Ddox?

Copy link
Member Author

Choose a reason for hiding this comment

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

Then I can't use in with LREF.

Maybe there could be a LREF variant that links to the new hierarchical anchors, i.e. HLREF = <a href="#.$1">$(D $1)</a>, but... more macros...

wilzbach added a commit that referenced this pull request Jul 7, 2017
Update a missed dirEntries overload (fixup for #5569)
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.

4 participants