---

yaml
---
r: 14810
b: refs/heads/try
c: 1598527
h: refs/heads/master
v: v3

Author: kevina

[refs]

@@ -2,5 +2,5 @@
 refs/heads/master: 61b1875c16de39c166b0f4d54bba19f9c6777d1a
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 4a81779abd786ff22d71434c6d9a5917ea4cdfff
-refs/heads/try: 6f5853f5a1767e0c418fd5f348a795b76d701b3e
+refs/heads/try: 15985277b98e53014b52c1e243eefcfb0335089f
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105

branches/try/src/rustc/syntax/ext/qquote.rs

@@ -103,6 +103,9 @@ fn gather_anti_quotes<N: qq_helper>(lo: uint, node: N) -> aq_ctxt
               with *default_visitor()};
     let cx = @{lo:lo, mutable gather: []};
     node.visit(cx, mk_vt(v));
+    // FIXME: Maybe this is an overkill (merge_sort), it might be better
+    //   to just keep the gather array in sorted order ...
+    cx.gather = std::sort::merge_sort({|a,b| a.lo < b.lo}, copy cx.gather);
     ret cx;
 }
 
@@ -200,9 +203,11 @@ fn finish<T: qq_helper>
     let qcx = gather_anti_quotes(sp.lo, node);
     let cx = qcx;
 
-    // assert that the vector is sorted by position:
     uint::range(1u, vec::len(cx.gather)) {|i|
         assert cx.gather[i-1u].lo < cx.gather[i].lo;
+        // ^^ check that the vector is sorted
+        assert cx.gather[i-1u].hi <= cx.gather[i].lo;
+        // ^^ check that the spans are non-overlapping
     }
 
     let str2 = "";

branches/try/src/test/run-pass/qquote.rs

@@ -91,6 +91,12 @@ fn main() {
 
     let crate = #ast(crate) { fn a() { } };
     check_pp(crate, pprust::print_crate_, "fn a() { }\n");
+
+    // issue #1926
+    let s = #ast(expr){__s};
+    let e = #ast(expr){__e};
+    let call = #ast(expr){$(s).foo {|__e| $(e)}};
+    check_pp(call, pprust::print_expr, "__s.foo {|__e| __e }")
 }
 
 fn check_pp<T>(expr: T, f: fn(pprust::ps, T), expect: str) {