replace Module.eval_quoted/4 with Code.eval_quoted/3

Author: luisguilhermemsalmeida

lib/exprotobuf/builder.ex

@@ -1,18 +1,19 @@
 defmodule Protobuf.Builder do
   @moduledoc false
 
-  alias Protobuf.Config
-
   import Protobuf.DefineEnum, only: [def_enum: 3]
   import Protobuf.DefineMessage, only: [def_message: 3]
 
+  alias Protobuf.Config
+
   def define(msgs, %Config{inject: inject} = config) do
     # When injecting, use_in is not available, so we don't need to use @before_compile
     if inject do
       quote location: :keep do
-        Module.register_attribute(__MODULE__, :use_in, accumulate: true)
         import unquote(__MODULE__), only: [use_in: 2]
 
+        Module.register_attribute(__MODULE__, :use_in, accumulate: true)
+
         unless is_nil(unquote(config.from_file)) do
           case unquote(config.from_file) do
             file when is_binary(file) ->
@@ -28,13 +29,14 @@ defmodule Protobuf.Builder do
         @config unquote(Macro.escape(Map.to_list(%{config | :schema => nil})))
         @msgs unquote(Macro.escape(msgs))
         contents = unquote(__MODULE__).generate(@msgs, @config)
-        Module.eval_quoted(__MODULE__, contents, [], __ENV__)
+        Code.eval_quoted(contents, [], __ENV__)
       end
     else
       quote do
-        Module.register_attribute(__MODULE__, :use_in, accumulate: true)
         import unquote(__MODULE__), only: [use_in: 2]
 
+        Module.register_attribute(__MODULE__, :use_in, accumulate: true)
+
         unless is_nil(unquote(config.from_file)) do
           case unquote(config.from_file) do
             file when is_binary(file) ->
@@ -57,7 +59,7 @@ defmodule Protobuf.Builder do
   defmacro __before_compile__(_env) do
     quote location: :keep do
       contents = unquote(__MODULE__).generate(@msgs, @config)
-      Module.eval_quoted(__MODULE__, contents, [], __ENV__)
+      Code.eval_quoted(contents, [], __ENV__)
     end
   end
 
@@ -83,12 +85,13 @@ defmodule Protobuf.Builder do
           item_type in [:msg, :proto3_msg, :enum],
           into: [] do
         if only != [] do
-          is_child? = 
-            Enum.any?(only, fn o -> 
+          is_child? =
+            Enum.any?(only, fn o ->
               o != item_name and is_child_type?(item_name, o)
             end)
-          
+
           last_mod = last_module(item_name)
+
           if last_mod in only or is_child? do
             case item_type do
               :msg when is_child? ->

lib/exprotobuf/decoder.ex

@@ -1,6 +1,6 @@
 defmodule Protobuf.Decoder do
   @moduledoc false
-  use Bitwise, only_operators: true
+  import Bitwise
 
   alias Protobuf.Field
   alias Protobuf.OneOfField

lib/exprotobuf/define_message.ex

@@ -2,10 +2,10 @@ defmodule Protobuf.DefineMessage do
   @moduledoc false
 
   alias Protobuf.Decoder
+  alias Protobuf.Delimited
   alias Protobuf.Encoder
   alias Protobuf.Field
   alias Protobuf.OneOfField
-  alias Protobuf.Delimited
   alias Protobuf.Utils
 
   def def_message(name, fields, inject: inject, doc: doc, syntax: syntax) when is_list(fields) do
@@ -61,7 +61,7 @@ defmodule Protobuf.DefineMessage do
           unquote(constructors(name))
 
           if use_in != nil do
-            Module.eval_quoted(__MODULE__, use_in, [], __ENV__)
+            Code.eval_quoted(use_in, [], __ENV__)
           end
 
           defimpl Protobuf.Serializable do
@@ -76,7 +76,7 @@ defmodule Protobuf.DefineMessage do
 
   defp constructors(name) do
     quote location: :keep do
-      def new(), do: new([])
+      def new, do: new([])
 
       def new(values) do
         struct(unquote(name), values)