debugtrace 0.2.5 → 0.2.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 665b9852fb34caab345012ab6bac6e18c0e8951d689125ea9cf0c399a0c7637e
-  data.tar.gz: d0940c27f8beec19e0aecfc609ddab7a35ecefedcde6251b1de5382d9d8ea319
+  metadata.gz: 7a1011528c478c032c1223c1fceb442bb32de6ba1ef40d9df2436b27718327c4
+  data.tar.gz: 5e11c1af69b68a0d8b380d7e4c09b703ad773ea43a23fb11cee1289827dfd926
 SHA512:
-  metadata.gz: 3b3a87106db56eb5c88c42c28f5c25eaf17a19a502c82411853554200c7ddaa7c5117248d5b0059206a467b53a47abdf9e37a4aa4b736199f051b76cbe4ecd0d
-  data.tar.gz: db33d2a6c4df894006aa1daada1fa777cdb0908ffeb4ccdb9b5de95e86dbece0835d0bea23a39c7f185dcf3fa36b6304ad680a185f58598a8cbe74b6d6bcb328
+  metadata.gz: a74c1b4784462f0ebc6a6b40d55c9ebba4df3bd087ec5e421fecd8b0c79cdda06404fa7e5c914d83ddccd96b6aa3dadeef0d21b1f53a3164cbcf0c7932b86298
+  data.tar.gz: e6b556af241a5976d6e7731138cb48d3059ce90b0ec4b274d3844da134532ed0156280bb37d85ae7d03106d1ea855dd399723ba41546fbdd42b71221a41b65e1

data/lib/debugtrace/common.rb

@@ -30,6 +30,6 @@ module Common
         top_type_name == 'E' || top_type_name == 'O' ? 'an' : 'a'
       raise "Argument #{value_name} (=#{value_string}) must be #{a} #{type}"
     end
-    value
+    return value
   end
 end

data/lib/debugtrace/config.rb

@@ -77,7 +77,7 @@ class Config
   # Returns true if logging is enabled, false otherwise.
   # @return true if logging is enabled, false otherwise
   def enabled?
-    @enabled
+    return @enabled
   end
 
   private
@@ -97,6 +97,6 @@ class Config
         Common.check_type("config[#{key}]", value, defalut_value.class)
       end
     end
-    value
+    return value
   end
 end

data/lib/debugtrace/log_buffer.rb

@@ -15,7 +15,7 @@ class LogBuffer
     attr_reader :nest_level, :log
 
     def to_s
-      "(LogBuffer.LevelAndLog){nest_level: #{@nest_level}, log: \"#{@log}\"}"
+      return "(LogBuffer.LevelAndLog){nest_level: #{@nest_level}, log: \"#{@log}\"}"
     end
   end
 
@@ -64,7 +64,7 @@ class LogBuffer
       @append_nest_level = nest_level
       @last_line += string
     end
-    self
+    return self
   end
 
   # Appends a string representation of the value.
@@ -72,7 +72,7 @@ class LogBuffer
   # @param value [Object] The value to append
   # @return LogBuffer: This object
   def no_break_append(value)
-    append(value, 0, true)
+    return append(value, 0, true)
   end
 
   # Appends lines of another LogBuffer.
@@ -90,23 +90,23 @@ class LogBuffer
       append(line.log, line.nest_level, index == 0 && separator != '')
       index += 1
     end
-    self
+    return self
   end
 
   # The length of the last line.
   def length
-    @last_line.length
+    return @last_line.length
   end
 
   # true if multiple line, false otherwise.
   def multi_lines?
-    @lines.length > 1 || @lines.length == 1 && length > 0
+    return @lines.length > 1 || @lines.length == 1 && length > 0
   end
 
   # A list of tuple of data indentation level && log string.
   def lines
     lines = @lines.dup
     lines << LevelAndLog.new(@nest_level, @last_line) if length > 0
-    lines
+    return lines
   end
 end

data/lib/debugtrace/loggers.rb

@@ -16,7 +16,7 @@ class LoggerBase
   # Returns a string representation of this object.
   # @return [String] A string representation of this object
   def to_s
-    "#{self.class.name}"
+    return "#{self.class.name}"
   end
 end
 
@@ -85,12 +85,13 @@ class RubyLogger
   def print(message)
     Common::check_type("message", message, String)
     @logger.log(Logger::Severity::DEBUG, message, 'DebugTrace-rb')
+    return message
   end
 
   # Returns a string representation of this object.
   # @return [String] A string representation of this object
   def to_s
-    "Ruby #{Logger.name} path: #{@config.log_path}"
+    return "Ruby #{Logger.name} path: #{@config.log_path}"
   end
 end
 
@@ -132,11 +133,12 @@ class FileLogger < LoggerBase
         file.puts "#{datetime_str} #{message}"
       }
     end
+    return message
   end
 
   # Returns a string representation of this object.
   # @return [String] A string representation of this object
   def to_s
-    "#{self.class.name} path: #{@log_path}, append: #{@append}"
+    return "#{self.class.name} path: #{@log_path}, append: #{@append}"
   end
 end

data/lib/debugtrace/state.rb

@@ -5,46 +5,23 @@ require_relative 'common'
 # Have the trace state for a thread
 # @author Masato Kokubo
 class State
+  attr_reader :thread_id
+  attr_reader :nest_level
+  attr_reader :previous_nest_level
+
   def initialize(thread_id)
-    @thread_id = Common::check_type('thread_id', thread_id, Integer)
+    @thread_id = thread_id
     reset()
   end
 
-  # @return the thread id.
-  def thread_id
-    @thread_id
-  end
-
-  # @return the nest level.
-  def nest_level
-    @nest_level
-  end
-
-  # @return the previous nest level.
-  def previous_nest_level
-    @previous_nest_level
-  end
-
-  # @return the previous line count.
-  def previous_line_count
-    @previous_line_count
-  end
-
-  # Sets the previous line count.
-  # @param value the previous line count
-  def previous_line_count=(value)
-    @previous_line_count = Common::check_type('value', value, Integer)
-  end
-
   def reset
     @nest_level = 0
     @previous_nest_level = 0
-    @previous_line_count = 0
     @times = []
   end
 
   def to_s()
-      "(State){thread_id: #{@thread_id}, nest_level: #{@nest_level}, previous_nest_level: #{@previous_nest_level}, previous_line_count: #{@previous_line_count}, times: #{@times}}"
+      return "(State){thread_id: #{@thread_id}, nest_level: #{@nest_level}, previous_nest_level: #{@previous_nest_level}, times: #{@times}}"
   end
 
   # Ups the nest level.
@@ -54,6 +31,7 @@ class State
           @times.push(Time.now)
       end
       @nest_level += 1
+      return nil
   end
 
   #Downs the nest level.

data/lib/debugtrace/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module DebugTrace
-  VERSION = '0.2.5'
+  VERSION = '0.2.6'
 end

data/lib/debugtrace.rb

@@ -16,7 +16,7 @@ module DebugTrace
   @@config = nil
 
   def self.config
-    @@config
+    return @@config
   end
 
   # A Mutex for thread safety
@@ -59,8 +59,9 @@ module DebugTrace
 
     return unless @@config.enabled?
 
+    config_path = File.expand_path(@@config.config_path)
     @@logger.print("DebugTrace-rb #{DebugTrace::VERSION} on Ruby #{RUBY_VERSION}")
-    @@logger.print("  config file: #{@@config.config_path}")
+    @@logger.print("  config file: #{config_path}")
     @@logger.print("  logger: #{@@logger}")
   end
 
@@ -95,13 +96,13 @@ module DebugTrace
       @@state_hash[thread_id] = state
     end
 
-    state
+    return state
   end
 
   def self.get_indent_string(nest_level, data_nest_level)
     indent_str = @@config.indent_string * [[0, nest_level].max, @@config.maximum_indents].min
     data_indent_str = @@config.data_indent_string * [[0, data_nest_level].max, @@config.maximum_indents].min
-    indent_str + data_indent_str
+    return indent_str + data_indent_str
   end
 
   def self.to_string(name, value, print_options)
@@ -153,7 +154,7 @@ module DebugTrace
       buff.append_buffer(separator, value_buff)
     end
 
-    buff
+    return buff
   end
 
   def self.to_string_str(value, print_options)
@@ -215,9 +216,7 @@ module DebugTrace
     double_quote_buff.no_break_append('"')
     single_quote_buff.no_break_append("'")
 
-    return double_quote_buff if has_single_quote && !has_double_quote
-
-    single_quote_buff
+    return has_single_quote && !has_double_quote ? double_quote_buff : single_quote_buff
   end
 
   def self.to_string_bytes(value, print_options)
@@ -278,7 +277,7 @@ module DebugTrace
     end
     buff.no_break_append(']')
 
-    buff
+    return buff
   end
 
   def self.to_string_reflection(value, print_options)
@@ -304,7 +303,7 @@ module DebugTrace
     end
     buff.no_break_append('}')
 
-    buff
+    return buff
   end
 
   def self.to_string_reflection_body(value, print_options)
@@ -328,7 +327,7 @@ module DebugTrace
       index += 1
     end
 
-    buff
+    return buff
   end
 
   def self.to_string_enumerable(values, print_options)
@@ -367,7 +366,7 @@ module DebugTrace
 
     buff.no_break_append(close_char)
 
-    buff
+    return buff
   end
 
   def self.to_string_enumerable_body(values, print_options)
@@ -401,7 +400,7 @@ module DebugTrace
 
     buff.no_break_append(':') if values.is_a?(Hash) && values.empty?
 
-    buff
+    return buff
   end
 
   def self.to_string_key_value(key, value, print_options)
@@ -420,7 +419,7 @@ module DebugTrace
       type_name += @@config.size_format % count
     end
 
-    type_name
+    return type_name
   end
 
   def self.has_to_s_method?(value)
@@ -501,7 +500,7 @@ module DebugTrace
       end
     end
 
-    value
+    return value
   end
 
   def self.enter
@@ -537,10 +536,10 @@ module DebugTrace
     end
   end
 
-  def self.leave
+  def self.leave(return_value = nil)
     @@thread_mutex.synchronize do
       print_start
-      return unless @@config.enabled?
+      return return_value unless @@config.enabled?
 
       state = current_state
 
@@ -561,6 +560,7 @@ module DebugTrace
       )
       @@last_log_buff.line_feed
       @@logger.print(get_indent_string(state.nest_level, 0) + @@last_log_buff.lines[0].log)
+      return return_value
     end
   end
 
@@ -573,6 +573,6 @@ module DebugTrace
       state = current_state
     end
 
-    "#{get_indent_string(state.nest_level, 0)}#{buff_string}"
+    return "#{get_indent_string(state.nest_level, 0)}#{buff_string}"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: debugtrace
 version: !ruby/object:Gem::Version
-  version: 0.2.5
+  version: 0.2.6
 platform: ruby
 authors:
 - Masato Kokubo