skeptic 0.0.12 → 0.0.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b6e1fc6b21c5b3fa8b5ba3c556552cdcce8086ba
-  data.tar.gz: 31ce95b2d523a1043f8048b50f196a3e8beab10b
+  metadata.gz: c50ddb8500c67652b5d76236fed471b66e7c10a2
+  data.tar.gz: 701c0cfc317879fcb8d23834dcaffab02ba5d7a7
 SHA512:
-  metadata.gz: 8eb3496b7e1e23ec3e8f7ec89a7b84e5de5bdf95f0f33a5f5eb1a24608e716381c9bf7b0343e5eb2265af64a021f2f98e07ed535f9418e273de34747e4228b29
-  data.tar.gz: acc962c666098071ead2296fa73f415b4fc62e6d9fd8a8f06544e2a9794a3f542ba5a698dabfaf50f93b54ba958891c22c9d47b4332d224549e2babc52b16ecd
+  metadata.gz: 3af0a80ff4d84495ae3cbe21163bb368913494a69a172d0027fa616e391a9e442170aa10e58e8c195a43ecc2a63fc2ec4b209af7694e95ca77e9e5292d69ce93
+  data.tar.gz: d459748904f7e8fd71e199318f93065cd352a723955cf15912dfbfa291c4e0a77166b5d04da53c5718b6e526a8dfa3cc4e26f9ebee3dfaf29f0a80dfbd46a9c5

data/VERSION

@@ -1 +1 @@
-0.0.12
+0.0.14

data/lib/skeptic/rules/spaces_around_operators.rb

@@ -9,15 +9,19 @@ module Skeptic
       IGNORED_TOKEN_TYPES = [:on_sp, :on_ignored_nl, :on_nl, :on_lparen, :on_symbeg, :on_lbracket, :on_lbrace]
       LEFT_LIMIT_TOKEN_TYPES = [:on_lparen, :on_lbracket]
       RIGHT_LIMIT_TOKEN_TYPES = [:on_rparen, :on_rbracket]
+      WHITESPACE_TOKEN_TYPES = [:on_sp, :on_nl, :on_ignored_nl]
 
       def initialize(data)
         @violations = []
         @special_tokens_locations = []
+        @unary_token_locations = []
       end
 
       def apply_to(code, tokens, sexp)
         visit sexp
 
+        mark_unary tokens
+
         @violations = tokens.each_cons(3).select do |_, token, _|
           operator_expecting_spaces? token
         end.select do |left, operator, right|
@@ -46,6 +50,8 @@ module Skeptic
       end
 
       def no_spaces_around?(operator, left, right)
+        return false if unary_operator?(operator)
+
         if LEFT_LIMIT_TOKEN_TYPES.include?(left[1])
           mark_special_tokens right.first
         end
@@ -69,10 +75,6 @@ module Skeptic
         !special_token?(neighbour)
       end
 
-      def whitespace_token?(token)
-        token[1] == :on_sp or token[1] == :on_ignored_nl
-      end
-
       def range_operator?(operator)
         operator.last[0..1] == '..'
       end
@@ -86,6 +88,25 @@ module Skeptic
           @special_tokens_locations.include? token.first
       end
 
+      def unary_operator?(token)
+        token[1] == :on_op and @unary_token_locations.include?(token[0])
+      end
+
+      def mark_unary(tokens)
+        @unary_token_locations = []
+        last_significant_token = nil
+        tokens.each do |token|
+          if token[1] == :on_op
+            if last_significant_token == :on_op
+              @unary_token_locations << token[0]
+            end
+            last_significant_token = :on_op
+          elsif !WHITESPACE_TOKEN_TYPES.include?(token[1])
+            last_significant_token = token[1]
+          end
+        end
+      end
+
       on :blockarg, :rest_param do |ident|
         mark_special_tokens ident.last
       end

data/skeptic.gemspec

@@ -2,16 +2,16 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: skeptic 0.0.12 ruby lib
+# stub: skeptic 0.0.14 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "skeptic"
-  s.version = "0.0.12"
+  s.version = "0.0.14"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib"]
   s.authors = ["Stefan Kanev"]
-  s.date = "2015-10-13"
+  s.date = "2015-10-16"
   s.description = "An experimental, half-assed, bug-ridden and highly opinionated code analyzer."
   s.email = "stefan.kanev@gmail.com"
   s.executables = ["skeptic"]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: skeptic
 version: !ruby/object:Gem::Version
-  version: 0.0.12
+  version: 0.0.14
 platform: ruby
 authors:
 - Stefan Kanev
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-10-13 00:00:00.000000000 Z
+date: 2015-10-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: trollop