fix parser

compiler/ast/ast.go

@@ -312,28 +312,28 @@ func (f *FString) End() int {
 }
 
 type FStringElem interface {
-	FStringElem()
 	Node
+	FStringElem()
 }
 
 type FStringText struct {
 	Kind    string `json:"kind" unpack:""`
-	TextPos int    `json:"text_pos"`
 	Text    string `json:"text"`
+	TextPos int    `json:"text_pos"`
 }
 
 func (f *FStringText) Pos() int { return f.TextPos }
 func (f *FStringText) End() int { return f.TextPos + len(f.Text) }
 
 type FStringExpr struct {
 	Kind   string `json:"kind" unpack:""`
-	Lbrack int    `json:"lbrack"`
+	Lbrace int    `json:"lbrack"`
 	Expr   Expr   `json:"expr"`
-	Rbrack int    `json:"rbrack"`
+	Rbrace int    `json:"rbrack"`
 }
 
-func (f *FStringExpr) Pos() int { return f.Lbrack }
-func (f *FStringExpr) End() int { return f.Rbrack + 1 }
+func (f *FStringExpr) Pos() int { return f.Lbrace }
+func (f *FStringExpr) End() int { return f.Rbrace + 1 }
 
 func (*FStringText) FStringElem() {}
 func (*FStringExpr) FStringElem() {}

compiler/parser/parser.peg

@@ -1291,8 +1291,8 @@ ComplexType
     }
 
 StringLiteral
-  = '"' v:DoubleQuotedChar* '"' { return newPrimitive(c, "string", joinChars(v), nil }
-  / "'" v:SingleQuotedChar* "'" { return newPrimitive(c, "string", joinChars(V), nil }
+  = '"' v:DoubleQuotedChar* '"' { return newPrimitive(c, "string", joinChars(v)), nil }
+  / "'" v:SingleQuotedChar* "'" { return newPrimitive(c, "string", joinChars(v)), nil }
 
 FString
   = "f\"" v:FStringDoubleQuotedElem* '"' {
@@ -1334,9 +1334,9 @@ FStringExpr
   = "{" __ e:Expr __ "}" {
       return &ast.FStringExpr{
         Kind: "FStringExpr",
-        Lbrack: c.pos.offset, 
+        Lbrace: c.pos.offset, 
         Expr: e.(ast.Expr),
-        Rbrack: lastPos(c, "}"),
+        Rbrace: lastPos(c, "}"),
       }, nil
     }
 

docs/language/expressions.md

@@ -274,7 +274,7 @@ in a string the `$` character must be escaped, i.e., `\$`.
 
 A formatted string literal (or f-string) is a string literal prefixed with `f`.
 These strings may include replacement expressions which are delimited by curly
-braces
+braces:
 ```
 f"{ <expr> }"
 ```

docs/tutorials/zq.md

@@ -929,7 +929,7 @@ DATE                 NUMBER TITLE
 2019-11-12T16:49:07Z PR #6  a few clarifications to the zson spec
 ...
 ```
-Note that we used [formatted string literals](../language/expressions.md#formatted-string-literals)
+Note that we used a [formatted string literal](../language/expressions.md#formatted-string-literals)
 to convert the field `number` into a string and format it with surrounding text.
 
 Instead of old PRs, we can get the latest list of PRs using the

zfmt/ast.go

@@ -208,7 +208,7 @@ func (c *canon) expr(e ast.Expr, parent string) {
 		c.flush()
 		c.write(")")
 	case *ast.FString:
-		c.write("f\"")
+		c.write(`f"`)
 		for _, elem := range e.Elems {
 			switch elem := elem.(type) {
 			case *ast.FStringExpr:
@@ -217,9 +217,11 @@ func (c *canon) expr(e ast.Expr, parent string) {
 				c.write("}")
 			case *ast.FStringText:
 				c.write(elem.Text)
+			default:
+				c.write("(unknown f-strint element %T)", elem)
 			}
 		}
-		c.write("\"")
+		c.write(`"`)
 	default:
 		c.write("(unknown expr %T)", e)
 	}