diff options
| author | Bruce Hill <bitbucket@bruce-hill.com> | 2018-07-22 14:56:12 -0700 |
|---|---|---|
| committer | Bruce Hill <bitbucket@bruce-hill.com> | 2018-07-22 14:56:39 -0700 |
| commit | 0fe94570b5c33e01472f61fcd564809f93e1b08a (patch) | |
| tree | a9260bc99bdc8acdc3d3ee3a01753c870ebbe9d2 /parser.moon | |
| parent | dfd39f0b14794b360fd6c961c65ab45d229e00b1 (diff) | |
Better handling of invalid line numbers (returning nil instead of random
garbage).
Diffstat (limited to 'parser.moon')
| -rw-r--r-- | parser.moon | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/parser.moon b/parser.moon index 1ffa2d4..cf67205 100644 --- a/parser.moon +++ b/parser.moon @@ -46,17 +46,18 @@ NOMSU_DEFS = with {} err_pos = start_pos line_no = files.get_line_number(src, err_pos) --src = files.read(userdata.source.filename) - prev_line = line_no == 1 and "" or files.get_line(src, line_no-1) + prev_line = line_no == 1 and nil or files.get_line(src, line_no-1) err_line = files.get_line(src, line_no) next_line = files.get_line(src, line_no+1) i = err_pos-files.get_line_starts(src)[line_no] j = i + (end_pos-start_pos) pointer = ("-")\rep(i) .. "^" err_msg = colored.bright colored.yellow colored.onred (err_msg or "Parse error").." at #{userdata.source.filename}:#{line_no}:" - if #prev_line > 0 then err_msg ..= "\n"..colored.dim(prev_line) - err_line = colored.white(err_line\sub(1, i))..colored.bright(colored.red(err_line\sub(i+1,j+1)))..colored.dim(err_line\sub(j+2,-1)) - err_msg ..= "\n#{err_line}\n#{colored.red pointer}" - if #next_line > 0 then err_msg ..= "\n"..colored.dim(next_line) + if prev_line then err_msg ..= "\n"..colored.dim(prev_line) + if err_line + err_line = colored.white(err_line\sub(1, i))..colored.bright(colored.red(err_line\sub(i+1,j+1)))..colored.dim(err_line\sub(j+2,-1)) + err_msg ..= "\n#{err_line}\n#{colored.red pointer}" + if next_line then err_msg ..= "\n"..colored.dim(next_line) seen_errors[start_pos] = err_msg return true @@ -110,10 +111,8 @@ Parser.parse = (nomsu_code, source=nil, version=nil)-> errors: {}, :source, comments: {} } tree = Parser.patterns[syntax_version]\match(nomsu_code, nil, userdata) - unless tree + if not tree or type(tree) == 'number' error "In file #{colored.blue tostring(source or "<unknown>")} failed to parse:\n#{colored.onyellow colored.black nomsu_code}" - if type(tree) == 'number' - return nil if next(userdata.errors) keys = [k for k,v in pairs(userdata.errors)] |