Skip to content

Conversation

@wilzbach
Copy link
Contributor

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

@wilzbach wilzbach force-pushed the stdio_byrecord branch 3 times, most recently from fb5ad93 to ecc43b7 Compare March 13, 2016 19:00
std/stdio.d Outdated

Params:
file = file handle to parse from
format = tuple record $(XREF format,formattedRead format)
Copy link
Contributor 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
std/stdio.d Outdated
The input range set up to parse one line at a time into a record tuple.

Example:
---------------------
Copy link
Contributor

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
Contributor 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

Development

Successfully merging this pull request may close these issues.

3 participants