summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHiroshi SHIBATA <hsbt@ruby-lang.org>2023-03-16 13:40:35 +0900
committerHiroshi SHIBATA <hsbt@ruby-lang.org>2023-03-17 18:50:55 +0900
commit4ab23df2c4540578e77c79a8aaf06178362edb76 (patch)
tree5011e2338036662371dfa8e805c0ec391e1e4840
parentb8914a9d5de581c96abeac9290e5d505a570d00b (diff)
downloadruby-4ab23df2c4540578e77c79a8aaf06178362edb76.tar.gz
[rubygems/rubygems] util/rubocop -A --only Style/RedundantParentheses
https://github.com/rubygems/rubygems/commit/295691d4dc
-rw-r--r--lib/rubygems/ext/builder.rb2
-rw-r--r--lib/rubygems/remote_fetcher.rb2
-rw-r--r--lib/rubygems/s3_uri_signer.rb4
-rw-r--r--test/rubygems/test_gem_package_tar_writer.rb2
-rw-r--r--test/rubygems/test_gem_specification.rb2
5 files changed, 6 insertions, 6 deletions
diff --git a/lib/rubygems/ext/builder.rb b/lib/rubygems/ext/builder.rb
index 9ddcf988e5..02ca000984 100644
--- a/lib/rubygems/ext/builder.rb
+++ b/lib/rubygems/ext/builder.rb
@@ -26,7 +26,7 @@ class Gem::Ext::Builder
RbConfig::CONFIG["configure_args"] =~ /with-make-prog\=(\w+)/
make_program_name = ENV["MAKE"] || ENV["make"] || $1
unless make_program_name
- make_program_name = (RUBY_PLATFORM.include?("mswin")) ? "nmake" : "make"
+ make_program_name = RUBY_PLATFORM.include?("mswin") ? "nmake" : "make"
end
make_program = Shellwords.split(make_program_name)
diff --git a/lib/rubygems/remote_fetcher.rb b/lib/rubygems/remote_fetcher.rb
index 6af0fea00f..5a0898f084 100644
--- a/lib/rubygems/remote_fetcher.rb
+++ b/lib/rubygems/remote_fetcher.rb
@@ -114,7 +114,7 @@ class Gem::RemoteFetcher
cache_dir =
if Dir.pwd == install_dir # see fetch_command
install_dir
- elsif File.writable?(install_cache_dir) || (File.writable?(install_dir) && (!File.exist?(install_cache_dir)))
+ elsif File.writable?(install_cache_dir) || (File.writable?(install_dir) && !File.exist?(install_cache_dir))
install_cache_dir
else
File.join Gem.user_dir, "cache"
diff --git a/lib/rubygems/s3_uri_signer.rb b/lib/rubygems/s3_uri_signer.rb
index c13852337b..911468024d 100644
--- a/lib/rubygems/s3_uri_signer.rb
+++ b/lib/rubygems/s3_uri_signer.rb
@@ -10,7 +10,7 @@ class Gem::S3URISigner
end
def to_s # :nodoc:
- (super).to_s
+ super.to_s
end
end
@@ -20,7 +20,7 @@ class Gem::S3URISigner
end
def to_s # :nodoc:
- (super).to_s
+ super.to_s
end
end
diff --git a/test/rubygems/test_gem_package_tar_writer.rb b/test/rubygems/test_gem_package_tar_writer.rb
index 7bcf90012b..84869bdae1 100644
--- a/test/rubygems/test_gem_package_tar_writer.rb
+++ b/test/rubygems/test_gem_package_tar_writer.rb
@@ -282,7 +282,7 @@ class TestGemPackageTarWriter < Gem::Package::TarTestCase
]
names.each do |name|
newname, prefix = @tar_writer.split_name(name)
- assert(!(newname.empty?), "split_name() returned empty name")
+ assert(!newname.empty?, "split_name() returned empty name")
assert(newname.bytesize <= 100, "split_name() returned name longer than 100 bytes: '#{newname}' for '#{name}'")
assert(prefix.bytesize <= 155, "split_name() returned prefix longer than 155 bytes: '#{prefix}' for '#{name}'")
newname = [prefix, newname].join("/") unless prefix.empty?
diff --git a/test/rubygems/test_gem_specification.rb b/test/rubygems/test_gem_specification.rb
index 6776368dc2..30fa845cc6 100644
--- a/test/rubygems/test_gem_specification.rb
+++ b/test/rubygems/test_gem_specification.rb
@@ -1199,7 +1199,7 @@ dependencies: []
s.version = "1"
end
- spec.instance_variable_set :@licenses, (class << (Object.new);self;end)
+ spec.instance_variable_set :@licenses, (class << Object.new;self;end)
spec.loaded_from = "/path/to/file"
e = assert_raise Gem::FormatException do