pygments.rb 1.1.0 → 2.0.0.rc2

data/bench.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require File.join(File.dirname(__FILE__), '/lib/pygments.rb')
 require 'benchmark'
 
@@ -8,15 +10,14 @@ num = ARGV[0] ? ARGV[0].to_i : 10
 # we can also repeat the code itself
 repeats = ARGV[1] ? ARGV[1].to_i : 1
 
-code = File.open('test/test_data.py').read.to_s * repeats
+code = File.open('test/test_pygments.rb').read.to_s * repeats
 
 puts "Benchmarking....\n"
-puts "Size: " + code.bytesize.to_s + " bytes\n"
-puts "Iterations: " + num.to_s + "\n"
+puts 'Size: ' + code.bytesize.to_s + " bytes\n"
+puts 'Iterations: ' + num.to_s + "\n"
 
 Benchmark.bm(40) do |x|
-  x.report("pygments popen                             ")  { for i in 1..num; Pygments.highlight(code, :lexer => 'python'); end }
-  x.report("pygments popen (process already started)   ")  { for i in 1..num; Pygments.highlight(code, :lexer => 'python'); end }
-  x.report("pygments popen (process already started 2) ")  { for i in 1..num; Pygments.highlight(code, :lexer => 'python'); end }
+  x.report('pygments popen                             ')  { (1..num).each { |_i|; Pygments.highlight(code, lexer: 'python'); } }
+  x.report('pygments popen (process already started)   ')  { (1..num).each { |_i|; Pygments.highlight(code, lexer: 'python'); } }
+  x.report('pygments popen (process already started 2) ')  { (1..num).each { |_i|; Pygments.highlight(code, lexer: 'python'); } }
 end
-

data/cache-lexers.rb

@@ -1,8 +1,9 @@
+# frozen_string_literal: true
+
 require File.join(File.dirname(__FILE__), '/lib/pygments.rb')
 
 # Simple marshalling
 serialized_lexers = Marshal.dump(Pygments.lexers!)
 
 # Write to a file
-File.open("lexers", 'wb') { |file| file.write(serialized_lexers) }
-
+File.open('lexers', 'wb') { |file| file.write(serialized_lexers) }

data/lib/pygments.rb

@@ -1,8 +1,9 @@
+# frozen_string_literal: true
+
 require File.join(File.dirname(__FILE__), 'pygments/popen')
 require 'forwardable'
 
 module Pygments
-
   autoload :Lexer, 'pygments/lexer'
 
   class << self
@@ -14,20 +15,14 @@ module Pygments
     end
 
     def_delegators :engine,
-      # public
-      :formatters,
-      :lexers,
-      :lexers!,
-      :filters,
-      :styles,
-      :css,
-      :lexer_name_for,
-      :highlight,
-      # not public but documented
-      :start,
-      # only for testing
-      :size_check,
-      :get_fixed_bits_from_header,
-      :add_ids
+                   :formatters,
+                   :lexers,
+                   :lexers!,
+                   :filters,
+                   :styles,
+                   :css,
+                   :lexer_name_for,
+                   :highlight,
+                   :start
   end
 end

data/lib/pygments/lexer.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Pygments
   class Lexer < Struct.new(:name, :aliases, :filenames, :mimetypes)
     @lexers          = []
@@ -32,13 +34,13 @@ module Pygments
           m[1].scan(/./).each do |s|
             extnames << extname.sub(m[0], s)
           end
-        elsif extname != ""
+        elsif extname != ''
           extnames << extname
         end
 
         extnames.each do |the_extname|
           @extname_index[the_extname] = lexer
-          @index[the_extname.downcase.sub(/^\./, "")] ||= lexer
+          @index[the_extname.downcase.sub(/^\./, '')] ||= lexer
         end
       end
 
@@ -140,8 +142,8 @@ module Pygments
       Pygments.highlight(text, options)
     end
 
-    alias_method :==, :equal?
-    alias_method :eql?, :equal?
+    alias == equal?
+    alias eql? equal?
   end
 
   lexers.values.each { |h| Lexer.create(h) }

data/lib/pygments/mentos.py

@@ -1,6 +1,7 @@
-#!/usr/bin/env python
+#!/usr/bin/env python3
 # -*- coding: utf-8 -*-
 
+import struct
 import sys, re, os, signal
 import traceback
 if 'PYGMENTS_PATH' in os.environ:
@@ -9,15 +10,11 @@ if 'PYGMENTS_PATH' in os.environ:
 dirname = os.path.dirname
 
 base_dir = dirname(dirname(dirname(os.path.abspath(__file__))))
-sys.path.append(base_dir + "/vendor")
 sys.path.append(base_dir + "/vendor/pygments-main")
-sys.path.append(base_dir + "/vendor/simplejson")
 
 import pygments
 from pygments import lexers, formatters, styles, filters
 
-from threading import Lock
-
 try:
     import json
 except ImportError:
@@ -27,22 +24,16 @@ def _convert_keys(dictionary):
     if not isinstance(dictionary, dict):
         return dictionary
     return dict((str(k), _convert_keys(v))
-        for k, v in dictionary.items())
+        for k, v in list(dictionary.items()))
 
 def _write_error(error):
     res = {"error": error}
-    out_header = json.dumps(res).encode('utf-8')
-    bits = _get_fixed_bits_from_header(out_header)
-    sys.stdout.write(bits + "\n")
-    sys.stdout.flush()
-    sys.stdout.write(out_header + "\n")
+    out_header_bytes = json.dumps(res).encode('utf-8')
+    sys.stdout.buffer.write(struct.pack('!i', len(out_header_bytes)))
+    sys.stdout.buffer.write(out_header_bytes)
     sys.stdout.flush()
     return
 
-def _get_fixed_bits_from_header(out_header):
-    size = len(out_header)
-    return "".join(map(lambda y:str((size>>y)&1), range(32-1, -1, -1)))
-
 def _signal_handler(signal, frame):
     """
     Handle the signal given in the first argument, exiting gracefully
@@ -163,12 +154,9 @@ class Mentos(object):
                 res = json.dumps(res)
 
             elif method == 'highlight':
-                try:
-                    text = text.decode('utf-8')
-                except UnicodeDecodeError:
-                    # The text may already be encoded
-                    text = text
                 res = self.highlight_text(text, lexer, formatter_name, args, _convert_keys(opts))
+                if type(res) is bytes:
+                    res = res.decode('utf-8')
 
             elif method == 'css':
                 kwargs = _convert_keys(kwargs)
@@ -193,29 +181,19 @@ class Mentos(object):
 
 
     def _send_data(self, res, method):
-
         # Base header. We'll build on this, adding keys as necessary.
         base_header = {"method": method}
 
-        res_bytes = len(res) + 1
-        base_header["bytes"] = res_bytes
-
-        out_header = json.dumps(base_header).encode('utf-8')
+        res_bytes = res.encode("utf-8")
+        bytes = len(res_bytes)
+        base_header["bytes"] = bytes
 
-        # Following the protocol, send over a fixed size represenation of the
-        # size of the JSON header
-        bits = _get_fixed_bits_from_header(out_header)
+        out_header_bytes = json.dumps(base_header).encode('utf-8')
 
         # Send it to Rubyland
-        sys.stdout.write(bits + "\n")
-        sys.stdout.flush()
-
-        # Send the header.
-        sys.stdout.write(out_header + "\n")
-        sys.stdout.flush()
-
-        # Finally, send the result
-        sys.stdout.write(res + "\n")
+        sys.stdout.buffer.write(struct.pack('!i', len(out_header_bytes)))
+        sys.stdout.buffer.write(out_header_bytes)
+        sys.stdout.buffer.write(res_bytes)
         sys.stdout.flush()
 
 
@@ -258,27 +236,16 @@ class Mentos(object):
         The header is of form:
         { "method": "highlight", "args": [], "kwargs": {"arg1": "v"}, "bytes": 128, "fd": "8"}
         """
-        lock = Lock()
 
         while True:
-            # The loop begins by reading off a simple 32-arity string
-            # representing an integer of 32 bits. This is the length of
-            # our JSON header.
-            size = sys.stdin.read(32)
+            header_size_bytes = sys.stdin.buffer.read(4)
+            if not header_size_bytes:
+                break
 
-            lock.acquire()
+            header_size = struct.unpack('!i', header_size_bytes)[0]
 
             try:
-                # Read from stdin the amount of bytes we were told to expect.
-                header_bytes = int(size, 2)
-
-                # Sanity check the size
-                size_regex = re.compile('[0-1]{32}')
-                if not size_regex.match(size):
-                    _write_error("Size received is not valid.")
-
-                line = sys.stdin.read(header_bytes)
-
+                line = sys.stdin.buffer.read(header_size).decode('utf-8')
                 header = json.loads(line)
 
                 method, args, kwargs, lexer = self._parse_header(header)
@@ -291,8 +258,8 @@ class Mentos(object):
                 if kwargs:
                     _bytes = kwargs.get("bytes", 0)
 
-                # Read up to the given number bytes (possibly 0)
-                text = sys.stdin.read(_bytes)
+                # Read up to the given number of *bytes* (not chars) (possibly 0)
+                text = sys.stdin.buffer.read(_bytes).decode('utf-8')
 
                 # Sanity check the return.
                 if _bytes:
@@ -312,9 +279,6 @@ class Mentos(object):
                 tb = traceback.format_exc()
                 _write_error(tb)
 
-            finally:
-                lock.release()
-
 def main():
 
     # Signal handlers to trap signals.
@@ -325,11 +289,7 @@ def main():
 
     mentos = Mentos()
 
-    if sys.platform == "win32":
-        # disable CRLF
-        import msvcrt
-        msvcrt.setmode(sys.stdout.fileno(), os.O_BINARY)
-    else:
+    if sys.platform != "win32":
         # close fd's inherited from the ruby parent
         import resource
         maxfd = resource.getrlimit(resource.RLIMIT_NOFILE)[1]
@@ -346,6 +306,3 @@ def main():
 
 if __name__ == "__main__":
     main()
-
-
-

data/lib/pygments/popen.rb

@@ -1,7 +1,7 @@
-# coding: utf-8
+# frozen_string_literal: true
+
+require 'json'
 require 'open3'
-require 'multi_json'
-require 'timeout'
 require 'logger'
 require 'time'
 
@@ -13,40 +13,38 @@ end
 # Python process.
 module Pygments
   class Popen
-
-    def popen4(cmd)
-      stdin, stdout, stderr, wait_thr = Open3.popen3(cmd)
-      [wait_thr[:pid], stdin, stdout, stderr]
+    def popen4(argv)
+      stdin, stdout, stderr, wait_thr = Open3.popen3(*argv)
+      while (pid = wait_thr.pid).nil? && wait_thr.alive?
+        # For unknown reasons, wait_thr.pid is not immediately available on JRuby
+      end
+      [pid, stdin, stdout, stderr]
     end
 
     # Get things started by opening a pipe to mentos (the freshmaker), a
     # Python process that talks to the Pygments library. We'll talk back and
     # forth across this pipe.
-    def start(pygments_path = File.expand_path('../../../vendor/pygments-main/', __FILE__))
+    def start(pygments_path = File.expand_path('../../vendor/pygments-main', __dir__))
       begin
-        @log = Logger.new(ENV['MENTOS_LOG'] ||= File::NULL)
+        @log = Logger.new(ENV['MENTOS_LOG'] || File::NULL)
         @log.level = Logger::INFO
-        @log.datetime_format = "%Y-%m-%d %H:%M "
-      rescue
-        @log = Logger.new(File::NULL)
+        @log.datetime_format = '%Y-%m-%d %H:%M '
       end
 
       ENV['PYGMENTS_PATH'] = pygments_path
 
       # Make sure we kill off the child when we're done
-      at_exit { stop "Exiting" }
+      at_exit { stop 'Exiting' }
 
       # A pipe to the mentos python process. #popen4 gives us
       # the pid and three IO objects to write and read.
-      script = "#{python_binary} #{File.expand_path('../mentos.py', __FILE__)}"
-      @pid, @in, @out, @err = popen4(script)
+      argv = [*python_binary, File.expand_path('mentos.py', __dir__)]
+      @pid, @in, @out, @err = popen4(argv)
+      @in.binmode
+      @out.binmode
       @log.info "Starting pid #{@pid} with fd #{@out.to_i} and python #{python_binary}."
     end
 
-    def windows?
-      RUBY_PLATFORM =~ /mswin|mingw/
-    end
-
     def python_binary
       @python_binary ||= find_python_binary
     end
@@ -56,14 +54,13 @@ module Pygments
     end
 
     # Detect a suitable Python binary to use.
-    # Or return $PYGMENTS_RB_PYTHON if it's exists.
     def find_python_binary
-      if ENV['PYGMENTS_RB_PYTHON']
-        return which(ENV['PYGMENTS_RB_PYTHON'])
-      elsif windows? && which('py')
-        return 'py -2'
+      if Gem.win_platform?
+        return %w[py python3 python].first { |py| !which(py).nil? }
       end
-      return which('python2') || which('python')
+
+      # On non-Windows platforms, we simply rely on shebang
+      []
     end
 
     # Cross platform which command
@@ -71,12 +68,12 @@ module Pygments
     def which(command)
       exts = ENV['PATHEXT'] ? ENV['PATHEXT'].split(';') : ['']
       ENV['PATH'].split(File::PATH_SEPARATOR).each do |dir|
-        exts.each { |ext|
+        exts.each do |ext|
           path = File.join(dir, "#{command}#{ext}")
           return path if File.executable?(path) && !File.directory?(path)
-        }
+        end
       end
-      return nil
+      nil
     end
 
     # Stop the child process by issuing a kill -9.
@@ -92,14 +89,14 @@ module Pygments
     # the signal isn't sent); but we have permissions, and
     # we're not doing anything invalid here.
     def stop(reason)
-      if @pid
+      unless @pid.nil?
+        @log.info "Killing pid: #{@pid}. Reason: #{reason}"
         begin
           Process.kill('KILL', @pid)
           Process.waitpid(@pid)
         rescue Errno::ESRCH, Errno::ECHILD
         end
       end
-      @log.info "Killing pid: #{@pid.to_s}. Reason: #{reason}"
       @pid = nil
     end
 
@@ -114,30 +111,29 @@ module Pygments
     # Returns true if the child is alive.
     def alive?
       return true if defined?(@pid) && @pid && Process.kill(0, @pid)
+
       false
     rescue Errno::ENOENT, Errno::ESRCH
       false
     rescue Errno::EPERM
-      raise MentosError, "EPERM checking if child process is alive."
+      raise MentosError, 'EPERM checking if child process is alive.'
     end
 
     # Public: Get an array of available Pygments formatters
     #
     # Returns an array of formatters.
     def formatters
-      mentos(:get_all_formatters).inject(Hash.new) do | hash, (name, desc, aliases) |
+      mentos(:get_all_formatters).each_with_object({}) do |(name, desc, aliases), hash|
         # Remove the long-winded and repetitive 'Formatter' suffix
         name.sub!(/Formatter$/, '')
         hash[name] = {
-          :name => name,
-          :description => desc,
-          :aliases => aliases
+          name: name,
+          description: desc,
+          aliases: aliases
         }
-        hash
       end
     end
 
-
     # Public: Get all lexers from a serialized array. This avoids needing to spawn
     # mentos when it's not really needed (e.g., one-off jobs, loading the Rails env, etc).
     #
@@ -145,33 +141,25 @@ module Pygments
     #
     # Returns an array of lexers.
     def lexers
-      begin
-        lexer_file = File.expand_path('../../../lexers', __FILE__)
-        raw = File.open(lexer_file, "rb").read
-        Marshal.load(raw)
-      rescue Errno::ENOENT
-        raise MentosError, "Error loading lexer file. Was it created and vendored?"
-      end
+      lexer_file = File.expand_path('../../lexers', __dir__)
+      raw = File.open(lexer_file, 'rb').read
+      Marshal.load(raw)
+    rescue Errno::ENOENT
+      raise MentosError, 'Error loading lexer file. Was it created and vendored?'
     end
 
     # Public: Get back all available lexers from mentos itself
     #
     # Returns an array of lexers.
     def lexers!
-      mentos(:get_all_lexers).inject(Hash.new) do |hash, lxr|
+      mentos(:get_all_lexers).each_with_object({}) do |lxr, hash|
         name = lxr[0]
         hash[name] = {
-          :name => name,
-          :aliases => lxr[1],
-          :filenames => lxr[2],
-          :mimetypes => lxr[3]
+          name: name,
+          aliases: lxr[1],
+          filenames: lxr[2],
+          mimetypes: lxr[3]
         }
-        hash["dasm16"] = {:name=>"dasm16", :aliases=>["DASM16"], :filenames=>["*.dasm16", "*.dasm"], :mimetypes=>['text/x-dasm16']}
-        hash["Puppet"] = {:name=>"Puppet", :aliases=>["puppet"], :filenames=>["*.pp"], :mimetypes=>[]}
-        hash["Augeas"] = {:name=>"Augeas", :aliases=>["augeas"], :filenames=>["*.aug"], :mimetypes=>[]}
-        hash["TOML"]   = {:name=>"TOML",   :aliases=>["toml"],   :filenames=>["*.toml"], :mimetypes=>[]}
-        hash["Slash"]  = {:name=>"Slash",  :aliases=>["slash"],  :filenames=>["*.sl"], :mimetypes=>[]}
-        hash
       end
     end
 
@@ -186,7 +174,7 @@ module Pygments
     end
 
     # Public: Return css for highlighted code
-    def css(klass='', opts={})
+    def css(klass = '', opts = {})
       if klass.is_a?(Hash)
         opts = klass
         klass = ''
@@ -197,17 +185,13 @@ module Pygments
     # Public: Return the name of a lexer.
     def lexer_name_for(*args)
       # Pop off the last arg if it's a hash, which becomes our opts
-      if args.last.is_a?(Hash)
-        opts = args.pop
-      else
-        opts = {}
-      end
+      opts = if args.last.is_a?(Hash)
+               args.pop
+             else
+               {}
+             end
 
-      if args.last.is_a?(String)
-        code = args.pop
-      else
-        code = nil
-      end
+      code = (args.pop if args.last.is_a?(String))
 
       mentos(:lexer_name_for, args, opts, code)
     end
@@ -219,7 +203,7 @@ module Pygments
     #
     # Returns the highlighted string
     # or nil when the request to the Python process timed out.
-    def highlight(code, opts={})
+    def highlight(code, opts = {})
       # If the caller didn't give us any code, we have nothing to do,
       # so return right away.
       return code if code.nil? || code.empty?
@@ -232,137 +216,163 @@ module Pygments
 
       # Get back the string from mentos and force encoding if we can
       str = mentos(:highlight, nil, opts, code)
-      str.force_encoding(opts[:options][:outencoding]) if str.respond_to?(:force_encoding)
+      if str.respond_to?(:force_encoding)
+        str.force_encoding(opts[:options][:outencoding])
+      end
       str
     end
 
     private
 
+    def with_watchdog(timeout_time, error_message)
+      state_mutex = Mutex.new
+      state = :alive
+
+      watchdog = timeout_time > 0 ? Thread.new do
+        state_mutex.synchronize do
+          state_mutex.sleep(timeout_time) if state != :finished
+          if state != :finished
+            @log.error error_message
+            stop error_message
+            state = :timeout
+          end
+        end
+      end : nil
+      begin
+        yield
+      ensure
+        if watchdog
+          state_mutex.synchronize do
+            state = :finished if state == :alive
+            watchdog.wakeup if watchdog.alive?
+          end
+          watchdog.join
+        end
+
+        raise MentosError, error_message if state == :timeout
+      end
+    end
+
     # Our 'rpc'-ish request to mentos. Requires a method name, and then optional
     # args, kwargs, code.
-    def mentos(method, args=[], kwargs={}, original_code=nil)
+    def mentos(method, args = [], kwargs = {}, original_code = nil)
       # Open the pipe if necessary
       start unless alive?
 
-      begin
-        # Timeout requests that take too long.
-        # Invalid MENTOS_TIMEOUT results in just using default.
-        timeout_time = Integer(ENV["MENTOS_TIMEOUT"]) rescue 8
-
-        Timeout::timeout(timeout_time) do
-          # For sanity checking on both sides of the pipe when highlighting, we prepend and
-          # append an id.  mentos checks that these are 8 character ids and that they match.
-          # It then returns the id's back to Rubyland.
-          id = (0...8).map{65.+(rand(25)).chr}.join
-          code = add_ids(original_code, id) if original_code
-
-          # Add metadata to the header and generate it.
-          if code
-            bytesize = code.bytesize
-          else
-            bytesize = 0
-          end
+      # Timeout requests that take too long.
+      # Invalid MENTOS_TIMEOUT results in just using default.
+      timeout_time = kwargs.delete(:timeout)
+      if timeout_time.nil?
+        timeout_time = begin
+                         Integer(ENV['MENTOS_TIMEOUT'])
+                       rescue TypeError
+                         0
+                       end
+      end
+
+      # For sanity checking on both sides of the pipe when highlighting, we prepend and
+      # append an id.  mentos checks that these are 8 character ids and that they match.
+      # It then returns the id's back to Rubyland.
+      id = (0...8).map { rand(65..89).chr }.join
+      code = original_code ? add_ids(original_code, id) : nil
+
+      # Add metadata to the header and generate it.
+      bytesize = if code
+                   code.bytesize
+                 else
+                   0
+                 end
 
-          kwargs.freeze
-          kwargs = kwargs.merge("fd" => @out.to_i, "id" => id, "bytes" => bytesize)
-          out_header = MultiJson.dump(:method => method, :args => args, :kwargs => kwargs)
+      kwargs.freeze
+      kwargs = kwargs.merge('fd' => @out.to_i, 'id' => id, 'bytes' => bytesize)
+      out_header = JSON.generate(method: method, args: args, kwargs: kwargs)
 
+      begin
+        res = with_watchdog(timeout_time, "Timeout on a mentos #{method} call") do
           # Get the size of the header itself and write that.
-          bits = get_fixed_bits_from_header(out_header)
-          @in.write(bits)
+          @in.write([out_header.bytesize].pack('N'))
+          @log.info "Size out: #{out_header.bytesize}"
 
           # mentos is now waiting for the header, and, potentially, code.
-          write_data(out_header, code)
+          @in.write(out_header)
+          @log.info "Out header: #{out_header}"
+          @in.write(code) unless code.nil?
 
-          check_for_error
+          @in.flush
 
           # mentos will now return data to us. First it sends the header.
-          header = get_header
-
-          # Now handle the header, any read any more data required.
-          res = handle_header_and_return(header, id)
-
-          # Finally, return what we got.
-          return_result(res, method)
-        end
-      rescue Timeout::Error
-        # If we timeout, we need to clear out the pipe and start over.
-        @log.error "Timeout on a mentos #{method} call"
-        stop "Timeout on mentos #{method} call."
-      end
-
-    rescue Errno::EPIPE, EOFError
-    stop "EPIPE"
-    raise MentosError, "EPIPE"
-    end
 
-    def check_for_error
-      return if @err.closed?
+          header_len_bytes = @out.read(4)
+          if header_len_bytes.nil?
+            raise Errno::EPIPE, %(Failed to read response from Python process on a mentos #{method} call)
+          end
 
-      timeout_time = 0.25 # set a very little timeout so that we do not hang the parser
+          header_len = header_len_bytes.unpack('N')[0]
+          @log.info "Size in: #{header_len}"
+          header = @out.read(header_len)
 
-      Timeout::timeout(timeout_time) do
-        error_msg = @err.read
+          # Now handle the header, any read any more data required.
+          handle_header_and_return(header, id)
+        end
 
-        unless error_msg.empty?
-          @log.error "Error running python script: #{error_msg}"
-          stop "Error running python script: #{error_msg}"
-          raise MentosError, error_msg
+        # Finally, return what we got.
+        return_result(res, method)
+      rescue Errno::EPIPE => e
+        begin
+          error_msg = @err.read
+          @log.error "Error running Python script: #{error_msg}"
+          stop "Error running Python script: #{error_msg}"
+          raise MentosError, %(#{e}: #{error_msg})
+        rescue Errno::EPIPE
+          @log.error e.to_s
+          stop e.to_s
+          raise e
         end
+      rescue StandardError => e
+        @log.error e.to_s
+        stop e.to_s
+        raise e
       end
-    rescue Timeout::Error
-      # during the specified time no error were found
-      @err.close
     end
 
-
     # Based on the header we receive, determine if we need
     # to read more bytes, and read those bytes if necessary.
     #
     # Then, do a sanity check with the ids.
     #
-    # Returns a result — either highlighted text or metadata.
+    # Returns a result - either highlighted text or metadata.
     def handle_header_and_return(header, id)
       if header
+        @log.info "In header: #{header}"
         header = header_to_json(header)
         bytes = header[:bytes]
 
         # Read more bytes (the actual response body)
         res = @out.read(bytes.to_i)
 
-        if header[:method] == "highlight"
+        if header[:method] == 'highlight'
           # Make sure we have a result back; else consider this an error.
-          if res.nil?
-            @log.warn "No highlight result back from mentos."
-            stop "No highlight result back from mentos."
-            raise MentosError, "No highlight result back from mentos."
-          end
+          raise MentosError, 'No highlight result back from mentos.' if res.nil?
 
-          # Remove the newline from Python
-          res = res[0..-2]
-          @log.info "Highlight in process."
+          @log.info 'Highlight in process.'
 
           # Get the id's
           start_id = res[0..7]
           end_id = res[-8..-1]
 
           # Sanity check.
-          if not (start_id == id and end_id == id)
-            @log.error "ID's did not match. Aborting."
-            stop "ID's did not match. Aborting."
+          if !((start_id == id) && (end_id == id))
             raise MentosError, "ID's did not match. Aborting."
           else
             # We're good. Remove the padding
             res = res[10..-11]
-            @log.info "Highlighting complete."
+            @log.info 'Highlighting complete.'
             res
           end
         end
         res
       else
-        @log.error "No header data back."
-        stop "No header data back."
-        raise MentosError, "No header received back."
+        raise MentosError, 'No header received back.'
       end
     end
 
@@ -371,62 +381,14 @@ module Pygments
     # id, with two padding also. This means we are sending over the 8 characters +
     # code + 8 characters.
     def add_ids(code, id)
-      code.freeze
-      code = id + "  #{code}  #{id}"
-      code
-    end
-
-    # Write data to mentos, the Python process.
-    #
-    # Returns nothing.
-    def write_data(out_header, code=nil)
-      @in.write(out_header)
-      @log.info "Out header: #{out_header}"
-      @in.write(code) if code
-    end
-
-    # Sanity check for size (32-arity of 0's and 1's)
-    def size_check(size)
-      size_regex = /[0-1]{32}/
-      if size_regex.match(size)
-        true
-      else
-        false
-      end
-    end
-
-    # Read the header via the pipe.
-    #
-    # Returns a header.
-    def get_header
-      begin
-        size = @out.read(33)
-        size = size[0..-2]
-
-        # Sanity check the size
-        if not size_check(size)
-          @log.error "Size returned from mentos.py invalid."
-          stop "Size returned from mentos.py invalid."
-          raise MentosError, "Size returned from mentos.py invalid."
-        end
-
-        # Read the amount of bytes we should be expecting. We first
-        # convert the string of bits into an integer.
-        header_bytes = size.to_s.to_i(2) + 1
-        @log.info "Size in: #{size.to_s} (#{header_bytes.to_s})"
-        @out.read(header_bytes)
-      rescue
-        @log.error "Failed to get header."
-        stop "Failed to get header."
-        raise MentosError, "Failed to get header."
-      end
+      (id + "  #{code}  #{id}").freeze
     end
 
     # Return the final result for the API. Return Ruby objects for the methods that
     # want them, text otherwise.
     def return_result(res, method)
       unless method == :lexer_name_for || method == :highlight || method == :css
-        res = MultiJson.load(res, :symbolize_keys => true)
+        res = JSON.parse(res, symbolize_names: true)
       end
       res = res.rstrip if res.class == String
       res
@@ -434,27 +396,13 @@ module Pygments
 
     # Convert a text header into JSON for easy access.
     def header_to_json(header)
-      @log.info "[In header: #{header} "
-      header = MultiJson.load(header, :symbolize_keys => true)
+      header = JSON.parse(header, symbolize_names: true)
 
       if header[:error]
-        # Raise this as a Ruby exception of the MentosError class.
-        # Stop so we don't leave the pipe in an inconsistent state.
-        @log.error "Failed to convert header to JSON."
-        stop header[:error]
         raise MentosError, header[:error]
       else
         header
       end
     end
-
-    def get_fixed_bits_from_header(out_header)
-      size = out_header.bytesize
-
-      # Fixed 32 bits to represent the int. We return a string
-      # representation: e.g, "00000000000000000000000000011110"
-      Array.new(32) { |i| size[i] }.reverse!.join
-    end
   end
 end
-