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

For beginners: Change TEST_DUMP_MAXSIZE into a run-time limit. #34

Open
mity opened this issue Jan 11, 2020 · 0 comments
Open

For beginners: Change TEST_DUMP_MAXSIZE into a run-time limit. #34

mity opened this issue Jan 11, 2020 · 0 comments
Labels
enhancement good first issue Feasible for new contributors.

Comments

@mity
Copy link
Owner

mity commented Jan 11, 2020

Currently, there is a compile-time constant limiting length of helper messages output via TEST_DUMP macro. It would be good to change it so that people do not have to re-compile of they need to see longer message.

Because Acutest should not output too long things by default, it still should have a limit, but it should be changed into run-time one. I'd suggest that TEST_DUMP_MAXSIZE will be kept but will only determine the default length and user could specify another one by a new command line option, say --max-dump=NUMBER.

(Similar to #33.)

@mity mity added good first issue Feasible for new contributors. enhancement labels Jan 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement good first issue Feasible for new contributors.
Projects
None yet
Development

No branches or pull requests

1 participant