Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Rename Keyword to RPkeyword #3

Closed
wants to merge 1 commit into from

2 participants

@ghost

We renamed the Keyword class to RPkeyword, cause we already using a Keyword class in our app.

I guess we can make a better solution by surrounding most classes by a module. Please tell me if you prefer such solution.

Sebastien Savater Renamed Keyword class 6e8f053
@thoughtless

+1 Keyword is a pretty common class name.

@ghost

It seems this has been fixed with the new release (f06294c). Thanks for all.

@zenspider zenspider closed this
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Apr 22, 2011
  1. Renamed Keyword class

    Sebastien Savater authored
This page is out of date. Refresh to see the latest.
Showing with 2 additions and 2 deletions.
  1. +1 −1  lib/ruby_lexer.rb
  2. +1 −1  lib/ruby_parser_extras.rb
View
2  lib/ruby_lexer.rb
@@ -1244,7 +1244,7 @@ def process_token(command_state)
unless lex_state == :expr_dot then
# See if it is a reserved word.
- keyword = Keyword.keyword token
+ keyword = RPKeyword.keyword token
if keyword then
state = lex_state
View
2  lib/ruby_parser_extras.rb
@@ -815,7 +815,7 @@ def yyerror msg
end
end
-class Keyword
+class RPKeyword
class KWtable
attr_accessor :name, :state, :id0, :id1
def initialize(name, id=[], state=nil)
Something went wrong with that request. Please try again.