---

yaml
---
r: 2787
b: refs/heads/master
c: c4e9d8f
h: refs/heads/master
i:
  2785: 576dc0c
  2783: fcf0626
v: v3

Author: brson

[refs]

@@ -1,2 +1,2 @@
 ---
-refs/heads/master: 1b29892b5b09f7f69150e1e8eaab2b3c8a12b163
+refs/heads/master: c4e9d8fb911a2e70162c7f61ac70e11c2f604650

trunk/src/comp/front/extfmt.rs

@@ -499,6 +499,7 @@ fn pieces_to_expr(parser p, vec[piece] pieces, vec[@ast::expr] args)
     auto sp = args.(0).span;
     auto n = 0u;
     auto tmp_expr = make_new_str(p, sp, "");
+    auto nargs = vec::len[@ast::expr](args);
 
     for (piece pc in pieces) {
         alt (pc) {
@@ -507,7 +508,7 @@ fn pieces_to_expr(parser p, vec[piece] pieces, vec[@ast::expr] args)
                 tmp_expr = make_add_expr(p, sp, tmp_expr, s_expr);
             }
             case (piece_conv(?conv)) {
-                if (n >= vec::len[@ast::expr](args)) {
+                if (n >= nargs) {
                     log_err "too many conversions in #fmt string";
                     fail;
                 }
@@ -524,6 +525,13 @@ fn pieces_to_expr(parser p, vec[piece] pieces, vec[@ast::expr] args)
         }
     }
 
+    auto expected_nargs = n + 1u; // n conversions + the fmt string
+    if (expected_nargs < nargs) {
+        log_err #fmt("too many arguments to #fmt. found %u, expected %u",
+                     nargs, expected_nargs);
+        fail;
+    }
+
     // TODO: Remove this debug logging
     // log "dumping expanded ast:";
     // log pretty::print_expr(tmp_expr);

trunk/src/test/compile-fail/extfmt-too-many-args.rs

@@ -0,0 +1,8 @@
+// xfail-stage0
+// error-pattern:too many arguments
+
+use std;
+
+fn main() {
+  auto s = #fmt("%s", "test", "test");
+}