fast_ignore 0.17.0 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 78cb85cea0888d67c91de09bd54ad644510167c993dc355d187593003d341393
-  data.tar.gz: fb1fda9c1675ee28f4a6ff23191f95c2dd6e497ca2a70c4791dc5dfb85baecd1
+  metadata.gz: 88853f2613fe610772a13666845d500a4b377bbf5606e3fba3bd8842d66c17a2
+  data.tar.gz: c158117c0234816a067b3e68d7686e247bf18ea81e39b3c339fcf06ecc4026ae
 SHA512:
-  metadata.gz: 10531a1d819b74fcc442ce7591be124430d3c55c55afbccb79ba295dd89c0e85e4a1a0a75b11996557057fa2f5dbad71e9284789143395d229e3abbbbd42ba3c
-  data.tar.gz: d0b07a2757c3b962a59973581234ec9e346717d24cdbd5443073c0fd14f840fb287419d63eeb31d4aa52d3fff2e1c0842cc119ed52e836cd55ed73bcd29a3fc7
+  metadata.gz: a409a69b31021dedca95858ebed16abb4a36efbb758067dcab41bd6cbeb988acf0a62d4324f31cbb68ba4787efca4564cfc51721a50dc710f3357923775cc6e2
+  data.tar.gz: 4f0e0a8f900d6c0d008850eb8899c1c4446914149697539b9f708620054b614b310305e83df8743a3d5fb558a82ac08f0d48e44450486ad5de966943916a6ac2

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# v0.17.1
+- fix handling of backward character classes `[z-a]`
+  previously this raised a RegexpError, but git just considered it to be identical to `[z]`, now we match the git behaviour (but why would you ever do this?, i only found it because of the fuzz spec in the `leftovers` gem)
+
 # v0.17.0
 - allow overriding `exists:` in `allowed?`
 - allow setting `include_directories: true` in `allowed?`

data/lib/fast_ignore/gitignore_rule_builder.rb

@@ -108,8 +108,8 @@ class FastIgnore
       unmatchable_rule! if @s.character_class_end?
 
       until @s.character_class_end?
+        next if process_character_class_range
         next if process_backslash
-        next @re.append_character_class_dash if @s.dash?
         next if @re.append_escaped(@s.character_class_literal)
 
         unmatchable_rule!
@@ -118,6 +118,22 @@ class FastIgnore
       @re.append_character_class_close
     end
 
+    def process_character_class_range
+      start = @s.character_class_range_start
+      return unless start
+
+      start = start.delete_prefix('\\')
+
+      @re.append_escaped(start)
+
+      finish = @s.character_class_range_end.delete_prefix('\\')
+
+      return true unless start < finish
+
+      @re.append_character_class_dash
+      @re.append_escaped(finish)
+    end
+
     def process_end
       blank! if nothing_emitted?
 

data/lib/fast_ignore/gitignore_rule_scanner.rb

@@ -46,12 +46,20 @@ class FastIgnore
       skip(/\?/)
     end
 
-    def dash?
-      skip(/-/)
+    def character_class_literal
+      matched if scan(/[^\]\\][^\]\\-]*(?!-)/)
     end
 
-    def character_class_literal
-      matched if scan(/[^\]\-\\]+/)
+    def character_class_range_start
+      matched if scan(/(\\.|[^\\\]])(?=-(\\.|[^\\\]]))/)
+    end
+
+    def character_class_range_end
+      # we already confirmed this was going to match
+      # with the lookahead in character_class_range_start
+      skip(/-/)
+      scan(/(\\.|[^\\\]])/)
+      matched
     end
 
     def literal

data/lib/fast_ignore/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 class FastIgnore
-  VERSION = '0.17.0'
+  VERSION = '0.17.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: fast_ignore
 version: !ruby/object:Gem::Version
-  version: 0.17.0
+  version: 0.17.1
 platform: ruby
 authors:
 - Dana Sherson
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-03 00:00:00.000000000 Z
+date: 2022-03-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -218,7 +218,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.15
+rubygems_version: 3.1.6
 signing_key:
 specification_version: 4
 summary: Parse gitignore files, quickly