Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP
Browse files

Stop SafeBuffer#clone_empty from issuing warnings

Logic in clone_empty method was dealing with old @dirty variable, which
has changed by @html_safe in this commit:
rails@139963c

This was issuing a "not initialized variable" warning - related to:
rails#5237

The logic applied by this method is already handled by the [] override,
so there is no need to reset the variable here.
  • Loading branch information...
commit 5fd360133a569c033dc2904fe3f4eed615586106 1 parent 1a71e84
@carlosantoniodasilva carlosantoniodasilva authored drogus committed
View
4 activesupport/lib/active_support/core_ext/string/output_safety.rb
@@ -126,9 +126,7 @@ def initialize_copy(other)
end
def clone_empty
- new_safe_buffer = self[0, 0]
- new_safe_buffer.instance_variable_set(:@dirty, @dirty)
- new_safe_buffer
+ self[0, 0]
end
def concat(value)
Please sign in to comment.
Something went wrong with that request. Please try again.