Change gsub to tr (#543)

Little performance improve see https://github.com/fastruby/fast-ruby#stringgsub-vs-stringtr-code
minimagick · May 28, 2022 · 575ba0f · 575ba0f
1 parent 67903eb
commit 575ba0f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/mini_magick/shell.rb b/lib/mini_magick/shell.rb
@@ -25,7 +25,7 @@ def run(command, options = {})
     def execute(command, options = {})
       stdout, stderr, status =
         log(command.join(" ")) do
-          send("execute_#{MiniMagick.shell_api.gsub("-", "_")}", command, options)
+          send("execute_#{MiniMagick.shell_api.tr("-", "_")}", command, options)
         end
 
       [stdout, stderr, status.exitstatus]

diff --git a/lib/mini_magick/tool.rb b/lib/mini_magick/tool.rb
@@ -241,7 +241,7 @@ def stdout
     #   mogrify.command.join(" ") #=> "mogrify canvas:khaki"
     #
     CREATION_OPERATORS.each do |operator|
-      define_method(operator.gsub('-', '_')) do |value = nil|
+      define_method(operator.tr('-', '_')) do |value = nil|
         self << "#{operator}:#{value}"
         self
       end