pilfer 0.0.1.pre2 → 0.0.1.pre3

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NTM1Y2FiYTgxMTU1OGZlZGRiYjMwMGUzNjExMmQwMDA5NmM4MmUwMw==
+    NzkwNzViNDg0NTFlMDIyNmU5YWMwYTYxNmY3NmE1YjJlYjgyY2IxOQ==
   data.tar.gz: !binary |-
-    YmU2MmM5ZDMzZGViNWJiNzliZmM2MzdjYzk1YTdhNTZjYWE4OTc0ZA==
+    ZjcwZjRkNzZiNzI0MjllYTE0NzUzYWQyNWZlOTM2MzQ4MzA3MGNkNA==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    ODFhZWVmNjM5NThhZmUyYWNhMTg2YzFlNWViMDllN2VlYzllZjVlNzM1NWVi
-    ZjA5OGM5ZGU1YWFiZDQ1ZjRhZjRkMDdhNDUzNjg0ZDdkZjAyZTExMTQ3YjRj
-    NjJkMjVkNWU5NDNhN2Y0NjQ1NDI3NjY5ODI5MDEyZmRhMmU2ZmI=
+    YTlhYzBiMGRjYTY5OTdiYzM4MTlmODdkZWI5MTBhZmRiN2Q1ZGYxOGNmZjVl
+    Yjk5ZDVhMGFmOWEyYTUwNTFmM2MzZTYwZDZhMzQ0MzExNDVhNTE0ODZlNGYy
+    OWM3NTY3N2NmZWQ2ZTUzM2Q3YmU2MzhjOTA1NzIxMmM3Y2U1OWY=
   data.tar.gz: !binary |-
-    YWNmNzEzYTNjMTE5MjM2YjZkZjkxMjY0ZmUzZDFmZmZjYTRjOGE3Y2NiNDM4
-    N2UwODUzODE5NjgyZTgxN2JhODM0M2MyOTcyY2QwNGI4OWYwZTZhMDRhY2U0
-    NzcwMDdkNDcyOTIzZWMxODFhZTYxOGQ1MGZmYWU2Y2ViYjczMTI=
+    NTE4OTU1OGQwYWExYjI0ZjE4YTFjN2RmNDExZGUxZTA5ZDUzYjZjYjE1MGNh
+    MWY3NjkxYjY2N2Q3YjYyNzJlODFlZjQ2NjVmODU5ZThkMWE1OWRjNDkzNDNj
+    MmUxYjAxMzZlNTcxMmRmYzZiZWIzNWU0Zjk0ZWVkMGI3ZTA2Njg=

data/lib/pilfer/logger.rb

@@ -1,11 +1,12 @@
+require 'logger'
 require 'pilfer/profile'
 
 module Pilfer
   class Logger
-    attr_reader :path, :app_root
+    attr_reader :app_root, :logger
 
-    def initialize(path, options = {})
-      @path = path
+    def initialize(path_or_io, options = {})
+      @logger = ::Logger.new(path_or_io)
       if (app_root = options[:app_root])
         app_root += '/' unless app_root[-1] == '/'
         @app_root = %r{^#{Regexp.escape(app_root)}}
@@ -14,46 +15,41 @@ module Pilfer
 
     def write(profile_data, profile_start)
       profile = Pilfer::Profile.new(profile_data, profile_start)
-      File.open(path, 'w') do |file|
-        print_report_banner file, profile_start
-        profile.each do |path, data|
-          print_file_banner file, path, data
-          print_file_source_with_profile file, path, data
-        end
+      print_report_banner profile_start
+      profile.each do |path, data|
+        print_file_banner path, data
+        print_file_source_with_profile path, data
       end
     end
 
     private
 
-    def print_report_banner(file, profile_start)
-      file.puts '#' * 50
-      file.puts "# #{profile_start.utc.to_s}"
-      file.puts '#' * 50
-      file.puts
+    def print_report_banner(profile_start)
+      logger.info "Profile start=#{profile_start.utc.to_s}"
     end
 
-    def print_file_banner(file, path, data)
+    def print_file_banner(path, data)
       wall = data['wall_time'] / 1000.0
       cpu  = data['cpu_time']  / 1000.0
-      file.puts sprintf("%s wall_time=%.1fms cpu_time=%.1fms",
-                        strip_app_root(path), wall, cpu)
+      logger.info sprintf("%s wall_time=%.1fms cpu_time=%.1fms",
+                          strip_app_root(path), wall, cpu)
     end
 
-    def print_file_source_with_profile(file, path, data)
+    def print_file_source_with_profile(path, data)
       return unless File.exists?(path)
       File.readlines(path).each_with_index do |line_source, index|
+        line_source  = line_source.chomp
         line_profile = data['lines'][index]
         if line_profile && line_profile['calls'] > 0
           total = line_profile['wall_time']
-          file.puts sprintf("% 8.1fms (% 5d) | %s",
-                            total/1000.0,
-                            line_profile['calls'],
-                            line_source)
+          logger.info sprintf("% 8.1fms (% 5d) | %s",
+                              total/1000.0,
+                              line_profile['calls'],
+                              line_source)
         else
-          file.puts sprintf("                   | %s", line_source)
+          logger.info sprintf("                   | %s", line_source)
         end
       end
-      file.puts
     end
 
     def strip_app_root(path)

data/lib/pilfer/version.rb

@@ -1,3 +1,3 @@
 module Pilfer
-  VERSION = '0.0.1.pre2'
+  VERSION = '0.0.1.pre3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pilfer
 version: !ruby/object:Gem::Version
-  version: 0.0.1.pre2
+  version: 0.0.1.pre3
 platform: ruby
 authors:
 - Eric Lindvall