Discource-C/discourse
1
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-02-21 11:25:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

DEV: Remove String#match? and Regexp.match? polyfills (#12454)

Browse Source 
This was added in Ruby 2.4. Discourse will no longer run on ruby versions older than that, so we can drop this polyfill
This commit is contained in:
David Taylor 2021-03-19 19:55:44 +00:00 committed by GitHub
parent 374ab82dd6
commit 56a573ab4b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 0 additions and 21 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
lib/freedom_patches/match.rb
View File

@ -1,10 +0,0 @@
# frozen_string_literal: true
class String
# new to Ruby 2.4, fastest way of matching a string to a regex
unless method_defined? :match?
def match?(regex)
!!(self =~ regex)
end
end
end

11
lib/freedom_patches/regexp.rb
View File

@ -1,11 +0,0 @@
# frozen_string_literal: true
unless ::Regexp.instance_methods.include?(:match?)
class ::Regexp
# this is the fast way of checking a regex (zero string allocs) added in Ruby 2.4
# backfill it for now
def match?(string)
!!(string =~ self)
end
end
end