Thread overview
[Issue 8908] New: Collapse of std.csv by the specifications change of std.conv.parse
Oct 29, 2012
SHOO
Dec 11, 2012
Kenji Hara
Dec 19, 2012
Kenji Hara
October 29, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=8908

           Summary: Collapse of std.csv by the specifications change of
                    std.conv.parse
           Product: D
           Version: D2
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: regression
          Priority: P2
         Component: Phobos
        AssignedTo: nobody@puremagic.com
        ReportedBy: zan77137@nifty.com


--- Comment #0 from SHOO <zan77137@nifty.com> 2012-10-29 06:27:45 PDT ---
This code doesn't work on git head(53527bbece), and the code works well on dmd
2.060:

----------------------------
import std.csv, std.stdio;

string csv = `  1.0, 1.1, 1.2
                2.0, 2.1, 2.2
`;

void main()
{
    static struct Data { real a, b, c; }
    foreach (data; csvReader!Data(csv)) with (data)
    {
        writeln([a, b, c]);
    }
}
---------------------------
In dmd 2.060:
[1, 1.1, 1.2]
[2, 2.1, 2.2]

In git head:
std.csv.CSVException@/usr/local/include/dmd2git/std/csv.d(1047): no digits seen
----------------
----------------
std.conv.ConvException@/usr/local/include/dmd2git/std/conv.d(2402): no digits
seen
----------------
----------------
std.conv.ConvException@/usr/local/include/dmd2git/std/conv.d(1631): Unexpected ' ' when converting from type string to type real
----------------
----------------

Maybe, this regression is caused by pull requests #828 and #833 https://github.com/D-Programming-Language/phobos/pull/828 https://github.com/D-Programming-Language/phobos/pull/833

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 11, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=8908


Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull, rejects-valid


--- Comment #1 from Kenji Hara <k.hara.pg@gmail.com> 2012-12-10 18:39:57 PST ---
https://github.com/D-Programming-Language/phobos/pull/1001

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 18, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=8908



--- Comment #2 from github-bugzilla@puremagic.com 2012-12-18 15:03:03 PST ---
Commits pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/b66012ef4ef73a913f15d6ff066980a090410324
fix Issue 8908 - Collapse of std.csv by the specifications change of
std.conv.parse

https://github.com/D-Programming-Language/phobos/commit/d9170a8c74c177fe6371718ac342a23718d08aaf Merge pull request #1001 from 9rnsr/fix8908

Issue 8908 - Collapse of std.csv by the specifications change of std.conv.parse

-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
December 19, 2012
http://d.puremagic.com/issues/show_bug.cgi?id=8908


Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED


-- 
Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email
------- You are receiving this mail because: -------