Remove package wide std.algorithm imports from Phobos

Author: JackStouffer

std/algorithm/comparison.d

@@ -789,7 +789,7 @@ template equal(alias pred = "a == b")
 @safe unittest
 {
     import std.math : approxEqual;
-    import std.algorithm : equal;
+    import std.algorithm.comparison : equal;
 
     int[] a = [ 1, 2, 4, 3 ];
     assert(!equal(a, a[1..$]));
@@ -815,7 +815,7 @@ range of range (of range...) comparisons.
 @safe unittest
 {
     import std.range : iota, chunks;
-    import std.algorithm : equal;
+    import std.algorithm.comparison : equal;
     assert(equal!(equal!equal)(
         [[[0, 1], [2, 3]], [[4, 5], [6, 7]]],
         iota(0, 8).chunks(2).chunks(2)

std/algorithm/iteration.d

@@ -3328,7 +3328,8 @@ The number of seeds must be correspondingly increased.
 */
 @safe unittest
 {
-    import std.algorithm : map, max, min;
+    import std.algorithm.comparison : max, min;
+    import std.algorithm.iteration : map;
     import std.math : approxEqual;
     import std.typecons : tuple;
 
@@ -3347,7 +3348,7 @@ The number of seeds must be correspondingly increased.
 
 @safe unittest
 {
-    import std.algorithm : equal, map, max, min;
+    import std.algorithm.comparison : equal, max, min;
     import std.conv : to;
     import std.range : chain;
     import std.typecons : tuple;
@@ -3406,7 +3407,6 @@ The number of seeds must be correspondingly increased.
 
 @safe unittest
 {
-    import std.algorithm : map;
     import std.math : approxEqual;
     import std.typecons : tuple;
 

std/algorithm/setops.d

@@ -339,7 +339,6 @@ auto cartesianProduct(R1, R2)(R1 range1, R2 range2)
 // Issue 13091
 pure nothrow @safe @nogc unittest
 {
-    import std.algorithm : cartesianProduct;
     int[1] a = [1];
     foreach (t; cartesianProduct(a[], a[])) {}
 }

std/base64.d

@@ -1737,7 +1737,8 @@ class Base64Exception : Exception
 
 @system unittest
 {
-    import std.algorithm : sort, equal;
+    import std.algorithm.sorting : sort;
+    import std.algorithm.comparison : equal;
     import std.conv;
     import std.file;
     import std.stdio;

std/container/package.d

@@ -148,7 +148,8 @@ as the one being worked with. It is important to note that many generic range
 algorithms return the same range type as their input range.
 
 ---
-import std.algorithm : equal, find;
+import std.algorithm.comparison : equal;
+import std.algorithm.iteration : find;
 import std.container;
 import std.range : take;
 
@@ -173,7 +174,7 @@ a member function, the documention usually refers to the parameter's templated
 type as $(D Stuff).
 
 ---
-import std.algorithm : equal;
+import std.algorithm.comparison : equal;
 import std.container;
 import std.range : iota;
 

std/csv.d

@@ -755,7 +755,7 @@ auto csvReader(Contents = string,
 
 @safe unittest // const/immutable dchars
 {
-    import std.algorithm : map;
+    import std.algorithm.iteration : map;
     import std.array : array;
     const(dchar)[] c = "foo,bar\n";
     assert(csvReader(c).map!array.array == [["foo", "bar"]]);
@@ -907,17 +907,18 @@ public:
         {
             static if (is(Contents T : T[U], U : string))
             {
-                import std.algorithm : sort;
+                import std.algorithm.sorting : sort;
                 sort(indices);
             }
             else static if (ErrorLevel == Malformed.ignore)
             {
-                import std.algorithm : sort;
+                import std.algorithm.sorting : sort;
                 sort(indices);
             }
             else
             {
-                import std.algorithm : isSorted, findAdjacent;
+                import std.algorithm.searching : findAdjacent;
+                import std.algorithm.sorting : isSorted;
                 if (!isSorted(indices))
                 {
                     auto ex = new HeaderMismatchException

std/digest/digest.d

@@ -256,7 +256,7 @@ version(ExampleDigest)
 unittest
 {
     //Using the OutputRange feature
-    import std.algorithm : copy;
+    import std.algorithm.mutation : copy;
     import std.range : repeat;
     import std.digest.md;
 
@@ -427,7 +427,7 @@ package template isDigestibleRange(Range)
 DigestType!Hash digest(Hash, Range)(auto ref Range range) if (!isArray!Range
     && isDigestibleRange!Range)
 {
-    import std.algorithm : copy;
+    import std.algorithm.mutation : copy;
     Hash hash;
     hash.start();
     copy(range, &hash);
@@ -621,7 +621,7 @@ interface Digest
 unittest
 {
     //Using the OutputRange feature
-    import std.algorithm : copy;
+    import std.algorithm.mutation : copy;
     import std.range : repeat;
     import std.digest.md;
 

std/digest/hmac.d

@@ -270,7 +270,7 @@ if (isDigest!H)
     {
         import std.digest.sha, std.digest.hmac;
         import std.string : representation;
-        import std.algorithm : map;
+        import std.algorithm.iteration : map;
         string data = "Hello, world";
         auto digest = data.representation
                       .map!(a => cast(ubyte)(a+1))

std/exception.d

@@ -1964,7 +1964,8 @@ auto handle(E : Throwable, RangePrimitive primitivesToHandle, alias handler, Ran
 ///
 pure @safe unittest
 {
-    import std.algorithm : equal, map, splitter;
+    import std.algorithm.comparison : equal;
+    import std.algorithm.iteration : map, splitter;
     import std.conv : to, ConvException;
 
     auto s = "12,1337z32,54,2,7,9,1z,6,8";
@@ -1981,7 +1982,7 @@ pure @safe unittest
 ///
 pure @safe unittest
 {
-    import std.algorithm : equal;
+    import std.algorithm.comparison : equal;
     import std.range : retro;
     import std.utf : UTFException;
 

std/experimental/logger/multilogger.d

@@ -69,7 +69,7 @@ class MultiLogger : Logger
     */
     Logger removeLogger(in char[] toRemove) @safe
     {
-        import std.algorithm : copy;
+        import std.algorithm.mutation : copy;
         import std.range.primitives : back, popBack;
         for (size_t i = 0; i < this.logger.length; ++i)
         {