diff options
| author |   Miloslav Trmač <mitr@redhat.com> | 2012-09-20 20:24:00 +0200 |
|---|---|---|
| committer |   Eric Blake <eblake@redhat.com> | 2012-09-21 12:55:13 -0600 |
| commit | fca338a06c54f272f05aa12e362279e113130284 (patch) | |
| tree | 76854ccc24fe3760ceaae9b5d399fa5a4ebf6acb /tests | |
| parent | Remove redundant lines in src/qemu/qemu_driver.c (diff) | |
| download | libvirt-fca338a06c54f272f05aa12e362279e113130284.tar.gz libvirt-fca338a06c54f272f05aa12e362279e113130284.tar.bz2 libvirt-fca338a06c54f272f05aa12e362279e113130284.zip | |
Drop unused return value of virLogOutputFunc
Nothing uses the return value, and creating it requries otherwise
unnecessary strlen () calls.
This cleanup is conceptually independent from the rest of the series
(although the later patches won't apply without it). This just seems
a good opportunity to clean this up, instead of entrenching the unnecessary
return value in the virLogOutputFunc instance that will be added in this
series.
Signed-off-by: Miloslav Trmač <mitr@redhat.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/testutils.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/tests/testutils.c b/tests/testutils.c index 34edde977..b555e0621 100644 --- a/tests/testutils.c +++ b/tests/testutils.c @@ -478,7 +478,7 @@ struct virtTestLogData { static struct virtTestLogData testLog = { VIR_BUFFER_INITIALIZER }; -static int +static void virtTestLogOutput(const char *category ATTRIBUTE_UNUSED, int priority ATTRIBUTE_UNUSED, const char *funcname ATTRIBUTE_UNUSED, @@ -489,9 +489,8 @@ virtTestLogOutput(const char *category ATTRIBUTE_UNUSED, void *data) { struct virtTestLogData *log = data; - virCheckFlags(VIR_LOG_STACK_TRACE, -1); + virCheckFlags(VIR_LOG_STACK_TRACE,); virBufferAsprintf(&log->buf, "%s: %s", timestamp, str); - return strlen(timestamp) + 2 + strlen(str); } static void |