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

logmsg: fix compiler warning #1654

Merged
merged 1 commit into from Aug 28, 2017
Merged

Conversation

MrAnno
Copy link
Collaborator

@MrAnno MrAnno commented Aug 28, 2017

In eee3bb4, a g_sprintf() call has been introduced, which is declared only in gprintf.h (because of the stdio dependency).

Note that the functions g_printf(), g_fprintf(), g_sprintf(), g_snprintf(), g_vprintf(), g_vfprintf(), g_vsprintf() and g_vsnprintf() are declared in the header gprintf.h which is not included in glib.h (otherwise using glib.h would drag in stdio.h), so you'll have to explicitly include <glib/gprintf.h> in order to use the GLib printf() functions.

This PR should not be added to the NEWS file.

In eee3bb4, a g_sprintf() call has been
introduced, which is declared only in gprintf.h (because of the stdio dependency).

Signed-off-by: László Várady <laszlo.varady@balabit.com>
@kira-syslogng
Copy link
Contributor

Build SUCCESS, the tests were executed on test branch: master and test suite: functions

@MrAnno
Copy link
Collaborator Author

MrAnno commented Aug 28, 2017

Our master is red, this PR fixes it.

@furiel furiel merged commit e401049 into syslog-ng:master Aug 28, 2017
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

Successfully merging this pull request may close these issues.

None yet

5 participants