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 | 82171c3406a790ac9740c439e2aafce21853e0a0 (patch) | |
tree | 71139d061c87fbde7aefc12d9364339df6d228b8 /lib/custodian/protocoltest/tcp.rb | |
parent | 1383d0f44f8ce524c297714bd288f95a548107ed (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 |