Skip to content
This repository has been archived by the owner on Sep 24, 2019. It is now read-only.

Allow incomplete AST results. #3

Closed
abargnesi opened this issue Mar 23, 2016 · 0 comments · Fixed by #58
Closed

Allow incomplete AST results. #3

abargnesi opened this issue Mar 23, 2016 · 0 comments · Fixed by #58
Assignees
@nbargnesi
Labels
in progress

Comments

@abargnesi
Copy link
Member

Each parser yields complete AST which precludes evaluation of partial AST.

Ensure each parser is iteratively building AST then yield on ragel parse errors and EOF.
@abargnesi abargnesi self-assigned this Mar 23, 2016
@abargnesi abargnesi mentioned this issue Mar 23, 2016
Closed
abargnesi pushed a commit that referenced this issue Mar 24, 2016
AST when error for identifier, string, and list
0d48dfd 
Error actions were added in order to construct partial results within
the AST. This AST is accessible in the buffers hash with the same name
as when completing without error.

The error state does not capture error metadata.

Added spec tests for identifier, string, and list using property tests.
Rantly was added in to the gem.deps.rb file.

refs #3
@abargnesi abargnesi self-assigned this Mar 25, 2016
@abargnesi abargnesi removed their assignment Mar 29, 2016
@abargnesi abargnesi assigned abargnesi and nbargnesi and unassigned abargnesi Apr 6, 2016
nbargnesi pushed a commit that referenced this issue Apr 13, 2016
squash merge nbargnesi/issue3; refs #3
cdac1e8 
Squashed commit of the following:

commit 2cadfaa
Author: Nick <nick@>
Date:   Wed Apr 13 14:59:38 2016 -0400

    un-xit node specs

commit 8509dec
Author: Nick <nick@>
Date:   Wed Apr 13 14:58:40 2016 -0400

    rev Node class

    - fix original_dup.send invocation depending on class (Node or a
      derivative of?)
    - Add complete accessor
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue Apr 15, 2016
expand v1 syntax spec for OpenBEL#3
d6b48e6
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue Apr 15, 2016
expand OpenBEL#3 to cover BLs, idents, and strings
41483a0 
Added incomplete instance variable to each rl.
@abargnesi abargnesi mentioned this issue Apr 19, 2016
Open
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue Apr 21, 2016
refactor towards OpenBEL#3 and OpenBEL#4
cd5269b
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue Apr 26, 2016
functional list support towards OpenBEL#3 and OpenBEL#4
03c4003 
Need to go back and prune the old rules out.
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue May 16, 2016
expand v1 syntax spec for OpenBEL#3
5897dfc
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue May 16, 2016
expand OpenBEL#3 to cover BLs, idents, and strings
c343082 
Added incomplete instance variable to each rl.
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue May 16, 2016
refactor towards OpenBEL#3 and OpenBEL#4
d975475
nbargnesi pushed a commit to nbargnesi/bel-parser that referenced this issue May 16, 2016
functional list support towards OpenBEL#3 and OpenBEL#4
05893d7 
Need to go back and prune the old rules out.
@nbargnesi nbargnesi mentioned this issue May 16, 2016
Merged
@nbargnesi nbargnesi removed the ragel label May 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nbargnesi nbargnesi

Labels
in progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue3 rebase nbargnesi/bel-parser
2 participants
@abargnesi @nbargnesi