Merge pull request #1654 from ericmj/fix-enum-find

Use ifnone in Enum.find for all collections

Author: José Valim

lib/elixir/lib/enum.ex

@@ -449,8 +449,8 @@ defmodule Enum do
   end
 
   def find(collection, ifnone, fun) do
-    Enumerable.reduce(collection, ifnone, fn(entry, _) ->
-      if fun.(entry), do: throw({ :enum_find, entry })
+    Enumerable.reduce(collection, ifnone, fn(entry, ifnone) ->
+      if fun.(entry), do: throw({ :enum_find, entry }), else: ifnone
     end)
   catch
     { :enum_find, entry } -> entry

lib/elixir/test/elixir/enum_test.exs

@@ -408,6 +408,8 @@ defmodule EnumTest.Range do
     range = Range.new(first: 2, last: 6)
     assert Enum.find(range, fn(x) -> rem(x, 2) == 0 end) == 2
     assert Enum.find(range, fn(x) -> rem(x, 2) == 1 end) == 3
+    assert Enum.find(range, fn _ -> false end) == nil
+    assert Enum.find(range, 0, fn _ -> false end) == 0
   end
 
   test :find_value do