Skip to content
New issue

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

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

Already on GitHub? # to your account

output debug messages to standard error stream #594

Open
petrovr opened this issue Dec 14, 2024 · 1 comment
Open

output debug messages to standard error stream #594

petrovr opened this issue Dec 14, 2024 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@petrovr
Copy link

petrovr commented Dec 14, 2024

Currently macros OQS_*_PRINTF? call function printf(..). Debug message is mixed with regular program output and this break functionality. For instance if output of one program is redirected as input to another one.

Technically this is request is to replace "printf" with "fprintf(stderr," and if needed to include stdio.h header.


In addition oqsprov/oqsprov.c also calls printf once.

@petrovr petrovr added the bug Something isn't working label Dec 14, 2024
@baentsch
Copy link
Member

Agree, this is "sub optimal" when using a Debug build... But I do not see it as a bug but as a nuisance (on my list for improvements in #375... As I don't know yet when/if I or anyone else in the community gets around to doing that, though, please feel free to submit a PR to change this if this would improve things for you).

@baentsch baentsch added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed and removed bug Something isn't working labels Dec 14, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants