---

yaml
---
r: 2788
b: refs/heads/master
c: 29a8015
h: refs/heads/master
v: v3

Author: brson

[refs]

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

trunk/src/comp/front/extfmt.rs

@@ -74,11 +74,12 @@ fn expr_to_str(@ast::expr expr) -> str {
                 case (ast::lit_str(?s)) {
                     ret s;
                 }
+                case (_) { /* fallthrough */ }
             }
         }
+        case (_) { /* fallthrough */ }
     }
-    // FIXME: Handle error correctly.
-    log_err "malformed #fmt call";
+    log_err "first argument to #fmt must be a string literal";
     fail;
 }
 

trunk/src/test/compile-fail/extfmt-non-literal.rs

@@ -0,0 +1,9 @@
+// xfail-stage0
+// error-pattern: literal
+
+fn main() {
+  // #fmt's first argument must be a literal.  Hopefully this
+  // restriction can be eased eventually to just require a
+  // compile-time constant.
+  auto x = #fmt("a" + "b");
+}