Bug 12260 – Improve error of std.stdio.readf when involving whitespace

Status
RESOLVED
Resolution
FIXED
Severity
minor
Priority
P2
Component
phobos
Product
D
Version
D2
Platform
All
OS
All
Creation time
2014-02-25T19:56:51Z
Last change time
2018-01-05T13:30:49Z
Assigned to
No Owner
Creator
Jesse Phillips
See also
https://issues.dlang.org/show_bug.cgi?id=17708

Comments

Comment #0 by Jesse.K.Phillips+D — 2014-02-25T19:56:51Z
The following program will throw an exception on the second read. ------ void main() { import std.stdio; int input; readf("%s", &input); readf("%s", &input); } ------ The exceptions message points out the remaining newline from the first input: -------- conv.d(1901): Unexpected ' ' when converting from type LockingTextReader to type int -------- It should probably convert whitespace into a more recognizable form: ------- conv.d(1901): Unexpected '\n' when converting from type LockingTextReader to type int -------
Comment #1 by dave — 2015-05-09T09:30:34Z
Current version of d 2.67.1 will skip the \n, for example the program... ------ void main() { import std.stdio; int input; readf("%s", &input); readf("%s", &input); } ------ Given the input... ------ 1 2 ------ Will emit the error... ------ std.conv.ConvException@/usr/include/dlang/dmd/std/conv.d(2013): Unexpected '2' when converting from type LockingTextReader to type int ------
Comment #2 by dlang-bugzilla — 2017-07-18T13:04:00Z
*** Issue 15297 has been marked as a duplicate of this issue. ***
Comment #3 by github-bugzilla — 2017-07-31T15:37:18Z
Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/afc732661a5cbdd69b22d5bffa57fddab76172ad Fix Issue 12260 - Improve error of std.stdio.readf when involving whitespace https://github.com/dlang/phobos/commit/63de27b5308b6bdb4e0489cc53e87e603f0c95cf Merge pull request #5591 from RazvanN7/Issue_12260 Fix Issue 12260 - Improve error of std.stdio.readf when involving whitespace merged-on-behalf-of: Andrei Alexandrescu <[email protected]>
Comment #4 by github-bugzilla — 2017-08-16T13:24:26Z
Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/afc732661a5cbdd69b22d5bffa57fddab76172ad Fix Issue 12260 - Improve error of std.stdio.readf when involving whitespace https://github.com/dlang/phobos/commit/63de27b5308b6bdb4e0489cc53e87e603f0c95cf Merge pull request #5591 from RazvanN7/Issue_12260
Comment #5 by github-bugzilla — 2018-01-05T13:30:49Z
Commits pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/afc732661a5cbdd69b22d5bffa57fddab76172ad Fix Issue 12260 - Improve error of std.stdio.readf when involving whitespace https://github.com/dlang/phobos/commit/63de27b5308b6bdb4e0489cc53e87e603f0c95cf Merge pull request #5591 from RazvanN7/Issue_12260