byebug 10.0.0 → 11.1.0

data/lib/byebug/frame.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/file"
+require_relative "helpers/file"
 
 module Byebug
   #
@@ -118,7 +118,7 @@ module Byebug
     # Properly formatted frame number of frame
     #
     def deco_pos
-      format("%-2d", pos)
+      format("%-2<pos>d", pos: pos)
     end
 
     #

data/lib/byebug/helpers/bin.rb

@@ -39,8 +39,8 @@ module Byebug
         ENV["PATHEXT"] ? ENV["PATHEXT"].split(";") : [""]
       end
 
-      def real_executable?(f)
-        File.executable?(f) && !File.directory?(f)
+      def real_executable?(file)
+        File.executable?(file) && !File.directory?(file)
       end
     end
   end

data/lib/byebug/helpers/eval.rb

@@ -63,22 +63,22 @@ module Byebug
         yield(e)
       end
 
-      def msg(e)
-        msg = Setting[:stack_on_error] ? error_msg(e) : warning_msg(e)
+      def msg(exception)
+        msg = Setting[:stack_on_error] ? error_msg(exception) : warning_msg(exception)
 
         pr("eval.exception", text_message: msg)
       end
 
-      def error_msg(e)
-        at = e.backtrace
+      def error_msg(exception)
+        at = exception.backtrace
 
-        locations = ["#{at.shift}: #{warning_msg(e)}"]
+        locations = ["#{at.shift}: #{warning_msg(exception)}"]
         locations += at.map { |path| "  from #{path}" }
         locations.join("\n")
       end
 
-      def warning_msg(e)
-        "#{e.class} Exception: #{e.message}"
+      def warning_msg(exception)
+        "#{exception.class} Exception: #{exception.message}"
       end
 
       #

data/lib/byebug/helpers/frame.rb

@@ -21,7 +21,7 @@ module Byebug
 
       def adjust_frame(new_frame)
         return frame_err("too_low") if new_frame >= context.stack_size
-        return frame_err("too_high") if new_frame < 0
+        return frame_err("too_high") if new_frame.negative?
 
         context.frame = new_frame
         processor.prev_line = nil
@@ -67,8 +67,8 @@ module Byebug
       #
       # @param i [Integer] Integer to be converted in a proper positive index.
       #
-      def index_from_start(i)
-        i >= 0 ? i : context.stack_size + i
+      def index_from_start(index)
+        index >= 0 ? index : context.stack_size + index
       end
     end
   end

data/lib/byebug/helpers/parse.rb

@@ -15,9 +15,7 @@ module Byebug
       # purpose.
       #
       def get_int(str, cmd, min = nil, max = nil)
-        if str !~ /\A-?[0-9]+\z/
-          return nil, pr("parse.errors.int.not_number", cmd: cmd, str: str)
-        end
+        return nil, pr("parse.errors.int.not_number", cmd: cmd, str: str) unless /\A-?[0-9]+\z/.match?(str)
 
         int = str.to_i
         if min && int < min

data/lib/byebug/helpers/string.rb

@@ -25,9 +25,6 @@ module Byebug
       #
       # Removes a number of leading whitespace for each input line.
       #
-      # @note Might be unnecessary when Ruby 2.2 support is dropped and we can
-      # use squiggly heredoc's.
-      #
       def deindent(str, leading_spaces: 6)
         str.gsub(/^ {#{leading_spaces}}/, "")
       end

data/lib/byebug/helpers/toggle.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/parse"
+require_relative "parse"
 
 module Byebug
   module Helpers
@@ -15,9 +15,7 @@ module Byebug
 
         select_breakpoints(is_enable, args).each do |b|
           enabled = (is_enable == "enable")
-          if enabled && !syntax_valid?(b.expr)
-            raise pr("toggle.errors.expression", expr: b.expr)
-          end
+          raise pr("toggle.errors.expression", expr: b.expr) if enabled && !syntax_valid?(b.expr)
 
           puts pr("toggle.messages.toggled", bpnum: b.id,
                                              endis: enabled ? "en" : "dis")

data/lib/byebug/helpers/var.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/eval"
+require_relative "eval"
 
 module Byebug
   module Helpers

data/lib/byebug/history.rb

@@ -45,7 +45,7 @@ module Byebug
     def save
       n_cmds = Setting[:histsize] > size ? size : Setting[:histsize]
 
-      open(Setting[:histfile], "w") do |file|
+      File.open(Setting[:histfile], "w") do |file|
         n_cmds.times { file.puts(pop) }
       end
 
@@ -91,10 +91,10 @@ module Byebug
     end
 
     #
-    # Array of ids of the last n commands.
+    # Array of ids of the last +number+ commands.
     #
-    def last_ids(n)
-      (1 + size - n..size).to_a
+    def last_ids(number)
+      (1 + size - number..size).to_a
     end
 
     #
@@ -121,7 +121,7 @@ module Byebug
     # For now, empty lines and consecutive duplicates.
     #
     def ignore?(buf)
-      return true if /^\s*$/ =~ buf
+      return true if /^\s*$/.match?(buf)
       return false if Readline::HISTORY.empty?
 
       buffer[Readline::HISTORY.length - 1] == buf

data/lib/byebug/interface.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
-require "byebug/setting"
-require "byebug/history"
-require "byebug/helpers/file"
+require_relative "setting"
+require_relative "history"
+require_relative "helpers/file"
 
 #
 # Namespace for all of byebug's code
@@ -141,6 +141,6 @@ module Byebug
   end
 end
 
-require "byebug/interfaces/local_interface"
-require "byebug/interfaces/script_interface"
-require "byebug/interfaces/remote_interface"
+require_relative "interfaces/local_interface"
+require_relative "interfaces/script_interface"
+require_relative "interfaces/remote_interface"

data/lib/byebug/interfaces/local_interface.rb

@@ -5,7 +5,7 @@ module Byebug
   # Interface class for standard byebug use.
   #
   class LocalInterface < Interface
-    EOF_ALIAS = "continue".freeze
+    EOF_ALIAS = "continue"
 
     def initialize
       super()

data/lib/byebug/interfaces/remote_interface.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/history"
+require_relative "../history"
 
 module Byebug
   #

data/lib/byebug/interfaces/script_interface.rb

@@ -24,7 +24,8 @@ module Byebug
     def readline(*)
       while (result = input.gets)
         output.puts "+ #{result}" if @verbose
-        next if result =~ /^\s*#/
+        next if /^\s*#/.match?(result)
+
         return result.chomp
       end
     end

data/lib/byebug/printers/base.rb

@@ -11,7 +11,7 @@ module Byebug
       class MissedPath < StandardError; end
       class MissedArgument < StandardError; end
 
-      SEPARATOR = ".".freeze
+      SEPARATOR = "."
 
       def type
         self.class.name.split("::").last.downcase
@@ -23,11 +23,12 @@ module Byebug
         result = nil
         contents.each_value do |contents|
           result = parts(path).reduce(contents) do |r, part|
-            r && r.key?(part) ? r[part] : nil
+            r&.key?(part) ? r[part] : nil
           end
           break if result
         end
         raise MissedPath, "Can't find part path '#{path}'" unless result
+
         result
       end
 
@@ -35,9 +36,7 @@ module Byebug
         # they may contain #{} string interpolation
         string.gsub(/\|\w+$/, "").gsub(/([^#]?){([^}]*)}/) do
           key = Regexp.last_match[2].to_s
-          unless args.key?(key.to_sym)
-            raise MissedArgument, "Missed argument #{key} for '#{string}'"
-          end
+          raise MissedArgument, "Missed argument #{key} for '#{string}'" unless args.key?(key.to_sym)
 
           "#{Regexp.last_match[1]}#{args[key.to_sym]}"
         end

data/lib/byebug/printers/plain.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/printers/base"
+require_relative "base"
 
 module Byebug
   module Printers
@@ -22,7 +22,7 @@ module Byebug
         lines.join
       end
 
-      def print_variables(variables, *_)
+      def print_variables(variables, *_unused)
         print_collection("variable.variable", variables) do |(key, value), _|
           value = value.nil? ? "nil" : value.to_s
           if "#{key} = #{value}".size > Setting[:width]

data/lib/byebug/processors/command_processor.rb

@@ -2,8 +2,8 @@
 
 require "forwardable"
 
-require "byebug/helpers/eval"
-require "byebug/errors"
+require_relative "../helpers/eval"
+require_relative "../errors"
 
 module Byebug
   #

data/lib/byebug/processors/control_processor.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/processors/command_processor"
+require_relative "command_processor"
 
 module Byebug
   #

data/lib/byebug/processors/post_mortem_processor.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/processors/command_processor"
+require_relative "command_processor"
 
 module Byebug
   #

data/lib/byebug/processors/script_processor.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/processors/command_processor"
+require_relative "command_processor"
 
 module Byebug
   #

data/lib/byebug/remote.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
 require "socket"
-require "byebug/processors/control_processor"
-require "byebug/remote/server"
-require "byebug/remote/client"
+require_relative "processors/control_processor"
+require_relative "remote/server"
+require_relative "remote/client"
 
 #
 # Remote debugging functionality.

data/lib/byebug/remote/client.rb

@@ -26,10 +26,12 @@ module Byebug
           when /^PROMPT (.*)$/
             input = interface.read_command(Regexp.last_match[1])
             break unless input
+
             socket.puts input
           when /^CONFIRM (.*)$/
             input = interface.readline(Regexp.last_match[1])
             break unless input
+
             socket.puts input
           else
             interface.puts line

data/lib/byebug/runner.rb

@@ -2,13 +2,13 @@
 
 require "optparse"
 require "English"
-require "byebug/core"
-require "byebug/version"
-require "byebug/helpers/bin"
-require "byebug/helpers/parse"
-require "byebug/helpers/string"
-require "byebug/option_setter"
-require "byebug/processors/control_processor"
+require_relative "core"
+require_relative "version"
+require_relative "helpers/bin"
+require_relative "helpers/parse"
+require_relative "helpers/string"
+require_relative "option_setter"
+require_relative "processors/control_processor"
 
 module Byebug
   #

data/lib/byebug/setting.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/string"
+require_relative "helpers/string"
 
 module Byebug
   #
@@ -52,7 +52,7 @@ module Byebug
       end
 
       def find(shortcut)
-        abbr = shortcut =~ /^no/ ? shortcut[2..-1] : shortcut
+        abbr = /^no/.match?(shortcut) ? shortcut[2..-1] : shortcut
         matches = settings.select do |key, value|
           key =~ (value.boolean? ? /#{abbr}/ : /#{shortcut}/)
         end

data/lib/byebug/settings/autoirb.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "byebug/setting"
-require "byebug/commands/irb"
+require_relative "../setting"
+require_relative "../commands/irb"
 
 module Byebug
   #
@@ -18,8 +18,8 @@ module Byebug
       "Invoke IRB on every stop"
     end
 
-    def value=(v)
-      IrbCommand.always_run = v ? 1 : 0
+    def value=(val)
+      IrbCommand.always_run = val ? 1 : 0
     end
 
     def value

data/lib/byebug/settings/autolist.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "byebug/setting"
-require "byebug/commands/list"
+require_relative "../setting"
+require_relative "../commands/list"
 
 module Byebug
   #
@@ -18,8 +18,8 @@ module Byebug
       "Invoke list command on every stop"
     end
 
-    def value=(v)
-      ListCommand.always_run = v ? 1 : 0
+    def value=(val)
+      ListCommand.always_run = val ? 1 : 0
     end
 
     def value

data/lib/byebug/settings/autopry.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "byebug/setting"
-require "byebug/commands/pry"
+require_relative "../setting"
+require_relative "../commands/pry"
 
 module Byebug
   #
@@ -18,8 +18,8 @@ module Byebug
       "Invoke Pry on every stop"
     end
 
-    def value=(v)
-      PryCommand.always_run = v ? 1 : 0
+    def value=(val)
+      PryCommand.always_run = val ? 1 : 0
     end
 
     def value

data/lib/byebug/settings/autosave.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/setting"
+require_relative "../setting"
 
 module Byebug
   #

data/lib/byebug/settings/basename.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/setting"
+require_relative "../setting"
 
 module Byebug
   #