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

document std.stdio.byRecord #4079

Merged
merged 1 commit into from Mar 28, 2016
Merged

Conversation

wilzbach
Copy link
Member

follow-up to #4075 / #4053 - this PR just adds documentation for byRecord (as suggested by @ntrel)


Params:
file = file handle to parse from
format = tuple record $(XREF format,formattedRead format)
Copy link
Member Author

Choose a reason for hiding this comment

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

I still have to figure out which of all those macro links actually works.

Copy link

Choose a reason for hiding this comment

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

Try to use REF instead, it's a quite new, more generic, macro that replaces LREF, XREF, etc.

dlang/dlang.org#1184

@wilzbach wilzbach force-pushed the stdio_byrecord branch 2 times, most recently from 13f52fa to c9e4cbb Compare March 14, 2016 18:46
The input range set up to parse one line at a time into a record tuple.

Example:
---------------------
Copy link
Member

Choose a reason for hiding this comment

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

Why not remove this and add a blank doc comment to the unit test below?

Copy link
Member

Choose a reason for hiding this comment

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

Ping @greenify

@wilzbach
Copy link
Member Author

Ping @greenify

Sorry lost track of it - I found the reason why REF_ALTTEXT wasn't working. format is a parameter and thus put in italic - using the underscore magic for format works :)

@DmitryOlshansky
Copy link
Member

Auto-merge toggled on

@DmitryOlshansky DmitryOlshansky merged commit 8ff1f0e into dlang:master Mar 28, 2016
@wilzbach wilzbach deleted the stdio_byrecord branch March 30, 2016 04:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants