Don't use parenthesis aroudn conditions in an if.

1 files changed, 5 insertions, 5 deletions

diff --git a/lib/custodian/testfactory.rb b/lib/custodian/testfactory.rb
index 03222a2..748e114 100644
--- a/lib/custodian/testfactory.rb
+++ b/lib/custodian/testfactory.rb
@@ -31,8 +31,8 @@ module Custodian
     def self.create( line )
 
 
-      raise ArgumentError, "The type of test to create cannot be nil" if ( line.nil? )
-      raise ArgumentError, "The type of test to create must be a string" unless ( line.kind_of? String )
+      raise ArgumentError, "The type of test to create cannot be nil" if  line.nil? 
+      raise ArgumentError, "The type of test to create must be a string" unless  line.kind_of? String 
 
       #
       #  The array we return.
@@ -43,12 +43,12 @@ module Custodian
       #
       # If this is an obvious protocol test.
       #
-      if ( line =~ /must\s+(not\s+)?run\s+(\S+)(\s+|\.|$)/ )
+      if  line =~ /must\s+(not\s+)?run\s+(\S+)(\s+|\.|$)/ 
 
         test_type = $2.dup
         test_type.chomp!( "." )
 
-        if ( @@subclasses[test_type].nil? )
+        if  @@subclasses[test_type].nil? 
             raise ArgumentError, "There is no handler registered for the '#{test_type}' test-type"
         end
 
@@ -66,7 +66,7 @@ module Custodian
             #
             # We do this only after we've instantiated the test.
             #
-            if ( line =~ /\s+otherwise\s+'([^']+)'/ )
+            if  line =~ /\s+otherwise\s+'([^']+)'/ 
               obj.set_notification_text( $1.dup )
             end