diff options
author | Steve Kemp <steve@steve.org.uk> | 2015-03-09 13:14:15 +0000 |
---|---|---|
committer | Steve Kemp <steve@steve.org.uk> | 2015-03-09 13:14:15 +0000 |
commit | 3acb2e1be6ddeb421efbd1b3598c1ed547d1b38b (patch) | |
tree | b07562ee35bcdfa714e906c40886168a80436bb9 /lib/custodian/protocoltest/tcp.rb | |
parent | 5a64aa7f443625305701f4db1e16340a87cb0b76 (diff) |
Avoid redudent returns.
The last expression of a method is the return value. So:
def foo; false ; end
Is the same as:
def foo; return false; end
Diffstat (limited to 'lib/custodian/protocoltest/tcp.rb')
-rw-r--r-- | lib/custodian/protocoltest/tcp.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/custodian/protocoltest/tcp.rb b/lib/custodian/protocoltest/tcp.rb index dfaab2d..60c7be7 100644 --- a/lib/custodian/protocoltest/tcp.rb +++ b/lib/custodian/protocoltest/tcp.rb @@ -126,7 +126,7 @@ module Custodian # reset the error, in case we were previously executed. @error = nil - return( run_test_internal( @host, @port, @banner, ( ! @banner.nil? ) ) ) + ( run_test_internal( @host, @port, @banner, ( ! @banner.nil? ) ) ) end @@ -246,7 +246,7 @@ module Custodian # All was OK # @error = nil - return true + true end @@ -318,7 +318,7 @@ module Custodian return false end @error = "Misc failure" - return false + false end |