Skip to content

Commit

Permalink
Fix issue 2626 by un-ignoring the mysterious test_leaks,
Browse files Browse the repository at this point in the history
which does currently seem to work on win32 (and linux).
  • Loading branch information
gareth authored and gareth committed Apr 16, 2013
1 parent 7cacd87 commit caffe12
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/libcore/run.rs
Original file line number Diff line number Diff line change
Expand Up @@ -520,7 +520,7 @@ mod tests {
use run;

// Regression test for memory leaks
#[ignore(cfg(windows))] // FIXME (#2626)
#[test]
fn test_leaks() {
run::run_program("echo", []);
run::start_program("echo", []);
Expand Down

5 comments on commit caffe12

@bors
Copy link
Contributor

@bors bors commented on caffe12 Apr 18, 2013

Choose a reason for hiding this comment

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

saw approval from graydon
at Dretch@caffe12

@bors
Copy link
Contributor

@bors bors commented on caffe12 Apr 18, 2013

Choose a reason for hiding this comment

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

merging Dretch/rust/issue-2626 = caffe12 into auto

@bors
Copy link
Contributor

@bors bors commented on caffe12 Apr 18, 2013

Choose a reason for hiding this comment

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

Dretch/rust/issue-2626 = caffe12 merged ok, testing candidate = 3b48063

@bors
Copy link
Contributor

@bors bors commented on caffe12 Apr 18, 2013

Choose a reason for hiding this comment

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

@bors
Copy link
Contributor

@bors bors commented on caffe12 Apr 18, 2013

Choose a reason for hiding this comment

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

fast-forwarding incoming to auto = 3b48063

Please sign in to comment.