aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib/active_record/attribute_methods
diff options
context:
space:
mode:
authorJon Leighton <j@jonathanleighton.com>2011-12-01 22:30:15 +0000
committerJon Leighton <j@jonathanleighton.com>2011-12-01 23:41:51 +0000
commit9ead4130def8bccfdd2c51ddd910ac6a2a8b21ee (patch)
tree63f9a9c533958ff4cea2e4fe286d9f02a70a728c /activerecord/lib/active_record/attribute_methods
parentbd920eae82f83268b2a6ed31c0275255e01bad9f (diff)
downloadrails-9ead4130def8bccfdd2c51ddd910ac6a2a8b21ee.tar.gz
rails-9ead4130def8bccfdd2c51ddd910ac6a2a8b21ee.tar.bz2
rails-9ead4130def8bccfdd2c51ddd910ac6a2a8b21ee.zip
Create method with known identifier then alias into place.
This means we never have to rely on define_method (which is slower and uses more memory), even when we have attributes containing characters that are not allowed in standard method names. (I am mainly changing this because the duplication annoys me, though.)
Diffstat (limited to 'activerecord/lib/active_record/attribute_methods')
-rw-r--r--activerecord/lib/active_record/attribute_methods/read.rb46
1 files changed, 21 insertions, 25 deletions
diff --git a/activerecord/lib/active_record/attribute_methods/read.rb b/activerecord/lib/active_record/attribute_methods/read.rb
index 245abeda65..ab2b7936f3 100644
--- a/activerecord/lib/active_record/attribute_methods/read.rb
+++ b/activerecord/lib/active_record/attribute_methods/read.rb
@@ -42,38 +42,34 @@ module ActiveRecord
end
protected
- # Where possible, generate the method by evalling a string, as this will result in
- # faster accesses because it avoids the block eval and then string eval incurred
- # by the second branch.
+ # We want to generate the methods via module_eval rather than define_method,
+ # because define_method is slower on dispatch and uses more memory (because it
+ # creates a closure).
#
- # The second, slower, branch is necessary to support instances where the database
- # returns columns with extra stuff in (like 'my_column(omg)').
+ # But sometimes the database might return columns with characters that are not
+ # allowed in normal method names (like 'my_column(omg)'. So to work around this
+ # we first define with the __temp__ identifier, and then use alias method to
+ # rename it to what we want.
def define_method_attribute(attr_name)
cast_code = attribute_cast_code(attr_name)
internal = internal_attribute_access_code(attr_name, cast_code)
external = external_attribute_access_code(attr_name, cast_code)
- if attr_name =~ ActiveModel::AttributeMethods::NAME_COMPILABLE_REGEXP
- generated_attribute_methods.module_eval <<-STR, __FILE__, __LINE__
- def #{attr_name}
- #{internal}
- end
-
- def self.attribute_#{attr_name}(v, attributes, attributes_cache, attr_name)
- #{external}
- end
- STR
- else
- generated_attribute_methods.module_eval do
- define_method(attr_name) do
- eval(internal)
- end
-
- singleton_class.send(:define_method, "attribute_#{attr_name}") do |v, attributes, attributes_cache, attr_name|
- eval(external)
- end
+ generated_attribute_methods.module_eval <<-STR, __FILE__, __LINE__
+ def __temp__
+ #{internal}
end
- end
+ alias_method '#{attr_name}', :__temp__
+ undef_method :__temp__
+ STR
+
+ generated_attribute_methods.singleton_class.module_eval <<-STR, __FILE__, __LINE__
+ def __temp__(v, attributes, attributes_cache, attr_name)
+ #{external}
+ end
+ alias_method 'attribute_#{attr_name}', :__temp__
+ undef_method :__temp__
+ STR
end
private