Remove warning of intance variable not initialized #8592

Merged
merged 1 commit into from Dec 25, 2012

Conversation

Projects
None yet
4 participants
Owner

rafaelfranca commented Dec 23, 2012

I'm not sure if this is the best fix.

@tenderlove chould you review this one?

activesupport/lib/active_support/core_ext/thread.rb:68: warning:
instance variable @locals not initialized
Member

steveklabnik commented Dec 25, 2012

👍

Owner

tenderlove commented Dec 25, 2012

Ya. @rafaelfranca I think master depends on the thread safe gem, so we could push this in to that gem and remove from Rails. (We don't have to, just an option)

Owner

tenderlove commented Dec 25, 2012

Forgot to say 👍 to merge.

@guilleiguaran guilleiguaran added a commit that referenced this pull request Dec 25, 2012

@guilleiguaran guilleiguaran Merge pull request #8592 from rafaelfranca/warning_fix
Remove warning of intance variable not initialized
501175e

@guilleiguaran guilleiguaran merged commit 501175e into rails:master Dec 25, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment