Merge pull request typelevel#357 from joroKr21/cogen-map

Drop Ordering bound on the values in cogenMap
ssanj · Oct 4, 2017 · bc672d9 · bc672d9
2 parents 4cf7f8f + 102654b
commit bc672d9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/.gitignore b/.gitignore
@@ -18,3 +18,6 @@ target
 
 /examples/simple-sbt/project/
 /examples/simple-sbt/target/
+
+# Intellij
+.idea
diff --git a/src/main/scala/org/scalacheck/Cogen.scala b/src/main/scala/org/scalacheck/Cogen.scala
@@ -126,8 +126,8 @@ object Cogen extends CogenArities with CogenLowPriority {
   implicit def cogenSet[A: Cogen: Ordering]: Cogen[Set[A]] =
     Cogen.it(_.toVector.sorted.iterator)
 
-  implicit def cogenMap[K: Cogen: Ordering, V: Cogen: Ordering]: Cogen[Map[K, V]] =
-    Cogen.it(_.toVector.sorted.iterator)
+  implicit def cogenMap[K: Cogen: Ordering, V: Cogen]: Cogen[Map[K, V]] =
+    Cogen.it(_.toVector.sortBy(_._1).iterator)
 
   implicit def cogenFunction0[Z: Cogen]: Cogen[() => Z] =
     Cogen[Z].contramap(f => f())