Skip to content

Commit

Permalink
Support Latin-1 supplement a0-ff as DSL string literals (#957)
Browse files Browse the repository at this point in the history
* Support Latin-1 supplement a0-ff as DSL string literals

* build artifacts for previous commit
  • Loading branch information
johnkerl authored Feb 21, 2022
1 parent 0756149 commit 472b42d
Show file tree
Hide file tree
Showing 4 changed files with 20 additions and 9 deletions.
15 changes: 8 additions & 7 deletions internal/pkg/parsing/lexer/lexer.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 8 additions & 0 deletions internal/pkg/parsing/lexer/transitiontable.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 3 additions & 2 deletions internal/pkg/parsing/mlr.bnf
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
// CHARACTER CLASSES
// ----------------------------------------------------------------

_letter : 'a'-'z' | 'A'-'Z' | '\u00a0'-'\u00ff' | '\u0100'-'\U0010FFFF';
_letter : 'a'-'z' | 'A'-'Z' | '\u00a0'-'\u00ff' | '\u0100'-'\U0010ffff';
_decdig : '0'-'9' ;
_hexdig : '0'-'9' | 'a'-'f' | 'A'-'F';
_octdig : '0'-'7' ;
Expand Down Expand Up @@ -95,7 +95,8 @@ _string_literal_element
| ( '\\' '*' )
| ( '\\' '0' ) | ( '\\' '1' ) | ( '\\' '2' ) | ( '\\' '3' ) | ( '\\' '4' )
| ( '\\' '5' ) | ( '\\' '6' ) | ( '\\' '7' ) | ( '\\' '8' ) | ( '\\' '9' )
| '\u0100'-'\U0010FFFF'
| '\u00a0'-'\u00ff'
| '\u0100'-'\U0010ffff'
;
string_literal : '"' {_string_literal_element} '"' ;

Expand Down
1 change: 1 addition & 0 deletions todo.txt
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ RELEASES
https://github.com/johnkerl/miller/issues?q=is%3Aissue+is%3Aopen+label%3Aneeds-documentation
o fmt/unfmt/regex doc
o FAQ/examples reorg
? ??? for doc-string contents search -- ? & mlr help namegoeshere foo ...

k strptime/882
k fmtifnum, & recursive fmtnum/fmtifnum
Expand Down

0 comments on commit 472b42d

Please sign in to comment.