From baa0018b8cc5005b402a8d026f342f2d83ebf45f Mon Sep 17 00:00:00 2001 From: Camden Narzt Date: Mon, 25 May 2015 15:16:13 -0600 Subject: [PATCH] Better sorting behaviour sort stacks alphabetically, not by time --- lib/flamegraph/renderer.rb | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/lib/flamegraph/renderer.rb b/lib/flamegraph/renderer.rb index 3d6dffe..b536f9d 100644 --- a/lib/flamegraph/renderer.rb +++ b/lib/flamegraph/renderer.rb @@ -26,13 +26,11 @@ def graph_data prev = [] # a 2d array makes collapsing easy - @stacks.each_with_index do |stack, pos| - - next unless stack + @stacks.map! { |s| s.map!(&:to_s).reverse! }.delete_if{|s| s.to_a.empty? }.sort_by!{|a| a }.each_with_index do |stack, pos| col = [] - stack.reverse.map{|r| r.to_s}.each_with_index do |frame, i| + stack.each_with_index do |frame, i| if !prev[i].nil? last_col = prev[i]