Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

remove "instance variable not initialized" warnings #126

Closed
wants to merge 1 commit into from

2 participants

@shime

I was receiving a lot of warnings about this when running the tests: /lib/arel/nodes/window.rb:20: warning: instance variable @frame not initialized

This removes them.

@jonleighton
Collaborator

@frame is actually never defined. I resolved this warning with b543204. thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 11, 2012
  1. @shime
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/arel/nodes/window.rb
View
4 lib/arel/nodes/window.rb
@@ -17,7 +17,7 @@ def order *expr
end
def frame(expr)
- raise ArgumentError, "Window frame cannot be set more than once" if @frame
+ raise ArgumentError, "Window frame cannot be set more than once" if defined? @frame
@framing = expr
end
@@ -75,4 +75,4 @@ def initialize(expr = nil)
end
end
end
-end
+end
Something went wrong with that request. Please try again.