Thread overview
[Issue 17525] std.algorithm.searching.skipOver should have a single argument with pred version
Jul 05, 2017
RazvanN
Jul 05, 2017
RazvanN
July 05, 2017
https://issues.dlang.org/show_bug.cgi?id=17525

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |razvan.nitu1305@gmail.com

--- Comment #1 from RazvanN <razvan.nitu1305@gmail.com> ---
skipOver returns a bool value and so do startsWith and endsWith, so you can't equal with "value". But I think that adding an overload to support the construction:

auto s = "\tvalue";
assert(s.skipOver!isWhite && s.equals("value"));

is useful and offers consistency.

--
July 05, 2017
https://issues.dlang.org/show_bug.cgi?id=17525

--- Comment #2 from RazvanN <razvan.nitu1305@gmail.com> ---
PR : https://github.com/dlang/phobos/pull/5543

--
July 07, 2017
https://issues.dlang.org/show_bug.cgi?id=17525

--- Comment #3 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/1d1b42f413c7b929dcd5c3081dd181638a912fd6 Fix Issue 17525 - std.algorithm.skipOver should have a single argument with pred version

https://github.com/dlang/phobos/commit/37d15a89c52967006585cb088dbe008e901aa123 Merge pull request #5543 from RazvanN7/Issue_17525

Fix Issue 17525 - std.algorithm.searching.skipOver should have a single
argument with pred version
merged-on-behalf-of: Sebastian Wilzbach <sebi.wilzbach@gmail.com>

--
July 07, 2017
https://issues.dlang.org/show_bug.cgi?id=17525

github-bugzilla@puremagic.com changed:

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

--
August 16, 2017
https://issues.dlang.org/show_bug.cgi?id=17525

--- Comment #4 from github-bugzilla@puremagic.com ---
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/1d1b42f413c7b929dcd5c3081dd181638a912fd6 Fix Issue 17525 - std.algorithm.skipOver should have a single argument with pred version

https://github.com/dlang/phobos/commit/37d15a89c52967006585cb088dbe008e901aa123 Merge pull request #5543 from RazvanN7/Issue_17525

--
January 05, 2018
https://issues.dlang.org/show_bug.cgi?id=17525

--- Comment #5 from github-bugzilla@puremagic.com ---
Commits pushed to dmd-cxx at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/1d1b42f413c7b929dcd5c3081dd181638a912fd6 Fix Issue 17525 - std.algorithm.skipOver should have a single argument with pred version

https://github.com/dlang/phobos/commit/37d15a89c52967006585cb088dbe008e901aa123 Merge pull request #5543 from RazvanN7/Issue_17525

--