Attempt at better modularity of parsing, w/r/t paths #192
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is an attempt to delegate more of the parsing of GFA
path
togfaline.rs
, along with other GFA lines, as opposed toparse.rs
.Because
gfaline::Path
stores references to strings and parsing of paths must be deferred until after steps are parsed,parse.rs
still stores entirepath
lines to be parsed later, but now makes use ofgfaline::parse_line
much in the same way that it already did forlink
s. Parts of the code are now more elegant, parts of them less so, and I think the only thing this really accomplished is satisfying the compiler's desire that all fields ofgfaline::Path
are read from.