Fix compile warnings (#186)

This was breaking the build because we were unintentionally duplicating these function clauses (see the `for {endian, endian_atom, modifier} . . .` stuff).
felt · Sep 19, 2023 · d56d8a4 · d56d8a4
1 parent 726667a
commit d56d8a4
Showing 1 changed file with 8 additions and 8 deletions.
diff --git a/lib/geo/wkb/encoder.ex b/lib/geo/wkb/encoder.ex
@@ -41,6 +41,14 @@ defmodule Geo.WKB.Encoder do
 
   def encode!(geom, endian \\ :ndr)
 
+  def do_encode(%Point{coordinates: nil}, :ndr) do
+    {@point, [<<00, 00, 00, 00, 00, 00, 248, 127>>, <<00, 00, 00, 00, 00, 00, 248, 127>>]}
+  end
+
+  def do_encode(%Point{coordinates: nil}, :xdr) do
+    {@point, [<<127, 248, 00, 00, 00, 00, 00, 00>>, <<127, 248, 00, 00, 00, 00, 00, 00>>]}
+  end
+
   for {endian, endian_atom, modifier} <- [{1, :ndr, quote(do: little)}, {0, :xdr, quote(do: big)}] do
     def encode!(geom, unquote(endian_atom)) do
       {type, rest} = do_encode(geom, unquote(endian_atom))
@@ -55,14 +63,6 @@ defmodule Geo.WKB.Encoder do
       [unquote(endian), binary, rest]
     end
 
-    def do_encode(%Point{coordinates: nil}, :ndr) do
-      {@point, [<<00, 00, 00, 00, 00, 00, 248, 127>>, <<00, 00, 00, 00, 00, 00, 248, 127>>]}
-    end
-
-    def do_encode(%Point{coordinates: nil}, :xdr) do
-      {@point, [<<127, 248, 00, 00, 00, 00, 00, 00>>, <<127, 248, 00, 00, 00, 00, 00, 00>>]}
-    end
-
     def do_encode(%Point{coordinates: {x, y}}, unquote(endian_atom)) do
       {@point, [<<x::unquote(modifier)-float-64>>, <<y::unquote(modifier)-float-64>>]}
     end