metric_fu-roodi 2.2.1 → 2.2.2

data/Gemfile

@@ -1,4 +1,4 @@
-source :rubygems
+source 'https://rubygems.org'
 
 gemspec
 

data/History.txt

@@ -1,7 +1,25 @@
+= Master
+
+Features
+Fixes
+Misc
+
+= 2.2.2
+
+Fixes
+  * Better check if OS is windows (#2 Martin Gotink)
+  * Accept 'next' in a rescue block (#1 Virgil Mihailovici)
+
+= 2.2.1
+
+Fixes
+  * Rescue line count when there are no lines
+
 = 2.2.0
 
 * Pull down updates from https://github.com/zdennis/roodi that includes updates from https://github.com/hooroo/roodi and https://github.com/aselder/roodi re: pull request https://github.com/martinjandrews/roodi/pull/12 https://github.com/martinjandrews/roodi/pull/11
 * Did not pull in updates from https://github.com/ssassi/roodi/commits/master re: https://github.com/martinjandrews/roodi/pull/10
+* Release under gem name metric_fu-roodi
 
 = 2.0.1
 

data/lib/roodi/checks/empty_rescue_body_check.rb

@@ -7,7 +7,7 @@ module Roodi
     # When the body of a rescue block is empty, exceptions can get caught and swallowed without
     # any feedback to the user.
     class EmptyRescueBodyCheck < Check
-      STATEMENT_NODES = [:fcall, :return, :attrasgn, :vcall, :nil, :call, :lasgn, :true, :false]
+      STATEMENT_NODES = [:fcall, :return, :attrasgn, :vcall, :nil, :call, :lasgn, :true, :false, :next]
       
       def interesting_nodes
         [:resbody]

data/lib/roodi/core/parser.rb

@@ -1,5 +1,6 @@
 require 'rubygems'
 require 'ruby_parser'
+require 'rbconfig'
 
 module Roodi
   module Core
@@ -14,7 +15,7 @@ module Roodi
       
       def silence_stream(stream)
         old_stream = stream.dup
-        stream.reopen(RUBY_PLATFORM =~ /mswin/ ? 'NUL:' : '/dev/null')
+        stream.reopen(null_stream_output)
         stream.sync = true
         yield
       ensure
@@ -25,6 +26,23 @@ module Roodi
         @parser ||= RubyParser.new
         @parser.parse(content, filename)
       end
+      
+      def null_stream_output
+        null_output_for_platform(RbConfig::CONFIG['host_os'])
+      end
+      
+      def null_output_for_platform(host_os)
+        case host_os
+        when windows_host_os_matcher
+          'NUL:'
+        else
+          '/dev/null'
+        end
+      end
+      
+      def windows_host_os_matcher
+        /mingw|mswin32|cygwin/o
+      end
     end
   end
 end

data/lib/roodi/version.rb

@@ -1,3 +1,3 @@
 module Roodi
-  VERSION = '2.2.1'
+  VERSION = '2.2.2'
 end

data/spec/roodi/checks/empty_rescue_body_check_spec.rb

@@ -137,4 +137,18 @@ describe Roodi::Checks::EmptyRescueBodyCheck do
     errors = @roodi.errors
     errors.should be_empty
   end
+
+  it "should accept a rescue block that has only a next statement" do
+    content = <<-END
+    begin
+      call_method
+    rescue Exception => e
+      next
+    end
+    END
+    @roodi.check_content(content)
+    errors = @roodi.errors
+    errors.should be_empty
+  end
+
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: metric_fu-roodi
 version: !ruby/object:Gem::Version
-  version: 2.2.1
+  version: 2.2.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-03 00:00:00.000000000 Z
+date: 2013-08-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby_parser
@@ -116,7 +116,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2672130625479100514
+      hash: -3420305304026942047
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -125,10 +125,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2672130625479100514
+      hash: -3420305304026942047
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Roodi stands for Ruby Object Oriented Design Inferometer