Skip to content
This repository

Missed a comma #5

Merged
merged 1 commit into from almost 2 years ago

2 participants

Andrew Ekstedt Brendan Gregg
Andrew Ekstedt

flamegraph.pl still prefixes each line with a comma instead of a semicolon.

Brendan Gregg
Owner

thanks!

Brendan Gregg brendangregg merged commit 9890030 into from June 21, 2012
Brendan Gregg brendangregg closed this June 21, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

Jun 18, 2012
Andrew Ekstedt Missed a comma. 2821b88
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. 2  flamegraph.pl
2  flamegraph.pl
@@ -174,7 +174,7 @@ sub flow {
174 174
 	my ($stack, $samples) = (/^(.*)\s+(\d+)$/);
175 175
 	$stack =~ s/</(/g;
176 176
 	$stack =~ s/>/)/g;
177  
-	$stack = ",$stack";
  177
+	$stack = ";$stack";
178 178
 	next unless defined $samples;
179 179
 	flow($last, $stack, $time);
180 180
 	$time += $samples;
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.