Use Ordering.fromInt() instead of a ternary operator in

dobesv · dobesv · commit f7334d32fb4a · 2015-01-03T22:09:59.000-08:00
Ord.comparableOrd().
diff --git a/core/src/main/java/fj/Ord.java b/core/src/main/java/fj/Ord.java
@@ -630,8 +630,7 @@ public static <A extends Comparable<A>> Ord<A> comparableOrd() {
       public F<A, Ordering> f(final A a1) {
         return new F<A, Ordering>() {
           public Ordering f(final A a2) {
-            final int x = a1.compareTo(a2);
-            return x < 0 ? Ordering.LT : x == 0 ? Ordering.EQ : Ordering.GT;
+            return Ordering.fromInt(a1.compareTo(a2));
           }
         };
       }

Original file line number	Diff line number	Diff line change
`@@ -630,8 +630,7 @@ public static <A extends Comparable<A>> Ord<A> comparableOrd() {`
`630`	`630`	`public F<A, Ordering> f(final A a1) {`
`631`	`631`	`return new F<A, Ordering>() {`
`632`	`632`	`public Ordering f(final A a2) {`
`633`		`- final int x = a1.compareTo(a2);`
`634`		`- return x < 0 ? Ordering.LT : x == 0 ? Ordering.EQ : Ordering.GT;`
	`633`	`+ return Ordering.fromInt(a1.compareTo(a2));`
`635`	`634`	`}`
`636`	`635`	`};`
`637`	`636`	`}`