diff options
| author | Keegan McAllister <kmcallister@mozilla.com> | 2014-09-16 14:18:37 -0700 |
|---|---|---|
| committer | Keegan McAllister <kmcallister@mozilla.com> | 2014-09-17 11:18:53 -0700 |
| commit | 5b42f79ff088bf3f9136f6f668a481097e22bed9 (patch) | |
| tree | 9c4d7112eeb42d948dd4c9f08b8aa00429379de2 /src/libsyntax | |
| parent | f85e4f75b9b60e2ce4b2ade8f04e2537e48f7095 (diff) | |
| download | rust-5b42f79ff088bf3f9136f6f668a481097e22bed9.tar.gz rust-5b42f79ff088bf3f9136f6f668a481097e22bed9.zip | |
Pop the expansion context after expanding a method macro
We were leaving these on the stack, causing spurious backtraces. I've confirmed that this test fails without the fix.
Diffstat (limited to 'src/libsyntax')
| -rw-r--r-- | src/libsyntax/ext/expand.rs | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/libsyntax/ext/expand.rs b/src/libsyntax/ext/expand.rs index 4ff9912645a..892213e684f 100644 --- a/src/libsyntax/ext/expand.rs +++ b/src/libsyntax/ext/expand.rs @@ -897,7 +897,10 @@ fn expand_method(m: P<ast::Method>, fld: &mut MacroExpander) -> SmallVector<P<as }; // expand again if necessary - new_methods.into_iter().flat_map(|m| fld.fold_method(m).into_iter()).collect() + let new_methods = new_methods.move_iter() + .flat_map(|m| fld.fold_method(m).into_iter()).collect(); + fld.cx.bt_pop(); + new_methods } }) } |
