flay 1.2.0 → 1.2.1

data/History.txt

@@ -1,3 +1,15 @@
+=== 1.2.1 / 2009-03-16
+
+* 3 minor enhancements:
+
+  * Added gauntlet_flay.rb
+  * Cached value of plugins loaded.
+  * Refactored and separated analysis phase from process phase
+
+* 1 bug fix:
+
+  * Added bin dir to default dirs list in FlayTask
+
 === 1.2.0 / 2009-03-09
 
 * 2 major enhancements:

data/Manifest.txt

@@ -6,4 +6,5 @@ bin/flay
 lib/flay.rb
 lib/flay_erb.rb
 lib/flay_task.rb
+lib/gauntlet_flay.rb
 test/test_flay.rb

data/Rakefile

@@ -6,7 +6,11 @@ require 'hoe'
 Hoe::add_include_dirs("../../sexp_processor/dev/lib",
                       "../../ruby_parser/dev/lib")
 
-require './lib/flay.rb'
+begin
+  require 'flay'
+rescue LoadError
+  load 'lib/flay.rb'
+end
 
 Hoe.new('flay', Flay::VERSION) do |flay|
   flay.rubyforge_name = 'seattlerb'

data/lib/flay.rb

@@ -11,7 +11,7 @@ require 'ruby_parser'
 abort "update rubygems to >= 1.3.1" unless  Gem.respond_to? :find_files
 
 class Flay
-  VERSION = '1.2.0'
+  VERSION = '1.2.1'
 
   def self.default_options
     {
@@ -71,17 +71,20 @@ class Flay
   end
 
   def self.load_plugins
-    plugins = Gem.find_files("flay_*.rb").reject { |path| path =~ /flay_task/ }
-
-    plugins.each do |plugin|
-      begin
-        load plugin
-      rescue LoadError => e
-        warn "error loading #{plugin.inspect}: #{e.message}. skipping..."
+    unless defined? @@plugins then
+      plugins = Gem.find_files("flay_*.rb").reject { |p| p =~ /flay_task/ }
+
+      plugins.each do |plugin|
+        begin
+          load plugin
+        rescue LoadError => e
+          warn "error loading #{plugin.inspect}: #{e.message}. skipping..."
+        end
       end
-    end
 
-    plugins.map { |f| File.basename(f, '.rb').sub(/^flay_/, '') }
+      @@plugins = plugins.map { |f| File.basename(f, '.rb').sub(/^flay_/, '') }
+    end
+    @@plugins
   end
 
   attr_accessor :mass_threshold, :total, :identical, :masses
@@ -99,7 +102,7 @@ class Flay
     require 'ruby2ruby' if @option[:verbose]
   end
 
-  def process(*files)
+  def process(*files) # TODO: rename from process - should act as SexpProcessor
     files.each do |file|
       warn "Processing #{file}"
 
@@ -127,6 +130,10 @@ class Flay
 
     process_fuzzy_similarities if option[:fuzzy]
 
+    analyze
+  end
+
+  def analyze
     self.prune
 
     self.hashes.each do |hash,nodes|

data/lib/flay_task.rb

@@ -6,7 +6,7 @@ class FlayTask < Rake::TaskLib
 
   def initialize name = :flay, threshold = 200, dirs = nil
     @name      = name
-    @dirs      = dirs || %w(app lib test spec)
+    @dirs      = dirs || %w(app bin lib spec test)
     @threshold = threshold
     @verbose   = Rake.application.options.trace
 
@@ -24,7 +24,8 @@ class FlayTask < Rake::TaskLib
       flay.process(*Flay.expand_dirs_to_files(dirs))
       flay.report if verbose
 
-      raise "Flay total too high! #{flay.total} > #{threshold}" if flay.total > threshold
+      raise "Flay total too high! #{flay.total} > #{threshold}" if
+        flay.total > threshold
     end
     self
   end

data/lib/gauntlet_flay.rb

@@ -0,0 +1,100 @@
+#!/usr/bin/ruby -ws
+
+$: << 'lib' << '../../ParseTree/dev/lib' << '../../flay/dev/lib'
+
+$v ||= false # HACK
+
+require 'rubygems'
+require 'flay'
+
+require 'gauntlet'
+require 'pp'
+
+class FlayGauntlet < Gauntlet
+  $owners       = {}
+  $score_file   = 'flay-scores.yml'
+  $misc_error   = {:total => -1, :average => -1, :methods => {}}
+  $syntax_error = {:total => -2, :average => -2, :methods => {}}
+  $no_gem       = {:total => -4, :average => -4, :methods => {}}
+
+  # copied straight from hoedown.rb 
+  my_projects = %w[InlineFortran ParseTree RubyInline RubyToC
+                   ZenHacks ZenTest bfts box_layout
+                   change_class flay flog gauntlet heckle
+                   hoe image_science miniunit minitest
+                   minitest_tu_shim png ruby2ruby ruby_parser
+                   rubyforge test-unit un vlad zenprofile
+                   zentest]
+
+  MY_PROJECTS = Regexp.union(*my_projects)
+
+  def run name
+    warn name
+    self.data[name] = score_for '.'
+    self.dirty = true
+  end
+
+  def display_report max
+    good_data  = {}
+    bad_count  = 0
+    zero_count = 0
+
+    @data.each do |name, flay|
+      case
+      when flay < 0 then
+        bad_count += 1
+      when flay == 0 then
+        zero_count += 1
+      else
+        good_data[name] = flay
+      end
+    end
+
+    scores = good_data.values
+
+    # SWEET JESUS:
+    #
+    # without zeros:
+    #   average flay: 1487.23 +/- 7800.16
+    # with zeros:
+    #   average flay:  988.69 +/- 6398.45
+
+    puts "broken projects : %d" % bad_count
+    puts "great projects  : %d" % zero_count
+    puts "bad projects    : %d" % good_data.size
+    puts "average flay    : %.2f +/- %.2f" % [scores.average, scores.stddev]
+
+    top = good_data.sort_by { |name,flay| -flay }.first max
+
+    puts
+    top.each_with_index do |(name, flay), i|
+      puts "%3d: %10.2f: %s" % [ i, flay, name ]
+    end
+  end
+
+  ############################################################
+  # OTHER
+  ############################################################
+
+  def score_for dir
+    # files = `find #{dir} -name \\*.rb | grep -v gen.*templ`.split(/\n/)
+    flayer = Flay.new
+
+    dirs = %w(app lib test spec).reject { |f| ! File.directory? f }
+
+    flay = Flay.new
+    flay.process(*Flay.expand_dirs_to_files(dirs))
+    flay.total
+  rescue Interrupt
+    # let us break out
+  rescue Exception
+    -1
+  end
+end
+
+max    = (ARGV.shift || 10).to_i
+filter = ARGV.shift
+filter = Regexp.new filter if filter
+flayer = FlayGauntlet.new
+flayer.run_the_gauntlet filter
+flayer.display_report max

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: flay
 version: !ruby/object:Gem::Version 
-  version: 1.2.0
+  version: 1.2.1
 platform: ruby
 authors: 
 - Ryan Davis
@@ -30,7 +30,7 @@ cert_chain:
   FBHgymkyj/AOSqKRIpXPhjC6
   -----END CERTIFICATE-----
 
-date: 2009-03-09 00:00:00 -07:00
+date: 2009-03-16 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -61,7 +61,7 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: 1.10.0
+        version: 1.11.0
     version: 
 description: Flay analyzes code for structural similarities. Differences in literal values, variable, class, method names, whitespace, programming style, braces vs do/end, etc are all ignored. Making this totally rad.
 email: 
@@ -83,6 +83,7 @@ files:
 - lib/flay.rb
 - lib/flay_erb.rb
 - lib/flay_task.rb
+- lib/gauntlet_flay.rb
 - test/test_flay.rb
 has_rdoc: true
 homepage: http://ruby.sadi.st/

metadata.gz.sig

@@ -1 +1,3 @@
-�B�쑊 �u�4`. �,�*s�O
+�]|��4�H��i?��?�e���ݙV���7�`��T�
+ɏbѹܢ*��S�
�H���z3�4m��.3�`�?����8�J~NK�y!ڃ�+�������[��P$�/��ޅ�Nq7o�&�Y(m����FN��m�"��G�j�Q��2���v)��i}g�uCdFg9�	="��1�i=���P���e��
+��v�k��@Οs2��+�HUĚe�:0������pF-*��Ȭ�խ�J\wj�*��n��