xcprofiler 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 619014224d306a694c7da11d1ed1306b9b2e3fac
-  data.tar.gz: c0b91191a1208dff867f74cde952515736439118
+  metadata.gz: 96174bfd64462a8f8a140c968cec38465568ece7
+  data.tar.gz: 5bc873480019dea46b48dba6e74633fc738d7e26
 SHA512:
-  metadata.gz: 3239189c1c076857ca30a1d68a1682452118e963d2491750c08b4ddacf0a1a9dc61c5027e0c35910469e38b7a4ec4ce268b08ea05f3429bdfd80614b542b09ed
-  data.tar.gz: ca139ec88be8f4d7f8429018b04df948e1eb961259c89733f16895cd6698fea69591c09f10aab7b64e8962812a51eba292bcbf8213f308443d97c22aa0172ee1
+  metadata.gz: 7907ec7bd31c2be0e522d8691e8258123f948325029f198e39d34e66aebaf43c279739c55f4a12e2dcd997f1b4fa016760ce36a9c4afd14203de98c260204a18
+  data.tar.gz: eeca6143e71a334f18acefe010cfd289d9675a505d7cfc3af6e21e23d0f4eaefeba094613e2fa5096a993682ffcfebef9690d65df5592c53bf2409bf8a22e32f

data/README.md

@@ -76,6 +76,7 @@ Sample output is here
 |--order|-o|Sort order (default,time,file)|
 |--derived-data-path||Root path of DerivedData directory|
 |--truncate-at|-t|Truncate the method name with specified length|
+|--no-unique|Show the duplicated results|
 
 ## Use custom reporters
 
@@ -86,8 +87,11 @@ require 'xcprofiler'
 
 profiler = Xcprofiler::Profiler.by_product_name('MyApp')
 profiler.reporters = [
-  Xcprofiler::StandardOutputReporter.new(limit: 20, order: :time)],
-  Xcprofiler::JSONReporter.new({output_path: 'result.json'})
+  Xcprofiler::StandardOutputReporter.new(limit: 20, order: :time),
+  Xcprofiler::JSONReporter.new(output_path: 'result.json'),
+  Xcprofiler::BlockReporter.new do |executions|
+    do_something(executions)
+  end,
 ]
 profiler.report!
 ```

data/lib/xcprofiler.rb

@@ -4,6 +4,7 @@ require "xcprofiler/execution"
 require "xcprofiler/profiler"
 require "xcprofiler/version"
 require "xcprofiler/reporters/abstract_reporter"
+require "xcprofiler/reporters/block_reporter"
 require "xcprofiler/reporters/standard_output_reporter"
 require "xcprofiler/reporters/json_reporter"
 require "colorize"
@@ -26,6 +27,7 @@ module Xcprofiler
         opts.on("--threshold [THRESHOLD]", Integer, "Threshold of time to display(ms)") { |v| options.threshold = v }
         opts.on("--derived-data-path", String, "Root path of DerivedData") { |v| options.derived_data_path = v }
         opts.on("-t", "--truncate-at [TRUNCATE_AT]", Integer, "Truncate the method name with specified length") { |v| options.truncate_at = v }
+        opts.on("--[no-]unique", "Reject duplicated location results or not") { |v| options.unique = v }
         opts.on_tail("-h", "--help", "Show this message") do
           puts opts
           exit
@@ -48,13 +50,12 @@ module Xcprofiler
           derived_data_path = options[:derived_data_path]
           profiler = Profiler.by_product_name(target, derived_data_path)
         end
-        profiler.reporters = [
-          StandardOutputReporter.new(limit: options[:limit],
+        profiler.reporters = [StandardOutputReporter.new(limit: options[:limit],
                                      threshold: options[:threshold],
                                      order: order,
                                      show_invalid_locations: options[:show_invalid_locations],
-                                     truncate_at: options[:truncate_at])
-        ]
+                                     truncate_at: options[:truncate_at],
+                                     unique: options[:unique])]
         profiler.report!
       rescue Exception => e
         puts e.message.red

data/lib/xcprofiler/execution.rb

@@ -1,14 +1,14 @@
 module Xcprofiler
   class Execution
-    Struct.new('Position', :path, :line, :column)
+    Struct.new('Location', :path, :line, :column)
 
-    attr_reader :time, :position, :method_name
+    attr_reader :time, :location, :method_name
 
-    def initialize(time, position, method_name)
+    def initialize(time, location, method_name)
       @time = time.to_f
-      unless position =~ /<invalid loc>/
-        path, line, column = position.split(':')
-        @position = Struct::Position.new(path, line.to_i, column.to_i)
+      unless location =~ /<invalid loc>/
+        path, line, column = location.split(':')
+        @location = Struct::Location.new(path, line.to_i, column.to_i)
       end
       @method_name = method_name
     end
@@ -24,12 +24,12 @@ module Xcprofiler
     end
 
     def invalid?
-      !position
+      !location
     end
 
     def path
-      if @position
-        @position.path
+      if @location
+        @location.path
       else
         nil
       end
@@ -44,16 +44,16 @@ module Xcprofiler
     end
 
     def column
-      if @position
-        @position.column
+      if @location
+        @location.column
       else
         nil
       end
     end
 
     def line
-      if @position
-        @position.line
+      if @location
+        @location.line
       else
         nil
       end

data/lib/xcprofiler/profiler.rb

@@ -21,7 +21,7 @@ module Xcprofiler
     end
 
     def report!
-      if !derived_data.flag_enabled?
+      unless derived_data.flag_enabled?
         raise BuildFlagIsNotEnabled, "'-Xfrontend -debug-time-function-bodies' flag is not enabled"
       end
 

data/lib/xcprofiler/reporters/abstract_reporter.rb

@@ -13,9 +13,10 @@ module Xcprofiler
     end
 
     def filter_executions(executions)
-      executions = sort_executions(executions, order)
       executions = executions.delete_if(&:invalid?) unless show_invalid_locations?
+      executions = delete_duplicated(executions) if unique
       executions = executions.delete_if { |v| v.time < threshold } if threshold
+      executions = sort_executions(executions, order)
       executions = executions[0...limit] if limit
       executions
     end
@@ -33,6 +34,14 @@ module Xcprofiler
       end
     end
 
+    def delete_duplicated(executions)
+      executions.group_by { |execution| 
+        execution.location 
+      }.map { |location, executions| 
+        executions.max { |execution| execution.time }
+      }
+    end
+
     def limit
       options[:limit]
     end
@@ -52,5 +61,10 @@ module Xcprofiler
     def truncate_at
       options[:truncate_at] ||= DEFAULT_TRUNCATE_AT
     end
+
+    def unique
+      return options[:unique] unless options[:unique].nil?
+      true
+    end
   end
 end

data/lib/xcprofiler/reporters/block_reporter.rb

@@ -0,0 +1,13 @@
+module Xcprofiler
+  class BlockReporter < AbstractReporter
+    def initialize(options = {}, &block)
+      raise ArgumentError, '[BlockReporter] block must be passed' unless block_given?
+      super(options)
+      @block = block
+    end
+
+    def report!(executions)
+      @block.call(executions)
+    end
+  end
+end

data/lib/xcprofiler/version.rb

@@ -1,3 +1,3 @@
 module Xcprofiler
-  VERSION = "0.5.0"
+  VERSION = "0.6.0"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: xcprofiler
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - giginet
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-04-07 00:00:00.000000000 Z
+date: 2017-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -120,6 +120,7 @@ files:
 - lib/xcprofiler/execution.rb
 - lib/xcprofiler/profiler.rb
 - lib/xcprofiler/reporters/abstract_reporter.rb
+- lib/xcprofiler/reporters/block_reporter.rb
 - lib/xcprofiler/reporters/json_reporter.rb
 - lib/xcprofiler/reporters/standard_output_reporter.rb
 - lib/xcprofiler/version.rb
@@ -143,7 +144,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.8
 signing_key: 
 specification_version: 4
 summary: CLI to profile compilation time of Swift projects