New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

remove global state from parse/print routines #64

Open

mk270 opened this issue Mar 8, 2016 · 1 comment

Assignees

Labels

help requested technical debt

Owner

mk270 commented Mar 8, 2016

placeholder - this will be filled in with proper details

Basically, I think this means making Xxx_Meaning, Yyy_Meaning, Rrr_Meaning and friends all be part of a record that is passed down the call stack, and not much more.

mk270 added the technical debt label

mk270 self-assigned this

mk270 mentioned this issue

make functionality available as a library #3

Closed

Owner Author

mk270 commented Mar 9, 2016

The tests now pass, but we'll need actually to audit the code for the existence of other variables; furthermore parts of the implementation are nasty

mk270 added the help requested label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment