Skip to content

Commit 0f34e8b

Browse files
authored
Remove then keywords from if statements (#51)
Co-authored-by: Herwin <herwinw@users.noreply.github.com>
1 parent caab355 commit 0f34e8b

3 files changed

Lines changed: 7 additions & 7 deletions

File tree

lib/xmlrpc/client.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ module XMLRPC # :nodoc:
4444
#
4545
# server = XMLRPC::Client.new("www.ruby-lang.org", "/RPC2", 80)
4646
# ok, param = server.call2("michael.add", 4, 5)
47-
# if ok then
47+
# if ok
4848
# puts "4 + 5 = #{param}"
4949
# else
5050
# puts "Error:"

lib/xmlrpc/parser.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -205,7 +205,7 @@ def nodeMustBe(node, name)
205205
raise "error"
206206
end
207207

208-
if not cmp then
208+
if not cmp
209209
raise "wrong xml-rpc (name)"
210210
end
211211

@@ -217,14 +217,14 @@ def hasOnlyOneChild(node, name=nil)
217217
if node.childNodes.to_a.size != 1
218218
raise "wrong xml-rpc (size)"
219219
end
220-
if name != nil then
220+
if name != nil
221221
nodeMustBe(node.firstChild, name)
222222
end
223223
end
224224

225225

226226
def assert(b)
227-
if not b then
227+
if not b
228228
raise "assert-fail"
229229
end
230230
end
@@ -436,7 +436,7 @@ def methodCall(node)
436436
assert( (1..2).include?( node.childNodes.to_a.size ) )
437437
name = methodName(node[0])
438438

439-
if node.childNodes.to_a.size == 2 then
439+
if node.childNodes.to_a.size == 2
440440
pa = params(node[1])
441441
else # no parameters given
442442
pa = []

lib/xmlrpc/utils.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,15 +46,15 @@ def set_parser(parser)
4646

4747
def create
4848
# if set_writer was not already called then call it now
49-
if @create.nil? then
49+
if @create.nil?
5050
set_writer(Config.default_writer.new)
5151
end
5252
@create
5353
end
5454

5555
def parser
5656
# if set_parser was not already called then call it now
57-
if @parser.nil? then
57+
if @parser.nil?
5858
set_parser(Config.default_parser.new)
5959
end
6060
@parser

0 commit comments

Comments
 (0)