Thread overview | |||||||||
---|---|---|---|---|---|---|---|---|---|
|
October 10, 2012 [Issue 8792] New: std.algorithm.joiner doesn't return a proper forward range` | ||||
---|---|---|---|---|
| ||||
http://d.puremagic.com/issues/show_bug.cgi?id=8792 Summary: std.algorithm.joiner doesn't return a proper forward range` Product: D Version: D2 Platform: All OS/Version: Linux Status: NEW Severity: major Priority: P2 Component: Phobos AssignedTo: nobody@puremagic.com ReportedBy: hsteoh@quickfur.ath.cx --- Comment #0 from hsteoh@quickfur.ath.cx 2012-10-09 20:25:31 PDT --- import std.algorithm; import std.range; import std.stdio; void main() { auto x = [[1],[2],[3]]; auto yy = x.joiner; assert(isForwardRange!(typeof(yy))); writeln(yy.save); writeln(yy); } The output is: [] [1, 2, 3] Which means that yy.save didn't save at all! This is a pretty major bug since it makes joiner unusable where the result needs to be a forward range. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
October 10, 2012 [Issue 8792] std.algorithm.joiner doesn't return a proper forward range | ||||
---|---|---|---|---|
| ||||
Posted in reply to hsteoh@quickfur.ath.cx | http://d.puremagic.com/issues/show_bug.cgi?id=8792 --- Comment #1 from hsteoh@quickfur.ath.cx 2012-10-09 22:15:59 PDT --- Found the bug: joiner.Result.save didn't copy 1 field over, so the saved range is defective. https://github.com/D-Programming-Language/phobos/pull/853 -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
October 13, 2012 [Issue 8792] std.algorithm.joiner doesn't return a proper forward range | ||||
---|---|---|---|---|
| ||||
Posted in reply to hsteoh@quickfur.ath.cx | http://d.puremagic.com/issues/show_bug.cgi?id=8792 Jonathan M Davis <jmdavisProg@gmx.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED CC| |jmdavisProg@gmx.com Resolution| |FIXED -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
October 14, 2012 [Issue 8792] std.algorithm.joiner doesn't return a proper forward range | ||||
---|---|---|---|---|
| ||||
Posted in reply to hsteoh@quickfur.ath.cx | http://d.puremagic.com/issues/show_bug.cgi?id=8792 --- Comment #3 from github-bugzilla@puremagic.com 2012-10-14 08:28:15 PDT --- Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/5a0c67f08553c05c532fe3c433e3bb944d8aa6dd Expanded test for issue# 8792. https://github.com/D-Programming-Language/phobos/commit/5037dd00bd66ee195da3b95dd7a5baa99f451106 Merge pull request #866 from jmdavis/tests Expanded test for issue# 8792. -- Configure issuemail: http://d.puremagic.com/issues/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- |
Copyright © 1999-2021 by the D Language Foundation