Skip to content
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.

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

Propagate warnings from C #207

Closed
achaikou opened this issue Dec 9, 2019 · 1 comment
Closed

Propagate warnings from C #207

achaikou opened this issue Dec 9, 2019 · 1 comment

Comments

@achaikou
Copy link
Contributor

achaikou commented Dec 9, 2019

In parse.cpp we have methods "user_warning" and "debug_warning", which are intended to be used when there is a break in specification from the Chapter 3 processing level.
Right now methods are empty, so warnings are swallowed.

We also need consistent model of errors/warnings/infos/debugs to not overload user with a lot of broken, but recoverable, specification (which user probably doesn't really care about and doesn't know what to do with), but do not bury it and give user access to it on request.
Some tests in test_parse would expect these warnings.

@ErlendHaa
Copy link
Contributor

ErlendHaa commented Nov 24, 2020

resolved by #296

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants