Switch accessor creation from define_method to class_eval

Author: film42

lib/protobuf/field/base_field.rb

@@ -172,16 +172,19 @@ def wire_type
       #
 
       def define_accessor(simple_field_name, fully_qualified_field_name)
-        message_class.class_eval do
-          define_method("#{simple_field_name}!") do
-            @values[fully_qualified_field_name]
+        message_class.class_eval <<-ruby, __FILE__, __LINE__
+          def #{simple_field_name}!
+            @values[:#{fully_qualified_field_name}]
           end
-        end
 
-        message_class.class_eval do
-          define_method(simple_field_name) { self[fully_qualified_field_name] }
-          define_method("#{simple_field_name}=") { |v| self[fully_qualified_field_name] = v }
-        end
+          def #{simple_field_name}
+            self[:#{fully_qualified_field_name}]
+          end
+
+          def #{simple_field_name}=(value)
+            self[:#{fully_qualified_field_name}] = value
+          end
+        ruby
 
         return unless deprecated?
 

protobuf.gemspec

@@ -31,6 +31,7 @@ require "protobuf/version"
   s.add_development_dependency 'simplecov'
   s.add_development_dependency 'timecop'
   s.add_development_dependency 'yard'
+  s.add_development_dependency 'benchmark-ips'
 
   # debuggers only work in MRI
   if RUBY_ENGINE.to_sym == :ruby