Skip to content
Browse files

free edge

Signed-off-by: Thiago Farina <tfarina@chromium.org>
  • Loading branch information...
1 parent 6d7e0ca commit eb143ea5109d2a650d2ac8241fbeab77e7da40e3 @tfarina committed Feb 18, 2013
Showing with 3 additions and 0 deletions.
  1. +3 −0 src/state_test.cc
View
3 src/state_test.cc
@@ -41,6 +41,9 @@ TEST(State, Basic) {
EXPECT_EQ("cat in1 in2 > out", edge->EvaluateCommand());
+ // Release the edge after we are done with it.
+ delete edge;
@martine
martine added a note Mar 11, 2013

This isn't right; the edges are owned by the State object.

@tfarina
Owner
tfarina added a note Mar 12, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
+
EXPECT_FALSE(state.GetNode("in1")->dirty());
EXPECT_FALSE(state.GetNode("in2")->dirty());
EXPECT_FALSE(state.GetNode("out")->dirty());

0 comments on commit eb143ea

Please sign in to comment.
Something went wrong with that request. Please try again.