Skip to content

Commit 123c6b0

Browse files
author
Awesome Code
committed
Auto corrected by following Lint Ruby Style/RedundantSelf
1 parent 22ecfa4 commit 123c6b0

File tree

3 files changed

+83
-83
lines changed

3 files changed

+83
-83
lines changed

‎lib/synvert/core/node_ext.rb‎

Lines changed: 78 additions & 78 deletions
Original file line numberDiff line numberDiff line change
@@ -31,15 +31,15 @@ class Node
3131
# @return [Parser::AST::Node] name node.
3232
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
3333
def name
34-
case self.type
34+
case type
3535
when :class, :module, :def, :arg, :blockarg, :restarg
36-
self.children[0]
36+
children[0]
3737
when :defs, :const
38-
self.children[1]
38+
children[1]
3939
when :mlhs
4040
self
4141
else
42-
raise Synvert::Core::MethodNotSupported.new "name is not handled for #{self.debug_info}"
42+
raise Synvert::Core::MethodNotSupported.new "name is not handled for #{debug_info}"
4343
end
4444
end
4545

@@ -48,10 +48,10 @@ def name
4848
# @return [Parser::AST::Node] parent_class node.
4949
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
5050
def parent_class
51-
if :class == self.type
52-
self.children[1]
51+
if :class == type
52+
children[1]
5353
else
54-
raise Synvert::Core::MethodNotSupported.new "parent_class is not handled for #{self.debug_info}"
54+
raise Synvert::Core::MethodNotSupported.new "parent_class is not handled for #{debug_info}"
5555
end
5656
end
5757

@@ -60,10 +60,10 @@ def parent_class
6060
# @return [Parser::AST::Node] parent const node.
6161
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
6262
def parent_const
63-
if :const == self.type
64-
self.children[0]
63+
if :const == type
64+
children[0]
6565
else
66-
raise Synvert::Core::MethodNotSupported.new "parent_const is not handled for #{self.debug_info}"
66+
raise Synvert::Core::MethodNotSupported.new "parent_const is not handled for #{debug_info}"
6767
end
6868
end
6969

@@ -72,10 +72,10 @@ def parent_const
7272
# @return [Parser::AST::Node] receiver node.
7373
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
7474
def receiver
75-
if :send == self.type
76-
self.children[0]
75+
if :send == type
76+
children[0]
7777
else
78-
raise Synvert::Core::MethodNotSupported.new "receiver is not handled for #{self.debug_info}"
78+
raise Synvert::Core::MethodNotSupported.new "receiver is not handled for #{debug_info}"
7979
end
8080
end
8181

@@ -84,13 +84,13 @@ def receiver
8484
# @return [Parser::AST::Node] mesage node.
8585
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
8686
def message
87-
case self.type
87+
case type
8888
when :super, :zsuper
8989
:super
9090
when :send
91-
self.children[1]
91+
children[1]
9292
else
93-
raise Synvert::Core::MethodNotSupported.new "message is not handled for #{self.debug_info}"
93+
raise Synvert::Core::MethodNotSupported.new "message is not handled for #{debug_info}"
9494
end
9595
end
9696

@@ -99,17 +99,17 @@ def message
9999
# @return [Array<Parser::AST::Node>] arguments node.
100100
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
101101
def arguments
102-
case self.type
102+
case type
103103
when :def, :block
104-
ArgumentsNode.new self.children[1]
104+
ArgumentsNode.new children[1]
105105
when :defs
106-
ArgumentsNode.new self.children[2]
106+
ArgumentsNode.new children[2]
107107
when :send
108-
self.children[2..-1]
108+
children[2..-1]
109109
when :defined?
110-
self.children
110+
children
111111
else
112-
raise Synvert::Core::MethodNotSupported.new "arguments is not handled for #{self.debug_info}"
112+
raise Synvert::Core::MethodNotSupported.new "arguments is not handled for #{debug_info}"
113113
end
114114
end
115115

@@ -118,10 +118,10 @@ def arguments
118118
# @return [Parser::AST::Node] caller node.
119119
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
120120
def caller
121-
if :block == self.type
122-
self.children[0]
121+
if :block == type
122+
children[0]
123123
else
124-
raise Synvert::Core::MethodNotSupported.new "caller is not handled for #{self.debug_info}"
124+
raise Synvert::Core::MethodNotSupported.new "caller is not handled for #{debug_info}"
125125
end
126126
end
127127

@@ -130,19 +130,19 @@ def caller
130130
# @return [Array<Parser::AST::Node>] body node.
131131
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
132132
def body
133-
case self.type
133+
case type
134134
when :begin
135-
self.children
135+
children
136136
when :def, :block
137-
return [] if self.children[2].nil?
137+
return [] if children[2].nil?
138138

139-
:begin == self.children[2].type ? self.children[2].body : self.children[2..-1]
139+
:begin == children[2].type ? children[2].body : children[2..-1]
140140
when :defs
141-
return [] if self.children[3].nil?
141+
return [] if children[3].nil?
142142

143-
:begin == self.children[3].type ? self.children[3].body : self.children[3..-1]
143+
:begin == children[3].type ? children[3].body : children[3..-1]
144144
else
145-
raise Synvert::Core::MethodNotSupported.new "body is not handled for #{self.debug_info}"
145+
raise Synvert::Core::MethodNotSupported.new "body is not handled for #{debug_info}"
146146
end
147147
end
148148

@@ -151,10 +151,10 @@ def body
151151
# @return [Parser::AST::Node] condition node.
152152
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
153153
def condition
154-
if :if == self.type
155-
self.children[0]
154+
if :if == type
155+
children[0]
156156
else
157-
raise Synvert::Core::MethodNotSupported.new "condition is not handled for #{self.debug_info}"
157+
raise Synvert::Core::MethodNotSupported.new "condition is not handled for #{debug_info}"
158158
end
159159
end
160160

@@ -163,10 +163,10 @@ def condition
163163
# @return [Array<Parser::AST::Node>] keys node.
164164
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
165165
def keys
166-
if :hash == self.type
167-
self.children.map { |child| child.children[0] }
166+
if :hash == type
167+
children.map { |child| child.children[0] }
168168
else
169-
raise Synvert::Core::MethodNotSupported.new "keys is not handled for #{self.debug_info}"
169+
raise Synvert::Core::MethodNotSupported.new "keys is not handled for #{debug_info}"
170170
end
171171
end
172172

@@ -175,10 +175,10 @@ def keys
175175
# @return [Array<Parser::AST::Node>] values node.
176176
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
177177
def values
178-
if :hash == self.type
179-
self.children.map { |child| child.children[1] }
178+
if :hash == type
179+
children.map { |child| child.children[1] }
180180
else
181-
raise Synvert::Core::MethodNotSupported.new "keys is not handled for #{self.debug_info}"
181+
raise Synvert::Core::MethodNotSupported.new "keys is not handled for #{debug_info}"
182182
end
183183
end
184184

@@ -188,10 +188,10 @@ def values
188188
# @return [Boolean] true if specified key exists.
189189
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
190190
def has_key?(key)
191-
if :hash == self.type
192-
self.children.any? { |pair_node| pair_node.key.to_value == key }
191+
if :hash == type
192+
children.any? { |pair_node| pair_node.key.to_value == key }
193193
else
194-
raise Synvert::Core::MethodNotSupported.new "has_key? is not handled for #{self.debug_info}"
194+
raise Synvert::Core::MethodNotSupported.new "has_key? is not handled for #{debug_info}"
195195
end
196196
end
197197

@@ -201,11 +201,11 @@ def has_key?(key)
201201
# @return [Parser::AST::Node] value node.
202202
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
203203
def hash_value(key)
204-
if :hash == self.type
205-
value_node = self.children.find { |pair_node| pair_node.key.to_value == key }
204+
if :hash == type
205+
value_node = children.find { |pair_node| pair_node.key.to_value == key }
206206
value_node ? value_node.value : nil
207207
else
208-
raise Synvert::Core::MethodNotSupported.new "has_key? is not handled for #{self.debug_info}"
208+
raise Synvert::Core::MethodNotSupported.new "has_key? is not handled for #{debug_info}"
209209
end
210210
end
211211

@@ -214,10 +214,10 @@ def hash_value(key)
214214
# @return [Parser::AST::Node] key node.
215215
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
216216
def key
217-
if :pair == self.type
218-
self.children.first
217+
if :pair == type
218+
children.first
219219
else
220-
raise Synvert::Core::MethodNotSupported.new "key is not handled for #{self.debug_info}"
220+
raise Synvert::Core::MethodNotSupported.new "key is not handled for #{debug_info}"
221221
end
222222
end
223223

@@ -226,10 +226,10 @@ def key
226226
# @return [Parser::AST::Node] value node.
227227
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
228228
def value
229-
if :pair == self.type
230-
self.children.last
229+
if :pair == type
230+
children.last
231231
else
232-
raise Synvert::Core::MethodNotSupported.new "value is not handled for #{self.debug_info}"
232+
raise Synvert::Core::MethodNotSupported.new "value is not handled for #{debug_info}"
233233
end
234234
end
235235

@@ -238,10 +238,10 @@ def value
238238
# @return [Parser::AST::Node] variable nodes.
239239
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
240240
def left_value
241-
if %i[masgn lvasgn ivasgn].include? self.type
242-
self.children[0]
241+
if %i[masgn lvasgn ivasgn].include? type
242+
children[0]
243243
else
244-
raise Synvert::Core::MethodNotSupported.new "left_value is not handled for #{self.debug_info}"
244+
raise Synvert::Core::MethodNotSupported.new "left_value is not handled for #{debug_info}"
245245
end
246246
end
247247

@@ -250,10 +250,10 @@ def left_value
250250
# @return [Array<Parser::AST::Node>] variable nodes.
251251
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
252252
def right_value
253-
if %i[masgn lvasgn ivasgn].include? self.type
254-
self.children[1]
253+
if %i[masgn lvasgn ivasgn].include? type
254+
children[1]
255255
else
256-
raise Synvert::Core::MethodNotSupported.new "right_value is not handled for #{self.debug_info}"
256+
raise Synvert::Core::MethodNotSupported.new "right_value is not handled for #{debug_info}"
257257
end
258258
end
259259

@@ -262,69 +262,69 @@ def right_value
262262
# @return [Object] exact value.
263263
# @raise [Synvert::Core::MethodNotSupported] if calls on other node.
264264
def to_value
265-
case self.type
265+
case type
266266
when :int, :str, :sym
267-
self.children.last
267+
children.last
268268
when :true
269269
true
270270
when :false
271271
false
272272
when :array
273-
self.children.map(&:to_value)
273+
children.map(&:to_value)
274274
when :irange
275-
(self.children.first.to_value..self.children.last.to_value)
275+
(children.first.to_value..children.last.to_value)
276276
when :begin
277-
self.children.first.to_value
277+
children.first.to_value
278278
else
279-
raise Synvert::Core::MethodNotSupported.new "to_value is not handled for #{self.debug_info}"
279+
raise Synvert::Core::MethodNotSupported.new "to_value is not handled for #{debug_info}"
280280
end
281281
end
282282

283283
def to_s
284-
if :mlhs == self.type
285-
"(#{self.children.map(&:name).join(', ')})"
284+
if :mlhs == type
285+
"(#{children.map(&:name).join(', ')})"
286286
end
287287
end
288288

289289
def debug_info
290290
"\n" +
291291
[
292-
"file: #{self.loc.expression.source_buffer.name}",
293-
"line: #{self.loc.expression.line}",
294-
"source: #{self.to_source}",
295-
"node: #{self.inspect}"
292+
"file: #{loc.expression.source_buffer.name}",
293+
"line: #{loc.expression.line}",
294+
"source: #{to_source}",
295+
"node: #{inspect}"
296296
].join("\n")
297297
end
298298

299299
# Get the source code of current node.
300300
#
301301
# @return [String] source code.
302302
def to_source
303-
if self.loc.expression
304-
self.loc.expression.source
303+
if loc.expression
304+
loc.expression.source
305305
end
306306
end
307307

308308
# Get the indent of current node.
309309
#
310310
# @return [Integer] indent.
311311
def indent
312-
self.loc.expression.column
312+
loc.expression.column
313313
end
314314

315315
# Get the line of current node.
316316
#
317317
# @return [Integer] line.
318318
def line
319-
self.loc.expression.line
319+
loc.expression.line
320320
end
321321

322322
# Recursively iterate all child nodes of current node.
323323
#
324324
# @yield [child] Gives a child node.
325325
# @yieldparam child [Parser::AST::Node] child node
326326
def recursive_children
327-
self.children.each do |child|
327+
children.each do |child|
328328
if Parser::AST::Node === child
329329
yield child
330330
child.recursive_children { |c| yield c }
@@ -366,8 +366,8 @@ def match?(rules)
366366
def rewritten_source(code)
367367
code.gsub(/{{(.*?)}}/m) do
368368
old_code = $1
369-
if self.respond_to? old_code.split(/\.|\[/).first
370-
evaluated = self.instance_eval old_code
369+
if respond_to? old_code.split(/\.|\[/).first
370+
evaluated = instance_eval old_code
371371
case evaluated
372372
when Parser::AST::Node
373373
evaluated.loc.expression.source

‎lib/synvert/core/rewriter.rb‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -151,14 +151,14 @@ def initialize(group, name, &block)
151151
# Process the rewriter.
152152
# It will call the block.
153153
def process
154-
self.instance_eval &@block
154+
instance_eval &@block
155155
end
156156

157157
# Process rewriter with sandbox mode.
158158
# It will call the block but doesn't change any file.
159159
def process_with_sandbox
160160
@sandbox = true
161-
self.process
161+
process
162162
@sandbox = false
163163
end
164164

0 commit comments

Comments
 (0)