chore: call module.__info__(:struct) just once

Author: Geekfish

lib/mimic/module.ex

@@ -126,47 +126,49 @@ defmodule Mimic.Module do
     module
   end
 
-  if @elixir_version >= 1.18 do
-    defp generate_mimic_struct(module) do
-      if function_exported?(module, :__info__, 1) && module.__info__(:struct) != nil do
-        struct_info = module.__info__(:struct) || []
+  defp generate_mimic_struct(module) do
+    if function_exported?(module, :__info__, 1) do
+      do_generate_mimic_struct(module, module.__info__(:struct))
+    end
+  end
 
-        struct_template = Map.from_struct(module.__struct__())
+  defp do_generate_mimic_struct(_module, nil), do: nil
 
-        struct_params =
-          for %{field: field} <- struct_info,
-              do: {field, Macro.escape(struct_template[field])}
+  if @elixir_version >= 1.18 do
+    defp do_generate_mimic_struct(module, struct_info) do
+      struct_template = Map.from_struct(module.__struct__())
 
-        quote do
-          defstruct unquote(struct_params)
-        end
+      struct_params =
+        for %{field: field} <- struct_info,
+            do: {field, Macro.escape(struct_template[field])}
+
+      quote do
+        defstruct unquote(struct_params)
       end
     end
   else
-    defp generate_mimic_struct(module) do
-      if function_exported?(module, :__info__, 1) && module.__info__(:struct) != nil do
-        struct_info =
-          module.__info__(:struct)
-          |> Enum.split_with(& &1.required)
-          |> Tuple.to_list()
-          |> List.flatten()
-
-        required_fields = for %{field: field, required: true} <- struct_info, do: field
-        struct_template = Map.from_struct(module.__struct__())
-
-        struct_params =
-          for %{field: field, required: required} <- struct_info do
-            if required do
-              field
-            else
-              {field, Macro.escape(struct_template[field])}
-            end
+    defp do_generate_mimic_struct(module, struct_info) do
+      struct_info =
+        module.__info__(:struct)
+        |> Enum.split_with(& &1.required)
+        |> Tuple.to_list()
+        |> List.flatten()
+
+      required_fields = for %{field: field, required: true} <- struct_info, do: field
+      struct_template = Map.from_struct(module.__struct__())
+
+      struct_params =
+        for %{field: field, required: required} <- struct_info do
+          if required do
+            field
+          else
+            {field, Macro.escape(struct_template[field])}
           end
-
-        quote do
-          @enforce_keys unquote(required_fields)
-          defstruct unquote(struct_params)
         end
+
+      quote do
+        @enforce_keys unquote(required_fields)
+        defstruct unquote(struct_params)
       end
     end
   end