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

Bug fix for RT 107921 #3

Open
wants to merge 2 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@openstrike

openstrike commented Oct 22, 2015

Hello,

Here's a tentative fix for the bug reported today. It certainly allows the compilation to proceed and after a manual link the tests pass OK. That said, the tests may not be fully exercising those parts of the lib which have been changed so a bit of manual validation would probably be advisable (by someone more familiar with the module than I am) before merging.

I hope this PR is useful.

Pete

openstrike added some commits Oct 22, 2015

Bug fix for RT 107921
Fixes compilation errors with fprintf and no args by using fputs instead.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment