← Back to index | Original Bugzilla link

Bug 15735 – std.algorithm.iteration.splitter returns empty range

Status
RESOLVED
Resolution
FIXED
Severity
major
Priority
P1
Component
phobos
Product
D
Version
D2
Platform
x86_64
OS
Linux
Creation time
2016-02-28T23:36:43Z
Last change time
2017-10-16T09:58:15Z
Assigned to
No Owner
Creator
Dragos Carp

Comments

Comment #0 by dragoscarp — 2016-02-28T23:36:43Z
The docs say "If the empty range is given, the result is a range with one empty element." --- unittest { import std.algorithm : equal, splitter; assert("".splitter('_').equal([""])); // asserts: returned range is empty }
Comment #1 by dragoscarp — 2016-02-29T00:31:36Z
Fixed with https://github.com/D-Programming-Language/phobos/pull/4030
Comment #2 by github-bugzilla — 2016-04-26T01:21:21Z
Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/b438bf5a0653b616ef6debe2a0dfe247d8fc5928 Fix issue 15735 https://github.com/dlang/phobos/commit/edbb7a1537e0aa0ea7904dcdd64723b92b5b6a7b Merge pull request #4030 from dcarp/issue15735 Fix issue 15735
Comment #3 by github-bugzilla — 2016-10-03T22:11:21Z
Commit pushed to revert-4030-issue15735 at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/4fa5ff684e946ef82927b6ff3937c27d23470c9a Revert "Fix issue 15735"
Comment #4 by andrei — 2016-10-04T13:25:03Z
I'm reopening this pending a change in the documentation.
Comment #5 by jrdemail2000-dlang — 2017-10-08T21:49:30Z
(In reply to Andrei Alexandrescu from comment #4) > I'm reopening this pending a change in the documentation. PR for the documentation change: https://github.com/dlang/phobos/pull/5767
Comment #6 by github-bugzilla — 2017-10-08T23:34:28Z
Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/c6ef7c3f5e5caafe5469af671a761e42c04ef7f9 Fix issue 15735: Correct splitter documentation when argument is an empty range. https://github.com/dlang/phobos/commit/181efe61db4d4cb33fee047c955b8bc00c2090f7 Merge pull request #5767 from jondegenhardt/issue-15735-splitter-emptyrange-doc Fix issue 15735: Correct splitter documentation when arg is an empty range. merged-on-behalf-of: Petar Kirov <[email protected]>
Comment #7 by github-bugzilla — 2017-10-16T09:58:15Z
Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/c6ef7c3f5e5caafe5469af671a761e42c04ef7f9 Fix issue 15735: Correct splitter documentation when argument is an empty range. https://github.com/dlang/phobos/commit/181efe61db4d4cb33fee047c955b8bc00c2090f7 Merge pull request #5767 from jondegenhardt/issue-15735-splitter-emptyrange-doc