Use "if" instead of "unless" for multiple conditions

droonga · Jun 27, 2014 · c75091d · c75091d
1 parent ccbfdbc
commit c75091d
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/lib/droonga/forwarder.rb b/lib/droonga/forwarder.rb
@@ -74,8 +74,7 @@ def resume
     private
     def output(receiver, message, command, arguments)
       logger.trace("output: start")
-      # TODO: IMPROVE ME: Should not use "unless" and "and". It is confused.
-      unless receiver.is_a?(String) and command.is_a?(String)
+      if not receiver.is_a?(String) or not command.is_a?(String))
         logger.trace("output: abort: invalid argument",
                      :receiver => receiver,
                      :command  => command)