pry 0.12.0 → 0.12.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA256:
-  metadata.gz: d07ef96499f7a86674cffc2a3716d4d922c33046c0134585292915b27af11f70
-  data.tar.gz: 544a3c577b01fc131ee3ab5c6a85dd89594675de8a4019a536a98afd29ee71a8
+SHA1:
+  metadata.gz: cbf5001726c32fca4ad53946241bc627724bac43
+  data.tar.gz: ca594bc15a8429ecf268bd50764a9b7bc0405d21
 SHA512:
-  metadata.gz: 0510a99ee011dd860352f7fdc7c7c5cd721fa44b2b72d7f2c354fc90af53a481ec53e1a6fc29a01fdf81f8130b756977c7d3dd885cc34418416d81f4ae7244e8
-  data.tar.gz: a8622184f84e20196899c1629c4f45733ed57416d2cb104345099fb64842d92eb105de264627f203b047816328e1201d94b6dd876719d3dc82ab49eba5f22e5b
+  metadata.gz: e533b613978de9ede7463d1bdc2cc0175d23a1f3f23fabe63c36154b587f4e04fb872de6265744f4eba77844290cb515d594ec9c9a4f94f03da9485358b6f11a
+  data.tar.gz: 4c179399551b6b22e891b07abdc9d6b1eace0e27c53c607147d1be7522a54c9506c1d8eee35e4d42b6bead1119eb979528865422a6cb5d71b67eb224600cfcd4

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 ### HEAD
 
+### [v0.12.1][v0.12.1] (November 12, 2018)
+
+#### Bug fixes
+
+* Stopped creating a new hash each time `Pry::Prompt#[]` is invoked
+  ([#1855](https://github.com/pry/pry/pull/1855))
+* Fixed `less` pager not working when it's available
+  ([#1861](https://github.com/pry/pry/pull/1861))
+
 ### [v0.12.0][v0.12.0] (November 5, 2018)
 
 #### Major changes
@@ -933,3 +942,4 @@ complete CHANGELOG:
 * release!
 
 [v0.12.0]: https://github.com/pry/pry/releases/tag/v0.12.0
+[v0.12.1]: https://github.com/pry/pry/releases/tag/v0.12.1

data/lib/pry.rb

@@ -2,20 +2,19 @@ require 'pp'
 require 'pry/forwardable'
 require 'pry/input_lock'
 require 'pry/exceptions'
-require 'pry/platform'
 require 'pry/helpers/base_helpers'
 require 'pry/hooks'
 
 class Pry
   # The default hooks - display messages when beginning and ending Pry sessions.
-  DEFAULT_HOOKS = Pry::Hooks.new.add_hook(:before_session, :default) do |out, target, _pry_|
+  DEFAULT_HOOKS = Pry::Hooks.new.add_hook(:before_session, :default) do |_out, _target, _pry_|
     next if _pry_.quiet?
 
     _pry_.run_command("whereami --quiet")
   end
 
   # The default print
-  DEFAULT_PRINT = proc do |output, value, _pry_|
+  DEFAULT_PRINT = proc do |_output, value, _pry_|
     _pry_.pager.open do |pager|
       pager.print _pry_.config.output_prefix
       Pry::ColorPrinter.pp(value, pager, Pry::Terminal.width! - 1)

data/lib/pry/cli.rb

@@ -191,7 +191,7 @@ Pry::CLI.add_options do
     Pry.config.requires << file
   end
 
-  on :I=, "Add a path to the $LOAD_PATH", as: Array, delimiter: ":" do |load_path|
+  on(:I=, "Add a path to the $LOAD_PATH", as: Array, delimiter: ":") do
     load_path.map! do |path|
       /\A\.\// =~ path ? path : File.expand_path(path)
     end
@@ -199,7 +199,7 @@ Pry::CLI.add_options do
     $LOAD_PATH.unshift(*load_path)
   end
 
-  on "gem", "Shorthand for -I./lib -rgemname" do |load_path|
+  on "gem", "Shorthand for -I./lib -rgemname" do |_load_path|
     $LOAD_PATH.unshift("./lib")
     Dir["./lib/*.rb"].each do |file|
       Pry.config.requires << file

data/lib/pry/command.rb

@@ -252,18 +252,6 @@ class Pry
       command_set.to_hash
     end
 
-    #
-    # @deprecated
-    #   Please use black(), white(), etc directly instead (as you would with helper
-    #   functions from BaseHelpers and CommandHelpers)
-    #
-    # @return [Module]
-    #   Returns Pry::Helpers::Text
-    #
-    def text
-      Pry::Helpers::Text
-    end
-
     def void
       VOID_VALUE
     end
@@ -319,7 +307,10 @@ class Pry
       collision_type ||= 'local-variable' if arg_string.match(%r{\A\s*[-+*/%&|^]*=})
 
       if collision_type
-        output.puts "#{text.bold('WARNING:')} Calling Pry command '#{command_match}', which conflicts with a #{collision_type}.\n\n"
+        output.puts(
+          "#{Helpers::Text.bold('WARNING:')} Calling Pry command '#{command_match}', " \
+          "which conflicts with a #{collision_type}.\n\n"
+        )
       end
     rescue Pry::RescuableException
     end
@@ -425,9 +416,13 @@ class Pry
       unless dependencies_met?
         gems_needed = Array(command_options[:requires_gem])
         gems_not_installed = gems_needed.select { |g| !Rubygem.installed?(g) }
-        output.puts "\nThe command '#{command_name}' is #{text.bold("unavailable")} because it requires the following gems to be installed: #{(gems_not_installed.join(", "))}"
-        output.puts "-"
-        output.puts "Type `install-command #{command_name}` to install the required gems and activate this command."
+        output.puts(<<WARN)
+The command #{Helpers::Text.bold(command_name)} is unavailable because it requires the following
+gems to be installed: #{(gems_not_installed.join(", "))}
+
+Type #{Helpers::Text.bold('install-command ' + command_name)} to install the required gems
+and activate this command.
+WARN
         return void
       end
 

data/lib/pry/commands/change_prompt.rb

@@ -28,7 +28,7 @@ class Pry::Command::ChangePrompt < Pry::ClassCommand
       "#{bold(name)}#{red(' (selected)') if _pry_.prompt == prompt[:value]}\n" +
         prompt[:description]
     end
-    output.puts(prompts.join("\n"))
+    output.puts(prompts.join("\n" * 2))
   end
 
   def change_prompt(prompt)

data/lib/pry/commands/ls/jruby_hacks.rb

@@ -19,7 +19,7 @@ module Pry::Command::Ls::JRubyHacks
       m.name.sub(/\A(is|get|set)(?=[A-Z_])/, '').gsub(/[_?=]/, '').downcase
     end
 
-    grouped.flat_map do |key, values|
+    grouped.flat_map do |_key, values|
       values = values.sort_by do |m|
         rubbishness(m.name)
       end

data/lib/pry/commands/ls/local_vars.rb

@@ -19,7 +19,7 @@ class Pry
       private
 
       def format(name_value_pairs)
-        name_value_pairs.sort_by { |name, value|
+        name_value_pairs.sort_by { |_name, value|
           value.to_s.size
         }.reverse.map { |name, value|
           colorized_assignment_style(name, format_value(value))

data/lib/pry/commands/show_info.rb

@@ -155,7 +155,7 @@ class Pry
         owner: "\n#{bold("Owner:")} #{code_object.owner || "N/A"}\n",
         visibility: "#{bold("Visibility:")} #{code_object.visibility}",
         signature: "\n#{bold("Signature:")} #{code_object.signature}"
-      }.merge(header_options) { |key, old, new| (new && old).to_s }
+      }.merge(header_options) { |_key, old, new| (new && old).to_s }
     end
 
     def header_options

data/lib/pry/commands/wtf.rb

@@ -36,7 +36,7 @@ class Pry
       if exception.respond_to? :cause
         cause = exception.cause
         while cause
-          output.puts "#{text.bold('Caused by:')} #{cause.class}: #{cause}\n--"
+          output.puts "#{bold('Caused by:')} #{cause.class}: #{cause}\n--"
           if opts.verbose?
             output.puts with_line_numbers(cause.backtrace)
           else

data/lib/pry/config/behavior.rb

@@ -163,8 +163,8 @@ class Pry
 
       def eager_load!
         default = @default
-        while default
-          default.memoized_methods.each { |method| self[key] = default.public_send(key) } if default.respond_to?(:memoized_methods)
+        while default && default.respond_to?(:memoized_methods)
+          default.memoized_methods.each { self[key] = default.public_send(key) }
           default = @default.default
         end
       end

data/lib/pry/helpers/base_helpers.rb

@@ -1,135 +1,8 @@
 module Pry::Helpers; end
 
-# rubocop:disable Metrics/ModuleLength
 module Pry::Helpers::BaseHelpers
   extend self
 
-  @mac_osx_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.mac_osx?} instead.
-  def mac_osx?
-    unless @mac_osx_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @mac_osx_warn = true
-    end
-    Pry::Helpers::Platform.mac_osx?
-  end
-
-  @linux_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.mac_osx?} instead.
-  def linux?
-    unless @linux_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @linux_warn = true
-    end
-    Pry::Helpers::Platform.linux?
-  end
-
-  @windows_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.windows?} instead.
-  def windows?
-    unless @windows_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @windows_warn = true
-    end
-    Pry::Helpers::Platform.windows?
-  end
-
-  @windows_ansi_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.windows_ansi?} instead.
-  def windows_ansi?
-    unless @windows_ansi_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @windows_ansi_warn = true
-    end
-    Pry::Helpers::Platform.windows_ansi?
-  end
-
-  @jruby_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.jruby?} instead.
-  def jruby?
-    unless @jruby_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @jruby_warn = true
-    end
-    Pry::Helpers::Platform.jruby?
-  end
-
-  @jruby19_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.jruby_19?} instead.
-  def jruby_19?
-    unless @jruby19_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @jruby19_warn = true
-    end
-    Pry::Helpers::Platform.jruby_19?
-  end
-
-  @mri_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.mri?} instead.
-  def mri?
-    unless @mri_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @mri_warn = true
-    end
-    Pry::Helpers::Platform.mri?
-  end
-
-  @mri19_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.mri_19?} instead.
-  def mri_19?
-    unless @mri19_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @mri19_warn = true
-    end
-    Pry::Helpers::Platform.mri_19?
-  end
-
-  @mri2_warn = false
-  # @deprecated Use {Pry::Helpers::Platform.mri_2?} instead.
-  def mri_2?
-    unless @mri2_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method BaseHelpers##{__method__} " \
-        "is deprecated. Use Pry:Helpers::Platform.#{__method__} instead"
-      )
-      @mri2_warn = true
-    end
-    Pry::Helpers::Platform.mri_2?
-  end
-
   def silence_warnings
     old_verbose = $VERBOSE
     $VERBOSE = nil
@@ -197,4 +70,3 @@ module Pry::Helpers::BaseHelpers
     end
   end
 end
-# rubocop:enable Metrics/ModuleLength

data/lib/pry/hooks.rb

@@ -34,7 +34,7 @@ class Pry
     # @return [Pry:Hooks] The receiver.
     # @see #merge
     def merge!(other)
-      @hooks.merge!(other.dup.hooks) do |key, array, other_array|
+      @hooks.merge!(other.dup.hooks) do |_key, array, other_array|
         temp_hash, output = {}, []
 
         (array + other_array).reverse_each do |pair|
@@ -79,7 +79,7 @@ class Pry
       end
 
       # ensure we only have one anonymous hook
-      @hooks[event_name].delete_if { |h, k| h.nil? } if hook_name.nil?
+      @hooks[event_name].delete_if { |h, _k| h.nil? } if hook_name.nil?
 
       if block
         @hooks[event_name] << [hook_name, block]
@@ -95,7 +95,7 @@ class Pry
     # @param [Array] args The arguments to pass to each hook function.
     # @return [Object] The return value of the last executed hook.
     def exec_hook(event_name, *args, &block)
-      @hooks[event_name.to_s].map do |hook_name, callable|
+      @hooks[event_name.to_s].map do |_hook_name, callable|
         begin
           callable.call(*args, &block)
         rescue RescuableException => e
@@ -115,7 +115,7 @@ class Pry
     # @param [Symbol] hook_name The name of the hook
     # @return [#call] a specific hook for a given event.
     def get_hook(event_name, hook_name)
-      hook = @hooks[event_name.to_s].find do |current_hook_name, callable|
+      hook = @hooks[event_name.to_s].find do |current_hook_name, _callable|
         current_hook_name == hook_name
       end
       hook.last if hook

data/lib/pry/indent.rb

@@ -145,7 +145,7 @@ class Pry
       input.lines.each do |line|
         if in_string?
           tokens = tokenize("#{open_delimiters_line}\n#{line}")
-          tokens = tokens.drop_while { |token, type| !(String === token && token.include?("\n")) }
+          tokens = tokens.drop_while { |token, _type| !(String === token && token.include?("\n")) }
           previously_in_string = true
         else
           tokens = tokenize(line)

data/lib/pry/method.rb

@@ -161,15 +161,6 @@ class Pry
         all_from_class(singleton_class_of(obj), include_super)
       end
 
-      #
-      # @deprecated
-      #  please use {all_from_obj} instead.
-      #  the `method_type` argument is ignored.
-      #
-      def all_from_common(obj, _method_type = nil, include_super = true)
-        all_from_obj(obj, include_super)
-      end
-
       # Get every `Class` and `Module`, in order, that will be checked when looking
       # for an instance method to call on this object.
       # @param [Object] obj

data/lib/pry/pager.rb

@@ -3,241 +3,248 @@ require 'pry/terminal'
 # A pager is an `IO`-like object that accepts text and either prints it
 # immediately, prints it one page at a time, or streams it to an external
 # program to print one page at a time.
-class Pry::Pager
-  class StopPaging < StandardError
-  end
-
-  attr_reader :_pry_
-
-  def initialize(_pry_)
-    @_pry_ = _pry_
-  end
-
-  # Send the given text through the best available pager (if `Pry.config.pager` is
-  # enabled). If you want to send text through in chunks as you generate it, use `open`
-  # to get a writable object instead.
-  #
-  # @param [String] text
-  #   Text to run through a pager.
-  #
-  def page(text)
-    open do |pager|
-      pager << text
+class Pry
+  class Pager
+    class StopPaging < StandardError
     end
-  end
 
-  # Yields a pager object (`NullPager`, `SimplePager`, or `SystemPager`).  All
-  # pagers accept output with `#puts`, `#print`, `#write`, and `#<<`.
-  def open
-    pager = best_available
-    yield pager
-  rescue StopPaging
-  ensure
-    pager.close if pager
-  end
+    attr_reader :_pry_
 
-  private
-
-  def enabled?; !!@enabled; end
-
-  def output; @output; end
-
-  # Return an instance of the "best" available pager class -- `SystemPager` if
-  # possible, `SimplePager` if `SystemPager` isn't available, and `NullPager`
-  # if the user has disabled paging. All pagers accept output with `#puts`,
-  # `#print`, `#write`, and `#<<`. You must call `#close` when you're done
-  # writing output to a pager, and you must rescue `Pry::Pager::StopPaging`.
-  # These requirements can be avoided by using `.open` instead.
-  def best_available
-    if !_pry_.config.pager
-      NullPager.new(_pry_.output)
-    elsif !SystemPager.available? || Helpers::Platform.jruby?
-      SimplePager.new(_pry_.output)
-    else
-      SystemPager.new(_pry_.output)
+    def initialize(_pry_)
+      @_pry_ = _pry_
     end
-  end
 
-  # `NullPager` is a "pager" that actually just prints all output as it comes
-  # in. Used when `Pry.config.pager` is false.
-  class NullPager
-    def initialize(out)
-      @out = out
+    # Send the given text through the best available pager (if
+    # `Pry.config.pager` is enabled). If you want to send text through in
+    # chunks as you generate it, use `open` to get a writable object
+    # instead.
+    #
+    # @param [String] text
+    #   Text to run through a pager.
+    #
+    def page(text)
+      open do |pager|
+        pager << text
+      end
     end
 
-    def puts(str)
-      print "#{str.chomp}\n"
+    # Yields a pager object (`NullPager`, `SimplePager`, or `SystemPager`).
+    # All pagers accept output with `#puts`, `#print`, `#write`, and `#<<`.
+    def open
+      pager = best_available
+      yield pager
+    rescue StopPaging
+    ensure
+      pager.close if pager
     end
 
-    def print(str)
-      write str
-    end
-    alias << print
+    private
 
-    def write(str)
-      @out.write str
+    def enabled?; !!@enabled; end
+
+    def output; @output; end
+
+    # Return an instance of the "best" available pager class --
+    # `SystemPager` if possible, `SimplePager` if `SystemPager` isn't
+    # available, and `NullPager` if the user has disabled paging. All
+    # pagers accept output with `#puts`, `#print`, `#write`, and `#<<`. You
+    # must call `#close` when you're done writing output to a pager, and
+    # you must rescue `Pry::Pager::StopPaging`. These requirements can be
+    # avoided by using `.open` instead.
+    def best_available
+      if !_pry_.config.pager
+        NullPager.new(_pry_.output)
+      elsif !SystemPager.available? || Helpers::Platform.jruby?
+        SimplePager.new(_pry_.output)
+      else
+        SystemPager.new(_pry_.output)
+      end
     end
 
-    def close
-    end
+    # `NullPager` is a "pager" that actually just prints all output as it
+    # comes in. Used when `Pry.config.pager` is false.
+    class NullPager
+      def initialize(out)
+        @out = out
+      end
 
-    private
+      def puts(str)
+        print "#{str.chomp}\n"
+      end
 
-    def height
-      @height ||= Pry::Terminal.height!
-    end
+      def print(str)
+        write str
+      end
+      alias << print
 
-    def width
-      @width ||= Pry::Terminal.width!
-    end
-  end
+      def write(str)
+        @out.write str
+      end
 
-  # `SimplePager` is a straightforward pure-Ruby pager. We use it on JRuby and
-  # when we can't find a usable external pager.
-  class SimplePager < NullPager
-    def initialize(*)
-      super
-      @tracker = PageTracker.new(height - 3, width)
-    end
+      def close
+      end
 
-    def write(str)
-      str.lines.each do |line|
-        @out.print line
-        @tracker.record line
+      private
 
-        if @tracker.page?
-          @out.print "\n"
-          @out.print "\e[0m"
-          @out.print "<page break> --- Press enter to continue " \
-                     "( q<enter> to break ) --- <page break>\n"
-          raise StopPaging if Readline.readline("").chomp == "q"
+      def height
+        @height ||= Pry::Terminal.height!
+      end
 
-          @tracker.reset
-        end
+      def width
+        @width ||= Pry::Terminal.width!
       end
     end
-  end
 
-  # `SystemPager` buffers output until we're pretty sure it's at least a page
-  # long, then invokes an external pager and starts streaming output to it. If
-  # `#close` is called before then, it just prints out the buffered content.
-  class SystemPager < NullPager
-    def self.default_pager
-      pager = ENV["PAGER"] || ""
-
-      # Default to less, and make sure less is being passed the correct options
-      if pager.strip.empty? or pager =~ /^less\b/
-        pager = "less -R -F -X"
+    # `SimplePager` is a straightforward pure-Ruby pager. We use it on
+    # JRuby and when we can't find a usable external pager.
+    class SimplePager < NullPager
+      def initialize(*)
+        super
+        @tracker = PageTracker.new(height - 3, width)
       end
 
-      pager
-    end
+      def write(str)
+        str.lines.each do |line|
+          @out.print line
+          @tracker.record line
 
-    @system_pager = nil
+          if @tracker.page?
+            @out.print "\n"
+            @out.print "\e[0m"
+            @out.print "<page break> --- Press enter to continue " \
+                       "( q<enter> to break ) --- <page break>\n"
+            raise StopPaging if Readline.readline("").chomp == "q"
 
-    def self.available?
-      if @system_pager.nil?
-        @system_pager = begin
-          pager_executable = default_pager.split(' ').first
-          if Helpers::Platform.windows? || Helpers::Platform.windows_ansi?
-            `where /Q #{pager_executable}`
-          else
-            `which #{pager_executable}`
+            @tracker.reset
           end
-          $?.success?
-        rescue
-          false
         end
-      else
-        @system_pager
       end
     end
 
-    def initialize(*)
-      super
-      @tracker = PageTracker.new(height, width)
-      @buffer  = ""
-      @pager   = nil
-    end
+    # `SystemPager` buffers output until we're pretty sure it's at least a
+    # page long, then invokes an external pager and starts streaming output
+    # to it. If `#close` is called before then, it just prints out the
+    # buffered content.
+    class SystemPager < NullPager
+      def self.default_pager
+        pager = ENV["PAGER"] || ""
 
-    def write(str)
-      if invoked_pager?
-        write_to_pager str
-      else
-        @tracker.record str
-        @buffer << str
+        # Default to less, and make sure less is being passed the correct
+        # options
+        if pager.strip.empty? or pager =~ /^less\b/
+          pager = "less -R -F -X"
+        end
+
+        pager
+      end
 
-        if @tracker.page?
-          write_to_pager @buffer
+      @system_pager = nil
+
+      def self.available?
+        if @system_pager.nil?
+          @system_pager = begin
+            pager_executable = default_pager.split(' ').first
+            if Helpers::Platform.windows? || Helpers::Platform.windows_ansi?
+              `where /Q #{pager_executable}`
+            else
+              `which #{pager_executable}`
+            end
+            $?.success?
+          rescue
+            false
+          end
+        else
+          @system_pager
         end
       end
-    rescue Errno::EPIPE
-      raise StopPaging
-    end
 
-    def close
-      if invoked_pager?
-        pager.close
-      else
-        @out.puts @buffer
+      def initialize(*)
+        super
+        @tracker = PageTracker.new(height, width)
+        @buffer  = ""
+        @pager   = nil
       end
-    end
 
-    private
+      def write(str)
+        if invoked_pager?
+          write_to_pager str
+        else
+          @tracker.record str
+          @buffer << str
 
-    def write_to_pager(text)
-      pager.write @out.decolorize_maybe(text)
-    end
+          if @tracker.page?
+            write_to_pager @buffer
+          end
+        end
+      rescue Errno::EPIPE
+        raise StopPaging
+      end
 
-    def invoked_pager?
-      @pager
-    end
+      def close
+        if invoked_pager?
+          pager.close
+        else
+          @out.puts @buffer
+        end
+      end
 
-    def pager
-      @pager ||= IO.popen(self.class.default_pager, 'w')
-    end
-  end
+      private
 
-  # `PageTracker` tracks output to determine whether it's likely to take up a
-  # whole page. This doesn't need to be super precise, but we can use it for
-  # `SimplePager` and to avoid invoking the system pager unnecessarily.
-  #
-  # One simplifying assumption is that we don't need `#page?` to return `true`
-  # on the basis of an incomplete line. Long lines should be counted as
-  # multiple lines, but we don't have to transition from `false` to `true`
-  # until we see a newline.
-  class PageTracker
-    def initialize(rows, cols)
-      @rows, @cols = rows, cols
-      reset
+      def write_to_pager(text)
+        pager.write @out.decolorize_maybe(text)
+      end
+
+      def invoked_pager?
+        @pager
+      end
+
+      def pager
+        @pager ||= IO.popen(self.class.default_pager, 'w')
+      end
     end
 
-    def record(str)
-      str.lines.each do |line|
-        if line.end_with? "\n"
-          @row += ((@col + line_length(line) - 1) / @cols) + 1
-          @col  = 0
-        else
-          @col += line_length(line)
+    # `PageTracker` tracks output to determine whether it's likely to take
+    # up a whole page. This doesn't need to be super precise, but we can
+    # use it for `SimplePager` and to avoid invoking the system pager
+    # unnecessarily.
+    #
+    # One simplifying assumption is that we don't need `#page?` to return
+    # `true` on the basis of an incomplete line. Long lines should be
+    # counted as multiple lines, but we don't have to transition from
+    # `false` to `true` until we see a newline.
+    class PageTracker
+      def initialize(rows, cols)
+        @rows, @cols = rows, cols
+        reset
+      end
+
+      def record(str)
+        str.lines.each do |line|
+          if line.end_with? "\n"
+            @row += ((@col + line_length(line) - 1) / @cols) + 1
+            @col  = 0
+          else
+            @col += line_length(line)
+          end
         end
       end
-    end
 
-    def page?
-      @row >= @rows
-    end
+      def page?
+        @row >= @rows
+      end
 
-    def reset
-      @row = 0
-      @col = 0
-    end
+      def reset
+        @row = 0
+        @col = 0
+      end
 
-    private
+      private
 
-    # Approximation of the printable length of a given line, without the
-    # newline and without ANSI color codes.
-    def line_length(line)
-      line.chomp.gsub(/\e\[[\d;]*m/, '').length
+      # Approximation of the printable length of a given line, without the
+      # newline and without ANSI color codes.
+      def line_length(line)
+        line.chomp.gsub(/\e\[[\d;]*m/, '').length
+      end
     end
   end
 end

data/lib/pry/prompt.rb

@@ -27,10 +27,6 @@ class Pry
     #   1-line #inspect output suitable for prompt
     SAFE_CONTEXTS = [String, Numeric, Symbol, nil, true, false].freeze
 
-    # @deprecated Use {Pry::Prompt.add} instead.
-    MAP = {}
-    deprecate_constant(:MAP) if respond_to?(:deprecate_constant)
-
     # A Hash that holds all prompts. The keys of the Hash are prompt
     # names, the values are Hash instances of the format {:description, :value}.
     @prompts = {}
@@ -45,7 +41,7 @@ class Pry
       # @return [Hash{Symbol=>Object}]
       # @since v0.12.0
       def [](prompt_name)
-        all[prompt_name.to_s]
+        @prompts[prompt_name.to_s]
       end
 
       # @return [Hash{Symbol=>Hash}] the duplicate of the internal prompts hash
@@ -94,11 +90,10 @@ class Pry
       end
     end
 
-    add(:default, <<DESC) do |context, nesting, _pry_, sep|
-The default Pry prompt. Includes information about the current expression
-number, evaluation context, and nesting level, plus a reminder that you're
-using Pry.
-DESC
+    add 'default',
+        "The default Pry prompt. Includes information about the current expression \n" \
+        "number, evaluation context, and nesting level, plus a reminder that you're \n" \
+        'using Pry.' do |context, nesting, _pry_, sep|
       format(
         "[%<in_count>s] %<name>s(%<context>s)%<nesting>s%<separator>s ",
         in_count: _pry_.input_ring.count,
@@ -109,14 +104,20 @@ DESC
       )
     end
 
-    add(:simple, "A simple `>>`.\n", ['>> ', ' | ']) do |_, _, _, sep|
+    add(
+      'simple',
+      "A simple `>>`.",
+      ['>> ', ' | ']
+    ) do |_, _, _, sep|
       sep
     end
 
-    add(:nav, <<DESC, %w[> *]) do |context, nesting, _pry_, sep|
-A prompt that displays the binding stack as a path and includes information
-about #{Helpers::Text.bold('_in_')} and #{Helpers::Text.bold('_out_')}.
-DESC
+    add(
+      'nav',
+      "A prompt that displays the binding stack as a path and includes information \n" \
+      "about #{Helpers::Text.bold('_in_')} and #{Helpers::Text.bold('_out_')}.",
+      %w[> *]
+    ) do |_context, _nesting, _pry_, sep|
       tree = _pry_.binding_stack.map { |b| Pry.view_clip(b.eval('self')) }
       format(
         "[%<in_count>s] (%<name>s) %<tree>s: %<stack_size>s%<separator>s ",
@@ -128,9 +129,11 @@ DESC
       )
     end
 
-    add(:shell, <<DESC, %w[$ *]) do |context, nesting, _pry_, sep|
-A prompt that displays `$PWD` as you change it.
-DESC
+    add(
+      'shell',
+      'A prompt that displays `$PWD` as you change it.',
+      %w[$ *]
+    ) do |context, _nesting, _pry_, sep|
       format(
         "%<name>s %<context>s:%<pwd>s %<separator>s ",
         name: prompt_name(_pry_.config.prompt_name),
@@ -140,6 +143,10 @@ DESC
       )
     end
 
-    add(:none, 'Wave goodbye to the Pry prompt.', Array.new(2)) { '' }
+    add(
+      'none',
+      'Wave goodbye to the Pry prompt.',
+      Array.new(2)
+    ) { '' }
   end
 end

data/lib/pry/pry_instance.rb

@@ -87,36 +87,6 @@ class Pry
     exec_hook(:when_started, target, options, self)
   end
 
-  @input_array_warn = false
-  # @deprecated Use {#input_ring} instead.
-  def input_array
-    unless @input_array_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method #{self.class}##{__method__} " \
-        "is deprecated. Use #{self.class}#input_ring instead"
-      )
-      @input_array_warn = true
-    end
-
-    @input_ring
-  end
-
-  @output_array_warn = false
-  # @deprecated Use {#output_ring} instead.
-  def output_array
-    unless @output_array_warn
-      loc = caller_locations(1..1).first
-      warn(
-        "#{loc.path}:#{loc.lineno}: warning: method #{self.class}##{__method__} " \
-        "is deprecated. Use #{self.class}#output_ring instead"
-      )
-      @output_array_warn = true
-    end
-
-    @output_ring
-  end
-
   # This is the prompt at the top of the prompt stack.
   # @return [Array<Proc>] the current prompt
   def prompt

data/lib/pry/repl_file_loader.rb

@@ -34,7 +34,7 @@ class Pry
     # @param [Pry] _pry_ the Pry instance to make non-interactive.
     def non_interactive_mode(_pry_, content)
       _pry_.print = proc {}
-      _pry_.exception_handler = proc do |o, e, _p_|
+      _pry_.exception_handler = proc do |o, _e, _p_|
         _p_.run_command "cat --ex"
         o.puts "...exception encountered, going interactive!"
         interactive_mode(_pry_)

data/lib/pry/slop.rb

@@ -224,7 +224,7 @@ class Pry::Slop
       autocreate(items, index) if config[:autocreate]
       process_item(items, index, &block) unless @trash.include?(index)
     end
-    items.reject!.with_index { |item, index| @trash.include?(index) }
+    items.reject!.with_index { |_item, index| @trash.include?(index) }
 
     missing_options = options.select { |opt| opt.required? && opt.count < 1 }
     if missing_options.any?

data/lib/pry/slop/option.rb

@@ -45,7 +45,7 @@ class Pry::Slop
         integer: proc { |v| value_to_integer(v) },
         float: proc { |v| value_to_float(v) },
         range: proc { |v| value_to_range(v) },
-        count: proc { |v| @count }
+        count: proc { @count }
       }
 
       if long && long.size > @slop.config[:longest_flag]

data/lib/pry/version.rb

@@ -1,3 +1,3 @@
 class Pry
-  VERSION = '0.12.0'
+  VERSION = '0.12.1'.freeze
 end

metadata

@@ -1,41 +1,41 @@
 --- !ruby/object:Gem::Specification
 name: pry
 version: !ruby/object:Gem::Version
-  version: 0.12.0
+  version: 0.12.1
 platform: ruby
 authors:
 - John Mair (banisterfiend)
 - Conrad Irwin
 - Ryan Fitzgerald
 - Kyrylo Silin
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-11-04 00:00:00.000000000 Z
+date: 2018-11-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
+  name: coderay
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.1.0
-  name: coderay
-  prerelease: false
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 1.1.0
 - !ruby/object:Gem::Dependency
+  name: method_source
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 0.9.0
-  name: method_source
-  prerelease: false
   type: :runtime
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
@@ -173,7 +173,6 @@ files:
 - lib/pry/object_path.rb
 - lib/pry/output.rb
 - lib/pry/pager.rb
-- lib/pry/platform.rb
 - lib/pry/plugins.rb
 - lib/pry/prompt.rb
 - lib/pry/pry_class.rb
@@ -200,7 +199,7 @@ homepage: http://pryrepl.org
 licenses:
 - MIT
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -215,9 +214,9 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project:
-rubygems_version: 2.7.6
-signing_key:
+rubyforge_project: 
+rubygems_version: 2.6.13
+signing_key: 
 specification_version: 4
 summary: An IRB alternative and runtime developer console
 test_files: []

data/lib/pry/platform.rb

@@ -1,91 +0,0 @@
-class Pry
-  module Platform
-    extend self
-
-    #
-    # @return [Boolean]
-    #  Returns true if Pry is running on Mac OSX.
-    #
-    # @note
-    #   Queries RbConfig::CONFIG['host_os'] with a best guess.
-    #
-    def mac_osx?
-      !!(RbConfig::CONFIG['host_os'] =~ /\Adarwin/i)
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true if Pry is running on Linux.
-    #
-    # @note
-    #   Queries RbConfig::CONFIG['host_os'] with a best guess.
-    #
-    def linux?
-      !!(RbConfig::CONFIG['host_os'] =~ /linux/i)
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true if Pry is running on Windows.
-    #
-    # @note
-    #   Queries RbConfig::CONFIG['host_os'] with a best guess.
-    #
-    def windows?
-      !!(RbConfig::CONFIG['host_os'] =~ /mswin|mingw/)
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true when Pry is running on Windows with ANSI support.
-    #
-    def windows_ansi?
-      return false if not windows?
-
-      !!(defined?(Win32::Console) or ENV['ANSICON'] or mri_2?)
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true when Pry is being run from JRuby.
-    #
-    def jruby?
-      RbConfig::CONFIG['ruby_install_name'] == 'jruby'
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true when Pry is being run from JRuby in 1.9 mode.
-    #
-    def jruby_19?
-      jruby? and RbConfig::CONFIG['ruby_version'] == '1.9'
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true when Pry is being run from MRI (CRuby).
-    #
-    def mri?
-      RbConfig::CONFIG['ruby_install_name'] == 'ruby'
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true when Pry is being run from MRI v1.9+ (CRuby).
-    #
-    def mri_19?
-      !!(mri? and RUBY_VERSION =~ /\A1\.9/)
-    end
-
-    #
-    # @return [Boolean]
-    #   Returns true when Pry is being run from MRI v2+ (CRuby).
-    #
-    def mri_2?
-      !!(mri? and RUBY_VERSION =~ /\A2/)
-    end
-  end
-
-  # Not supported on MRI 2.2 and lower.
-  deprecate_constant(:Platform) if respond_to?(:deprecate_constant)
-end