You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@buildr.apache.org by do...@apache.org on 2014/07/25 23:27:38 UTC

[5/7] git commit: Whitespace cleanup

Whitespace cleanup


Project: http://git-wip-us.apache.org/repos/asf/buildr/repo
Commit: http://git-wip-us.apache.org/repos/asf/buildr/commit/3e3c12c3
Tree: http://git-wip-us.apache.org/repos/asf/buildr/tree/3e3c12c3
Diff: http://git-wip-us.apache.org/repos/asf/buildr/diff/3e3c12c3

Branch: refs/heads/master
Commit: 3e3c12c39e65d4be5c7c8721dfca6b5c4fc367d4
Parents: 4e6792d
Author: Peter Donald <pe...@realityforge.org>
Authored: Sat Jul 26 07:24:21 2014 +1000
Committer: Peter Donald <pe...@realityforge.org>
Committed: Sat Jul 26 07:24:21 2014 +1000

----------------------------------------------------------------------
 addon/buildr/findbugs.rb | 22 +++++++++++-----------
 1 file changed, 11 insertions(+), 11 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/buildr/blob/3e3c12c3/addon/buildr/findbugs.rb
----------------------------------------------------------------------
diff --git a/addon/buildr/findbugs.rb b/addon/buildr/findbugs.rb
index 14b5a15..7eb4688 100644
--- a/addon/buildr/findbugs.rb
+++ b/addon/buildr/findbugs.rb
@@ -44,18 +44,19 @@ module Buildr
         ]
       end
 
-      def findbugs(output_file, source_paths, analyze_paths, options = { })
+      def findbugs(output_file, source_paths, analyze_paths, options = {})
         dependencies = (options[:dependencies] || []) + self.dependencies
         cp = Buildr.artifacts(dependencies).each { |a| a.invoke() if a.respond_to?(:invoke) }.map(&:to_s).join(File::PATH_SEPARATOR)
 
         args = {
+          :output => options[:output] || 'xml',
+          :outputFile => output_file,
             :output => "xml:withMessages",
-            :outputFile => output_file,
-            :effort => 'max',
-            :pluginList => '',
-            :classpath => cp,
-            :timeout => "90000000",
-            :debug => "false"
+          :effort => 'max',
+          :pluginList => '',
+          :classpath => cp,
+          :timeout => '90000000',
+          :debug => 'false'
         }
         args[:failOnError] = true if options[:fail_on_error]
         args[:excludeFilter] = options[:exclude_filter] if options[:exclude_filter]
@@ -64,8 +65,8 @@ module Buildr
         mkdir_p File.dirname(output_file)
 
         Buildr.ant('findBugs') do |ant|
-          ant.taskdef :name =>'findBugs',
-                      :classname =>'edu.umd.cs.findbugs.anttask.FindBugsTask',
+          ant.taskdef :name => 'findBugs',
+                      :classname => 'edu.umd.cs.findbugs.anttask.FindBugsTask',
                       :classpath => cp
           ant.findBugs args do
             source_paths.each do |source_path|
@@ -146,7 +147,7 @@ module Buildr
       end
 
       def properties
-        @properties ||= { }
+        @properties ||= {}
       end
 
       attr_writer :java_args
@@ -174,7 +175,6 @@ module Buildr
       end
 
       attr_reader :project
-
     end
 
     module ProjectExtension