diff options
author | Steve Kemp <steve@steve.org.uk> | 2015-03-09 13:10:32 +0000 |
---|---|---|
committer | Steve Kemp <steve@steve.org.uk> | 2015-03-09 13:10:32 +0000 |
commit | 1ce0906b3480702085d4596dbb4758c115ab298f (patch) | |
tree | 695cd365c2ff73904328a049f3b9cfcc5a0a1abd /lib/custodian/protocoltest/ldap.rb | |
parent | 891b720013f06f092f6be82adad95e5551c696b6 (diff) |
Don't use parenthesis aroudn conditions in an if.
Diffstat (limited to 'lib/custodian/protocoltest/ldap.rb')
-rw-r--r-- | lib/custodian/protocoltest/ldap.rb | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/lib/custodian/protocoltest/ldap.rb b/lib/custodian/protocoltest/ldap.rb index 56203b2..ba8b442 100644 --- a/lib/custodian/protocoltest/ldap.rb +++ b/lib/custodian/protocoltest/ldap.rb @@ -39,24 +39,24 @@ module Custodian @ldap_user = nil @ldap_pass = nil - if ( line =~ /with\s+username\s+'([^']+)'/ ) + if line =~ /with\s+username\s+'([^']+)'/ @ldap_user = $1.dup end - if ( line =~ /with\s+password\s+'([^']+)'/ ) + if line =~ /with\s+password\s+'([^']+)'/ @ldap_pass = $1.dup end - if ( @ldap_user.nil? ) + if @ldap_user.nil? raise ArgumentError, "No username specified: #{@line}" end - if ( @ldap_pass.nil? ) + if @ldap_pass.nil? raise ArgumentError, "No password specified: #{@line}" end # # Is this test inverted? # - if ( line =~ /must\s+not\s+run\s+/ ) + if line =~ /must\s+not\s+run\s+/ @inverted = true else @inverted = false @@ -65,7 +65,7 @@ module Custodian # # Save the port # - if ( line =~ /on\s+([0-9]+)/ ) + if line =~ /on\s+([0-9]+)/ @port = $1.dup.to_i else @port = 389 @@ -112,7 +112,7 @@ module Custodian # Bind. ldap.bind( @ldap_user, @ldap_pass ) - if ( ldap.bound? ) + if ldap.bound? # # Search |