Move the decide parameter out of the way in extremum

Author: jgonggrijp

modules/_extremum.js

@@ -6,11 +6,8 @@ import find from './find.js';
 // `true` if its first argument is more extreme than (i.e., should be preferred
 // over) its second argument, `false` otherwise. `iteratee` and `context`, like
 // in other collection functions, let you map the actual values in `collection`
-// to the values to `compare`. `decide` is an optional customization point
-// which is only present for historical reasons; please don't use it, as it will
-// likely be removed in the future.
+// to the values to `compare`.
 export default function extremum(collection, compare, iteratee, context, decide) {
-  decide || (decide = identity);
   // `extremum` is essentially a combined map+reduce with **two** accumulators:
   // `result` and `iterResult`, respectively the unmapped and the mapped version
   // corresponding to the same element.
@@ -25,9 +22,12 @@ export default function extremum(collection, compare, iteratee, context, decide)
       first = false;
     }
   });
-  // `extremum` normally returns an unmapped element from `collection`. However,
-  // `_.min` and `_.max` forcibly return a number even if there is no element
-  // that maps to a numeric value. Passing both accumulators through `decide`
-  // before returning enables this behavior.
-  return decide(result, iterResult);
+  // `extremum` would normally return `result`. However, `_.min` and `_.max`
+  // forcibly return a number even if there is no element that maps to a numeric
+  // value. Passing both accumulators through `decide` before returning enables
+  // this behavior.
+  // `decide` is an optional customization point which is only present for the
+  // above historical reason; please don't use it, as it will likely be removed
+  // in the future.
+  return (decide || identity)(result, iterResult);
 }