Bug 15750 – net/isemail uses lots of redundant helper methods

In my cleanup task I saw that std.net.isemail seems to do some redundant effort and defines those methods as its own helper: - substr - max - compareFirstN - grep - get There is `backPop`! All of these methods (at least max, substr) feel general purpose enough that they should existent in a general fashion. Btw for max: doesn't work get with ranges, but there is a pending pull request. https://github.com/D-Programming-Language/phobos/pull/4019

Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/5c31dd26ed289b397a4bad32ac29af6e77247ef7 Issue 15750 - remove substr from std.net.isemail https://github.com/dlang/phobos/commit/e9ff980095462ee88713bcea3cc9852de5081e5b Merge pull request #5585 from wilzbach/remove-substr Issue 15750 - remove substr from std.net.isemail merged-on-behalf-of: Jack Stouffer <[email protected]>

Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/5c31dd26ed289b397a4bad32ac29af6e77247ef7 Issue 15750 - remove substr from std.net.isemail https://github.com/dlang/phobos/commit/e9ff980095462ee88713bcea3cc9852de5081e5b Merge pull request #5585 from wilzbach/remove-substr

Commits pushed to dmd-cxx at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/5c31dd26ed289b397a4bad32ac29af6e77247ef7 Issue 15750 - remove substr from std.net.isemail https://github.com/dlang/phobos/commit/e9ff980095462ee88713bcea3cc9852de5081e5b Merge pull request #5585 from wilzbach/remove-substr

THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/10163 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB