Merge pull request #7128 from bayandin/patch-1

rewrite_python_shebang: move loop invariants out of loop
This commit is contained in:
Michka Popoff 2020-03-06 07:23:24 +01:00 committed by GitHub
commit d3459161de
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -88,9 +88,9 @@ module Language
end
def self.rewrite_python_shebang(python_path)
regex = %r{^#! ?/usr/bin/(env )?python([23](\.\d{1,2})?)$}
maximum_regex_length = 28 # the length of "#! /usr/bin/env pythonx.yyy$"
Pathname(".").find do |f|
regex = %r{^#! ?/usr/bin/(env )?python([23](\.\d{1,2})?)$}
maximum_regex_length = "#! /usr/bin/env pythonx.yyy$".length
next unless f.file?
next unless regex.match?(f.read(maximum_regex_length))