byebug 10.0.2 → 11.1.2

data/lib/byebug/commands/thread/current.rb

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

data/lib/byebug/commands/thread/list.rb

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

data/lib/byebug/commands/thread/resume.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/thread"
+require_relative "../../helpers/thread"
 
 module Byebug
   #
@@ -35,9 +35,7 @@ module Byebug
         context, err = context_from_thread(@match[1])
         return errmsg(err) if err
 
-        unless context.suspended?
-          return errmsg(pr("thread.errors.already_running"))
-        end
+        return errmsg(pr("thread.errors.already_running")) unless context.suspended?
 
         context.resume
         display_context(context)

data/lib/byebug/commands/thread/stop.rb

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

data/lib/byebug/commands/thread/switch.rb

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

data/lib/byebug/commands/tracevar.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/command"
+require_relative "../command"
 
 module Byebug
   #
@@ -32,10 +32,7 @@ module Byebug
     def execute
       var = @match[1]
       return errmsg(pr("trace.errors.needs_global_variable")) unless var
-
-      unless global_variables.include?(:"#{var}")
-        return errmsg(pr("trace.errors.var_is_not_global", name: var))
-      end
+      return errmsg(pr("trace.errors.var_is_not_global", name: var)) unless global_variables.include?(:"#{var}")
 
       stop = @match[2] && @match[2] !~ /nostop/
 

data/lib/byebug/commands/undisplay.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require "byebug/command"
-require "byebug/helpers/parse"
+require_relative "../command"
+require_relative "../helpers/parse"
 
 module Byebug
   #
@@ -37,11 +37,10 @@ module Byebug
         pos, err = get_int(@match[1], "Undisplay", 1, Byebug.displays.size)
         return errmsg(err) unless err.nil?
 
-        unless Byebug.displays[pos - 1]
-          return errmsg(pr("display.errors.undefined", expr: pos))
-        end
+        last_display = Byebug.displays[pos - 1]
+        return errmsg(pr("display.errors.undefined", expr: pos)) unless last_display
 
-        Byebug.displays[pos - 1][0] = nil
+        last_display[0] = nil
       else
         return unless confirm(pr("display.confirmations.clear_all"))
 

data/lib/byebug/commands/untracevar.rb

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

data/lib/byebug/commands/up.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
 require "pathname"
-require "byebug/command"
-require "byebug/helpers/frame"
-require "byebug/helpers/parse"
+require_relative "../command"
+require_relative "../helpers/frame"
+require_relative "../helpers/parse"
 
 module Byebug
   #

data/lib/byebug/commands/var.rb

@@ -1,13 +1,13 @@
 # frozen_string_literal: true
 
-require "byebug/subcommands"
-
-require "byebug/commands/var/all"
-require "byebug/commands/var/args"
-require "byebug/commands/var/const"
-require "byebug/commands/var/instance"
-require "byebug/commands/var/local"
-require "byebug/commands/var/global"
+require_relative "../subcommands"
+
+require_relative "var/all"
+require_relative "var/args"
+require_relative "var/const"
+require_relative "var/instance"
+require_relative "var/local"
+require_relative "var/global"
 
 module Byebug
   #

data/lib/byebug/commands/var/all.rb

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

data/lib/byebug/commands/var/args.rb

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

data/lib/byebug/commands/var/const.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/eval"
+require_relative "../../helpers/eval"
 
 module Byebug
   #
@@ -33,9 +33,7 @@ module Byebug
 
       def execute
         obj = warning_eval(str_obj)
-        unless obj.is_a?(Module)
-          return errmsg(pr("variable.errors.not_module", object: str_obj))
-        end
+        return errmsg(pr("variable.errors.not_module", object: str_obj)) unless obj.is_a?(Module)
 
         constants = warning_eval("#{str_obj}.constants")
         puts prv(constants.sort.map { |c| [c, obj.const_get(c)] }, "constant")

data/lib/byebug/commands/var/instance.rb

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

data/lib/byebug/commands/var/local.rb

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

data/lib/byebug/commands/where.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 require "pathname"
-require "byebug/command"
-require "byebug/helpers/frame"
+require_relative "../command"
+require_relative "../helpers/frame"
 
 module Byebug
   #

data/lib/byebug/context.rb

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require "byebug/frame"
-require "byebug/helpers/path"
-require "byebug/helpers/file"
-require "byebug/processors/command_processor"
+require_relative "frame"
+require_relative "helpers/path"
+require_relative "helpers/file"
+require_relative "processors/command_processor"
 
 module Byebug
   #

data/lib/byebug/core.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-require "byebug/helpers/reflection"
+require_relative "helpers/reflection"
 require "byebug/byebug"
-require "byebug/context"
-require "byebug/breakpoint"
-require "byebug/interface"
-require "byebug/processors/script_processor"
-require "byebug/processors/post_mortem_processor"
-require "byebug/commands"
-require "byebug/remote"
-require "byebug/printers/plain"
+require_relative "context"
+require_relative "breakpoint"
+require_relative "interface"
+require_relative "processors/script_processor"
+require_relative "processors/post_mortem_processor"
+require_relative "commands"
+require_relative "remote"
+require_relative "printers/plain"
 
 #
 # Main debugger's container module. Everything is defined under this module
@@ -36,6 +36,7 @@ module Byebug
   #
   # * :attached => Attached to a running program through the `byebug` method.
   # * :standalone => Started through `byebug` script.
+  # * :off => Ignoring any `byebug` method calls.
   #
   attr_accessor :mode
 

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/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

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

@@ -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()
@@ -21,7 +21,7 @@ module Byebug
     # @param prompt Prompt to be displayed.
     #
     def readline(prompt)
-      with_repl_like_sigint { Readline.readline(prompt) || EOF_ALIAS }
+      with_repl_like_sigint { without_readline_completion { Readline.readline(prompt) || EOF_ALIAS } }
     end
 
     #
@@ -40,5 +40,24 @@ module Byebug
     ensure
       trap("INT", orig_handler)
     end
+
+    #
+    # Disable any Readline completion procs.
+    #
+    # Other gems, for example, IRB could've installed completion procs that are
+    # dependent on them being loaded. Disable those while byebug is the REPL
+    # making use of Readline.
+    #
+    def without_readline_completion
+      orig_completion = Readline.completion_proc
+      return yield unless orig_completion
+
+      begin
+        Readline.completion_proc = nil
+        yield
+      ensure
+        Readline.completion_proc = orig_completion
+      end
+    end
   end
 end

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