Skip to content

Commit

Permalink
Dump the flow tree with println, instead of error
Browse files Browse the repository at this point in the history
  • Loading branch information
Clark Gaebel committed Oct 20, 2014
1 parent 8523782 commit 1749750
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion components/layout/flow.rs
View file
Expand Up @@ -1022,7 +1022,7 @@ impl<'a> ImmutableFlowUtils for &'a Flow + 'a {
indent.push_str("| ")
}

error!("{}+ {}", indent, self.to_string());
println!("{}+ {}", indent, self.to_string());

for kid in imm_child_iter(self) {
kid.dump_with_level(level + 1)
Expand Down

18 comments on commit 1749750

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from mrobinson
at cgaebel@1749750

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cgaebel/servo/dump-flow-tree-with-println = 1749750 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cgaebel/servo/dump-flow-tree-with-println = 1749750 merged ok, testing candidate = aae5be5

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some tests failed:
failure: http://build.servo.org/builders/linux/builds/355

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from mrobinson
at cgaebel@1749750

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cgaebel/servo/dump-flow-tree-with-println = 1749750 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cgaebel/servo/dump-flow-tree-with-println = 1749750 merged ok, testing candidate = 3f9a5d9

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some tests failed:
failure: http://build.servo.org/builders/mac/builds/362

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from mrobinson
at cgaebel@1749750

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cgaebel/servo/dump-flow-tree-with-println = 1749750 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cgaebel/servo/dump-flow-tree-with-println = 1749750 merged ok, testing candidate = c03aabe

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some tests failed:
failure: http://build.servo.org/builders/mac/builds/364

@cgaebel
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors: retry

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from mrobinson
at cgaebel@1749750

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging cgaebel/servo/dump-flow-tree-with-println = 1749750 into auto

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cgaebel/servo/dump-flow-tree-with-println = 1749750 merged ok, testing candidate = 018146b

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all tests pass:
success: http://build.servo.org/builders/linux/builds/383
success: http://build.servo.org/builders/mac/builds/390

@bors-servo
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 018146b

Please sign in to comment.