Brakeman.run should always return a Tracker

instead of returning false for options[:exit_on_warn]

bin/brakeman

@@ -42,9 +42,9 @@ unless options[:app_path]
 end
 
 #Run scan and output a report
-clean = Brakeman.run options.merge(:print_report => true, :quiet => options[:quiet])
+tracker = Brakeman.run options.merge(:print_report => true, :quiet => options[:quiet])
 
 #Return error code if --exit-on-warn is used and warnings were found
-if options[:exit_on_warn] and not clean
+if options[:exit_on_warn] and not tracker.checks.all_warnings.empty?
   exit Brakeman::Warnings_Found_Exit_Code
 end

lib/brakeman.rb

@@ -262,10 +262,6 @@ module Brakeman
       puts tracker.report.send(options[:output_format])
     end
 
-    if options[:exit_on_warn] and not tracker.checks.all_warnings.empty?
-      return false
-    end
-
     tracker
   end