aboutsummaryrefslogtreecommitdiffstats
path: root/lib/rubygems/command.rb
diff options
context:
space:
mode:
authorOlle Jonsson <olle.jonsson@auctionet.com>2020-04-19 18:06:02 +0200
committerHiroshi SHIBATA <hsbt@ruby-lang.org>2020-06-05 07:32:42 +0900
commitf61ee674d8c5871e8e81b1a7f5884635a0ee9803 (patch)
treeaa956731f75c20eaa39b7366d0428155df685a77 /lib/rubygems/command.rb
parentdd5b918cbead5fdb641118aa4e4d90e013f90df1 (diff)
downloadruby-f61ee674d8c5871e8e81b1a7f5884635a0ee9803.tar.gz
[rubygems/rubygems] Prefer start_with? and end_with? over regex.
- In one of the cases, filenames were checked for ending with "gz" - this is changed to check for ending with ".gz" - The change was made to make it even easier to read the code, and to match only from the start of the input (as opposed to start of the line) https://github.com/rubygems/rubygems/commit/aac4290271
Diffstat (limited to 'lib/rubygems/command.rb')
-rw-r--r--lib/rubygems/command.rb4
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/rubygems/command.rb b/lib/rubygems/command.rb
index 2fb3dab2fe..088f4add99 100644
--- a/lib/rubygems/command.rb
+++ b/lib/rubygems/command.rb
@@ -456,7 +456,9 @@ class Gem::Command
until extra.empty? do
ex = []
ex << extra.shift
- ex << extra.shift if extra.first.to_s =~ /^[^-]/
+ if (!extra.first.to_s.empty? && !extra.first.to_s.start_with?("-"))
+ ex << extra.shift
+ end
result << ex if handles?(ex)
end