Iterate based on feedback from @jnunemaker

Author: jonmagic

lib/grim/image_magick_processor.rb

@@ -48,18 +48,15 @@ def save(pdf, index, path, options)
         command_env['PATH'] = "#{File.dirname(@ghostscript_path)}#{File::PATH_SEPARATOR}#{ENV['PATH']}"
       end
 
-      log(command_env, command)
+      Grim.logger.debug { "Running imagemagick command" }
+      if command_env.any?
+        Grim.logger.debug { command_env.map {|k,v| "#{k}=#{v}" }.join(" ") }
+      end
+      Grim.logger.debug { command.join(" ") }
+
       result, status = Open3.capture2e(command_env, command.join(" "))
 
       status.success? || raise(UnprocessablePage, result)
     end
-
-    def log(command_env, command)
-      Grim.logger.debug "Running imagemagick command"
-      if command_env.any?
-        Grim.logger.debug command_env.map {|k,v| "#{k}=#{v}" }.join(" ")
-      end
-      Grim.logger.debug command.join(" ")
-    end
   end
 end

lib/grim/null_logger.rb

@@ -1,11 +1,9 @@
 require "logger"
 
 module Grim
-  class NullLogger < ::Logger
-    def initialize(*args)
-    end
-
-    def add(*args, &block)
-    end
+  class NullLogger
+    def initialize(*args); end
+    def debug(*args); end
+    def info(*args); end
   end
 end

lib/grim/page.rb

@@ -51,8 +51,8 @@ def save(path, options={})
     #
     def text
       command = [@pdftotext_path, "-enc", "UTF-8", "-f", @number, "-l", @number, Shellwords.escape(@pdf.path), "-"].join(' ')
-      Grim.logger.debug "Running pdftotext command"
-      Grim.logger.debug command
+      Grim.logger.debug { "Running pdftotext command" }
+      Grim.logger.debug { command }
       `#{command}`
     end
   end