[Spice-commits] 5 commits - configure.ac m4/manywarnings.m4 src/channel-base.c src/channel-cursor.c src/channel-display.c src/channel-display-gst.c src/channel-display-mjpeg.c src/channel-main.c src/channel-playback.c src/channel-record.c src/channel-smartcard.c src/decode-glz.c src/spice-channel.c src/spice-client-glib-usb-acl-helper.c src/spice-gstaudio.c src/spice-gtk-session.c src/spice-pulse.c src/spice-session.c src/spice-util.c src/spice-widget.c src/spice-widget-egl.c src/spicy.c src/spicy-screenshot.c src/spicy-stats.c src/usb-device-manager.c src/usbutil.c
Marc-André Lureau
elmarco at kemper.freedesktop.org
Wed Jun 1 11:36:41 UTC 2016
configure.ac | 5 +--
m4/manywarnings.m4 | 39 ++++++++++++++++++++++---
src/channel-base.c | 2 -
src/channel-cursor.c | 2 -
src/channel-display-gst.c | 14 ++++-----
src/channel-display-mjpeg.c | 2 -
src/channel-display.c | 20 ++++++-------
src/channel-main.c | 20 ++++++-------
src/channel-playback.c | 4 +-
src/channel-record.c | 2 -
src/channel-smartcard.c | 8 ++---
src/decode-glz.c | 4 +-
src/spice-channel.c | 50 ++++++++++++++++-----------------
src/spice-client-glib-usb-acl-helper.c | 2 -
src/spice-gstaudio.c | 4 +-
src/spice-gtk-session.c | 6 +--
src/spice-pulse.c | 8 ++---
src/spice-session.c | 4 +-
src/spice-util.c | 2 -
src/spice-widget-egl.c | 5 +--
src/spice-widget.c | 12 +++----
src/spicy-screenshot.c | 4 +-
src/spicy-stats.c | 2 -
src/spicy.c | 2 -
src/usb-device-manager.c | 2 -
src/usbutil.c | 2 -
26 files changed, 128 insertions(+), 99 deletions(-)
New commits:
commit 3b306b1430a001c64392f8cbf7c9a36dfd366193
Author: Francois Gouget <fgouget at codeweavers.com>
Date: Mon May 30 21:37:42 2016 +0200
streaming: Tweak the GStreamer decoder to avoid a compiler warning
We check that there is a matching frame in the queue before popping the
old ones. So we know the inner loop will find a match and thus that
frame will not be NULL. But figuring that out is too hard for the
compiler.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
Acked-by: Pavel Grunt <pgrunt at redhat.com>
Reported-by: Marc-André Lureau <marcandre.lureau at gmail.com>
diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c
index 73fad95..ca6b6e7 100644
--- a/src/channel-display-gst.c
+++ b/src/channel-display-gst.c
@@ -174,7 +174,11 @@ static GstFlowReturn new_sample(GstAppSink *gstappsink, gpointer video_decoder)
while (l) {
frame = l->data;
if (frame->timestamp == GST_BUFFER_PTS(buffer)) {
- /* Now that we know there is a match, remove the older
+ /* The frame is now ready for display */
+ frame->sample = sample;
+ g_queue_push_tail(decoder->display_queue, frame);
+
+ /* Now that we know there is a match, remove it and the older
* frames from the decoding queue.
*/
while ((frame = g_queue_pop_head(decoder->decoding_queue))) {
@@ -187,10 +191,6 @@ static GstFlowReturn new_sample(GstAppSink *gstappsink, gpointer video_decoder)
SPICE_DEBUG("the GStreamer pipeline dropped a frame");
free_frame(frame);
}
-
- /* The frame is now ready for display */
- frame->sample = sample;
- g_queue_push_tail(decoder->display_queue, frame);
break;
}
l = l->next;
commit da8ecf1f95382e8c12fd14915f471a3e76e4b178
Author: Marc-André Lureau <marcandre.lureau at gmail.com>
Date: Mon May 30 19:28:20 2016 +0200
build-sys: update manywarnings.m4
Signed-off-by: Marc-André Lureau <marcandre.lureau at gmail.com>
Acked-by: Pavel Grunt <pgrunt at redhat.com>
diff --git a/m4/manywarnings.m4 b/m4/manywarnings.m4
index 3e6dd21..90823b0 100644
--- a/m4/manywarnings.m4
+++ b/m4/manywarnings.m4
@@ -1,5 +1,5 @@
-# manywarnings.m4 serial 7
-dnl Copyright (C) 2008-2014 Free Software Foundation, Inc.
+# manywarnings.m4 serial 8
+dnl Copyright (C) 2008-2016 Free Software Foundation, Inc.
dnl This file is free software; the Free Software Foundation
dnl gives unlimited permission to copy and/or distribute it,
dnl with or without modifications, as long as this notice is preserved.
@@ -108,12 +108,13 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
-Waddress \
-Waggressive-loop-optimizations \
-Wall \
- -Warray-bounds \
-Wattributes \
-Wbad-function-cast \
+ -Wbool-compare \
-Wbuiltin-macro-redefined \
-Wcast-align \
-Wchar-subscripts \
+ -Wchkp \
-Wclobbered \
-Wcomment \
-Wcomments \
@@ -122,9 +123,13 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
-Wdate-time \
-Wdeprecated \
-Wdeprecated-declarations \
+ -Wdesignated-init \
-Wdisabled-optimization \
+ -Wdiscarded-array-qualifiers \
+ -Wdiscarded-qualifiers \
-Wdiv-by-zero \
-Wdouble-promotion \
+ -Wduplicated-cond \
-Wempty-body \
-Wendif-labels \
-Wenum-compare \
@@ -133,22 +138,31 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
-Wformat-extra-args \
-Wformat-nonliteral \
-Wformat-security \
+ -Wformat-signedness \
-Wformat-y2k \
-Wformat-zero-length \
+ -Wframe-address \
-Wfree-nonheap-object \
+ -Whsa \
+ -Wignored-attributes \
-Wignored-qualifiers \
-Wimplicit \
-Wimplicit-function-declaration \
-Wimplicit-int \
+ -Wincompatible-pointer-types \
-Winit-self \
-Winline \
+ -Wint-conversion \
-Wint-to-pointer-cast \
-Winvalid-memory-model \
-Winvalid-pch \
-Wjump-misses-init \
+ -Wlogical-not-parentheses \
-Wlogical-op \
-Wmain \
-Wmaybe-uninitialized \
+ -Wmemset-transposed-args \
+ -Wmisleading-indentation \
-Wmissing-braces \
-Wmissing-declarations \
-Wmissing-field-initializers \
@@ -159,6 +173,9 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
-Wnarrowing \
-Wnested-externs \
-Wnonnull \
+ -Wnonnull-compare \
+ -Wnull-dereference \
+ -Wodr \
-Wold-style-declaration \
-Wold-style-definition \
-Wopenmp-simd \
@@ -174,8 +191,13 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
-Wpragmas \
-Wreturn-local-addr \
-Wreturn-type \
+ -Wscalar-storage-order \
-Wsequence-point \
-Wshadow \
+ -Wshift-count-negative \
+ -Wshift-count-overflow \
+ -Wshift-negative-value \
+ -Wsizeof-array-argument \
-Wsizeof-pointer-memaccess \
-Wstack-protector \
-Wstrict-aliasing \
@@ -185,10 +207,14 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
-Wsuggest-attribute=format \
-Wsuggest-attribute=noreturn \
-Wsuggest-attribute=pure \
+ -Wsuggest-final-methods \
+ -Wsuggest-final-types \
-Wswitch \
+ -Wswitch-bool \
-Wswitch-default \
-Wsync-nand \
-Wsystem-headers \
+ -Wtautological-compare \
-Wtrampolines \
-Wtrigraphs \
-Wtype-limits \
@@ -217,9 +243,12 @@ AC_DEFUN([gl_MANYWARN_ALL_GCC],
gl_manywarn_set="$gl_manywarn_set $gl_manywarn_item"
done
- # gcc --help=warnings outputs an unusual form for this option; list
- # it here so that the above 'comm' command doesn't report a false match.
+ # gcc --help=warnings outputs an unusual form for these options; list
+ # them here so that the above 'comm' command doesn't report a false match.
+ gl_manywarn_set="$gl_manywarn_set -Warray-bounds=2"
gl_manywarn_set="$gl_manywarn_set -Wnormalized=nfc"
+ gl_manywarn_set="$gl_manywarn_set -Wshift-overflow=2"
+ gl_manywarn_set="$gl_manywarn_set -Wunused-const-variable=2"
# These are needed for older GCC versions.
if test -n "$GCC"; then
commit b32398ef4d141d37498915c4be27ef86c99dff7b
Author: Marc-André Lureau <marcandre.lureau at gmail.com>
Date: Mon May 30 19:30:14 2016 +0200
build-sys: -Wshift-overflow=2
manywarnings.m4 update will bring new flags that fail with some
glib/gst headers.
Signed-off-by: Marc-André Lureau <marcandre.lureau at gmail.com>
Acked-by: Pavel Grunt <pgrunt at redhat.com>
diff --git a/configure.ac b/configure.ac
index 8f1e91b..3fe8055 100644
--- a/configure.ac
+++ b/configure.ac
@@ -581,7 +581,10 @@ SPICE_CHECK_LZ4
dnl ===========================================================================
dnl check compiler flags
-SPICE_COMPILE_WARNINGS
+# some glib/gstreamer enums use 1 << 31
+dontwarn="-Wshift-overflow=2"
+
+SPICE_COMPILE_WARNINGS([$dontwarn])
SPICE_CFLAGS="$SPICE_CFLAGS $WARN_CFLAGS"
commit 48bbcbbc4936b12c92f6e4dbea7d42c073eba4e4
Author: Marc-André Lureau <marcandre.lureau at gmail.com>
Date: Mon May 30 19:29:24 2016 +0200
Fix many -Werror=format warnings
Signed-off-by: Marc-André Lureau <marcandre.lureau at gmail.com>
Acked-by: Pavel Grunt <pgrunt at redhat.com>
diff --git a/src/channel-base.c b/src/channel-base.c
index 20bb0cc..004dba9 100644
--- a/src/channel-base.c
+++ b/src/channel-base.c
@@ -77,7 +77,7 @@ spice_channel_handle_notify(SpiceChannel *channel, SpiceMsgIn *in)
CHANNEL_DEBUG(channel, "%s -- %s%s #%u%s%.*s", __FUNCTION__,
severity, visibility, notify->what,
- message_str ? ": " : "", notify->message_len,
+ message_str ? ": " : "", (int)notify->message_len,
message_str ? message_str : "");
}
diff --git a/src/channel-cursor.c b/src/channel-cursor.c
index 2ae8f86..609243b 100644
--- a/src/channel-cursor.c
+++ b/src/channel-cursor.c
@@ -304,7 +304,7 @@ static display_cursor *set_cursor(SpiceChannel *channel, SpiceCursor *scursor)
guint8 *rgba;
guint8 val;
- CHANNEL_DEBUG(channel, "%s: flags %d, size %d", __FUNCTION__,
+ CHANNEL_DEBUG(channel, "%s: flags %x, size %u", __FUNCTION__,
scursor->flags, scursor->data_size);
if (scursor->flags & SPICE_CURSOR_FLAGS_NONE)
diff --git a/src/channel-display-gst.c b/src/channel-display-gst.c
index 46a85ea..73fad95 100644
--- a/src/channel-display-gst.c
+++ b/src/channel-display-gst.c
@@ -366,7 +366,7 @@ static void spice_gst_decoder_queue_frame(VideoDecoder *video_decoder,
SpiceStreamDataHeader *frame_op = spice_msg_in_parsed(frame_msg);
if (frame_op->multi_media_time < decoder->last_mm_time) {
SPICE_DEBUG("new-frame-time < last-frame-time (%u < %u):"
- " resetting stream, id %d",
+ " resetting stream, id %u",
frame_op->multi_media_time,
decoder->last_mm_time, frame_op->id);
/* Let GStreamer deal with the frame anyway */
@@ -402,7 +402,7 @@ static void spice_gst_decoder_queue_frame(VideoDecoder *video_decoder,
g_mutex_unlock(&decoder->queues_mutex);
if (gst_app_src_push_buffer(decoder->appsrc, buffer) != GST_FLOW_OK) {
- SPICE_DEBUG("GStreamer error: unable to push frame of size %d", size);
+ SPICE_DEBUG("GStreamer error: unable to push frame of size %u", size);
stream_dropped_frame_on_playback(decoder->base.stream);
}
}
diff --git a/src/channel-display-mjpeg.c b/src/channel-display-mjpeg.c
index 1238b41..a2dae82 100644
--- a/src/channel-display-mjpeg.c
+++ b/src/channel-display-mjpeg.c
@@ -250,7 +250,7 @@ static void mjpeg_decoder_queue_frame(VideoDecoder *video_decoder,
if (frame_op->multi_media_time < last_op->multi_media_time) {
/* This should really not happen */
SPICE_DEBUG("new-frame-time < last-frame-time (%u < %u):"
- " resetting stream, id %d",
+ " resetting stream, id %u",
frame_op->multi_media_time,
last_op->multi_media_time, frame_op->id);
mjpeg_decoder_drop_queue(decoder);
diff --git a/src/channel-display.c b/src/channel-display.c
index a1ed493..54bc30e 100644
--- a/src/channel-display.c
+++ b/src/channel-display.c
@@ -1086,7 +1086,7 @@ static void display_handle_stream_create(SpiceChannel *channel, SpiceMsgIn *in)
SpiceMsgDisplayStreamCreate *op = spice_msg_in_parsed(in);
display_stream *st;
- CHANNEL_DEBUG(channel, "%s: id %d", __FUNCTION__, op->id);
+ CHANNEL_DEBUG(channel, "%s: id %u", __FUNCTION__, op->id);
if (op->id >= c->nstreams) {
int n = c->nstreams;
@@ -1127,7 +1127,7 @@ static void display_handle_stream_create(SpiceChannel *channel, SpiceMsgIn *in)
#endif
}
if (st->video_decoder == NULL) {
- spice_printerr("could not create a video decoder for codec %d", op->codec_type);
+ spice_printerr("could not create a video decoder for codec %u", op->codec_type);
}
}
@@ -1349,7 +1349,7 @@ static void display_session_mm_time_reset_cb(SpiceSession *session, gpointer dat
if (c->streams[i] == NULL) {
continue;
}
- SPICE_DEBUG("%s: stream-id %d", __FUNCTION__, i);
+ SPICE_DEBUG("%s: stream-id %u", __FUNCTION__, i);
st = c->streams[i];
st->video_decoder->reschedule(st->video_decoder);
}
@@ -1374,7 +1374,7 @@ static void display_handle_stream_data(SpiceChannel *channel, SpiceMsgIn *in)
mmtime = stream_get_time(st);
if (spice_msg_in_type(in) == SPICE_MSG_DISPLAY_STREAM_DATA_SIZED) {
- CHANNEL_DEBUG(channel, "stream %d contains sized data", op->id);
+ CHANNEL_DEBUG(channel, "stream %u contains sized data", op->id);
}
if (op->multi_media_time == 0) {
@@ -1465,9 +1465,9 @@ static void destroy_stream(SpiceChannel *channel, int id)
return;
num_out_frames = st->num_input_frames - st->arrive_late_count - st->num_drops_on_playback;
- CHANNEL_DEBUG(channel, "%s: id=%d #in-frames=%d out/in=%.2f "
- "#drops-on-receive=%d avg-late-time(ms)=%.2f "
- "#drops-on-playback=%d", __FUNCTION__,
+ CHANNEL_DEBUG(channel, "%s: id=%d #in-frames=%u out/in=%.2f "
+ "#drops-on-receive=%u avg-late-time(ms)=%.2f "
+ "#drops-on-playback=%u", __FUNCTION__,
id,
st->num_input_frames,
num_out_frames / (double)st->num_input_frames,
@@ -1523,7 +1523,7 @@ static void display_handle_stream_destroy(SpiceChannel *channel, SpiceMsgIn *in)
SpiceMsgDisplayStreamDestroy *op = spice_msg_in_parsed(in);
g_return_if_fail(op != NULL);
- CHANNEL_DEBUG(channel, "%s: id %d", __FUNCTION__, op->id);
+ CHANNEL_DEBUG(channel, "%s: id %u", __FUNCTION__, op->id);
destroy_stream(channel, op->id);
}
@@ -1665,7 +1665,7 @@ static void display_handle_surface_create(SpiceChannel *channel, SpiceMsgIn *in)
surface->size = surface->height * surface->stride;
if (create->flags & SPICE_SURFACE_FLAGS_PRIMARY) {
- SPICE_DEBUG("primary flags: %d", create->flags);
+ SPICE_DEBUG("primary flags: %x", create->flags);
surface->primary = true;
create_canvas(channel, surface);
if (c->mark_false_event_id != 0) {
@@ -1800,7 +1800,7 @@ static void display_handle_gl_draw(SpiceChannel *channel, SpiceMsgIn *in)
{
SpiceMsgDisplayGlDraw *draw = spice_msg_in_parsed(in);
- CHANNEL_DEBUG(channel, "gl draw %dx%d+%d+%d",
+ CHANNEL_DEBUG(channel, "gl draw %ux%u+%u+%u",
draw->w, draw->h, draw->x, draw->y);
g_coroutine_signal_emit(channel, signals[SPICE_DISPLAY_GL_DRAW], 0,
diff --git a/src/channel-main.c b/src/channel-main.c
index 2f29312..efe7f4e 100644
--- a/src/channel-main.c
+++ b/src/channel-main.c
@@ -1108,7 +1108,7 @@ static void monitors_align(VDAgentMonConfig *monitors, int nmonitors)
monitors[j].y = 0;
x += monitors[j].width;
if (monitors[j].width || monitors[j].height)
- SPICE_DEBUG("#%d +%d+%d-%dx%d", j, monitors[j].x, monitors[j].y,
+ SPICE_DEBUG("#%d +%d+%d-%ux%u", j, monitors[j].x, monitors[j].y,
monitors[j].width, monitors[j].height);
}
g_free(sorted_monitors);
@@ -1171,7 +1171,7 @@ gboolean spice_main_send_monitor_config(SpiceMainChannel *channel)
mon->monitors[j].height = c->display[i].height;
mon->monitors[j].x = c->display[i].x;
mon->monitors[j].y = c->display[i].y;
- CHANNEL_DEBUG(channel, "monitor #%d: %dx%d+%d+%d @ %d bpp", j,
+ CHANNEL_DEBUG(channel, "monitor #%d: %ux%u+%d+%d @ %u bpp", j,
mon->monitors[j].width, mon->monitors[j].height,
mon->monitors[j].x, mon->monitors[j].y,
mon->monitors[j].depth);
@@ -1967,7 +1967,7 @@ static void spice_file_transfer_task_handle_status(SpiceFileTransferTask *task,
GError *error = NULL;
g_return_if_fail(task != NULL);
- SPICE_DEBUG("task %d received response %d", msg->id, msg->result);
+ SPICE_DEBUG("task %u received response %u", msg->id, msg->result);
switch (msg->result) {
case VD_AGENT_FILE_XFER_STATUS_CAN_SEND_DATA:
@@ -2137,7 +2137,7 @@ static void main_agent_handle_msg(SpiceChannel *channel,
case VD_AGENT_REPLY:
{
VDAgentReply *reply = payload;
- SPICE_DEBUG("%s: reply: type %d, %s", __FUNCTION__, reply->type,
+ SPICE_DEBUG("%s: reply: type %u, %s", __FUNCTION__, reply->type,
reply->error == VD_AGENT_SUCCESS ? "success" : "error");
break;
}
@@ -2150,7 +2150,7 @@ static void main_agent_handle_msg(SpiceChannel *channel,
if (task != NULL) {
spice_file_transfer_task_handle_status(task, msg);
} else {
- SPICE_DEBUG("cannot find task %d", msg->id);
+ SPICE_DEBUG("cannot find task %u", msg->id);
}
break;
}
@@ -2173,7 +2173,7 @@ static void main_handle_agent_data_msg(SpiceChannel* channel, int* msg_size, guc
*msg_size -= n;
*msg_pos += n;
if (c->agent_msg_pos == sizeof(VDAgentMessage)) {
- SPICE_DEBUG("agent msg start: msg_size=%d, protocol=%d, type=%d",
+ SPICE_DEBUG("agent msg start: msg_size=%u, protocol=%u, type=%u",
c->agent_msg.size, c->agent_msg.protocol, c->agent_msg.type);
g_return_if_fail(c->agent_msg_data == NULL);
c->agent_msg_data = g_malloc0(c->agent_msg.size);
@@ -2314,12 +2314,12 @@ static void migrate_channel_event_cb(SpiceChannel *channel, SpiceChannelEvent ev
mig->nchannels--;
}
- SPICE_DEBUG("migration: channel opened chan:%p, left %d", channel, mig->nchannels);
+ SPICE_DEBUG("migration: channel opened chan:%p, left %u", channel, mig->nchannels);
if (mig->nchannels == 0)
coroutine_yieldto(mig->from, NULL);
break;
default:
- CHANNEL_DEBUG(channel, "error or unhandled channel event during migration: %d", event);
+ CHANNEL_DEBUG(channel, "error or unhandled channel event during migration: %u", event);
/* go back to main channel to report error */
coroutine_yieldto(mig->from, NULL);
}
@@ -2378,7 +2378,7 @@ static gboolean migrate_connect(gpointer data)
host = info->host;
} else {
SpiceMigrationDstInfo *info = mig->info;
- SPICE_DEBUG("migrate_begin %d %s %d %d",
+ SPICE_DEBUG("migrate_begin %u %s %d %d",
info->host_size, info->host_data, info->port, info->sport);
port = info->port;
sport = info->sport;
@@ -3109,7 +3109,7 @@ static void file_xfer_send_start_msg_async(SpiceMainChannel *channel,
task->callback = callback;
task->user_data = user_data;
- CHANNEL_DEBUG(channel, "Insert a xfer task:%d to task list", task->id);
+ CHANNEL_DEBUG(channel, "Insert a xfer task:%u to task list", task->id);
g_hash_table_insert(c->file_xfer_tasks,
GUINT_TO_POINTER(task->id),
task);
diff --git a/src/channel-playback.c b/src/channel-playback.c
index 8edcb22..39c2178 100644
--- a/src/channel-playback.c
+++ b/src/channel-playback.c
@@ -346,7 +346,7 @@ static void playback_handle_mode(SpiceChannel *channel, SpiceMsgIn *in)
SpicePlaybackChannelPrivate *c = SPICE_PLAYBACK_CHANNEL(channel)->priv;
SpiceMsgPlaybackMode *mode = spice_msg_in_parsed(in);
- CHANNEL_DEBUG(channel, "%s: time %u mode %d data %p size %d", __FUNCTION__,
+ CHANNEL_DEBUG(channel, "%s: time %u mode %u data %p size %u", __FUNCTION__,
mode->time, mode->mode, mode->data, mode->data_size);
c->mode = mode->mode;
@@ -367,7 +367,7 @@ static void playback_handle_start(SpiceChannel *channel, SpiceMsgIn *in)
SpicePlaybackChannelPrivate *c = SPICE_PLAYBACK_CHANNEL(channel)->priv;
SpiceMsgPlaybackStart *start = spice_msg_in_parsed(in);
- CHANNEL_DEBUG(channel, "%s: fmt %d channels %d freq %d time %d", __FUNCTION__,
+ CHANNEL_DEBUG(channel, "%s: fmt %u channels %u freq %u time %u", __FUNCTION__,
start->format, start->channels, start->frequency, start->time);
c->frame_count = 0;
diff --git a/src/channel-record.c b/src/channel-record.c
index 09598b7..c1d9afa 100644
--- a/src/channel-record.c
+++ b/src/channel-record.c
@@ -405,7 +405,7 @@ static void record_handle_start(SpiceChannel *channel, SpiceMsgIn *in)
c->mode = spice_record_desired_mode(channel, start->frequency);
- CHANNEL_DEBUG(channel, "%s: fmt %d channels %d freq %d", __FUNCTION__,
+ CHANNEL_DEBUG(channel, "%s: fmt %u channels %u freq %u", __FUNCTION__,
start->format, start->channels, start->frequency);
g_return_if_fail(start->format == SPICE_AUDIO_FMT_S16);
diff --git a/src/channel-smartcard.c b/src/channel-smartcard.c
index 09e784d..e086e92 100644
--- a/src/channel-smartcard.c
+++ b/src/channel-smartcard.c
@@ -316,7 +316,7 @@ static void smartcard_message_send(SpiceSmartcardChannel *channel,
if (spice_channel_get_read_only(SPICE_CHANNEL(channel)))
return;
- CHANNEL_DEBUG(channel, "send message %d, %s",
+ CHANNEL_DEBUG(channel, "send message %u, %s",
msg_type, queue ? "queued" : "now");
if (!queue) {
spice_msg_out_send(msg_out);
@@ -496,11 +496,11 @@ static void handle_smartcard_msg(SpiceChannel *channel, SpiceMsgIn *in)
SpiceMsgSmartcard *msg = spice_msg_in_parsed(in);
VReader *reader;
- CHANNEL_DEBUG(channel, "handle msg %d", msg->type);
+ CHANNEL_DEBUG(channel, "handle msg %u", msg->type);
switch (msg->type) {
case VSC_Error:
g_return_if_fail(priv->in_flight_message != NULL);
- CHANNEL_DEBUG(channel, "in flight %d", priv->in_flight_message->message_type);
+ CHANNEL_DEBUG(channel, "in flight %u", priv->in_flight_message->message_type);
switch (priv->in_flight_message->message_type) {
case VSC_ReaderAdd:
g_return_if_fail(priv->pending_reader_additions != NULL);
@@ -529,7 +529,7 @@ static void handle_smartcard_msg(SpiceChannel *channel, SpiceMsgIn *in)
case VSC_ReaderRemove:
break;
default:
- g_warning("Unexpected message: %d", priv->in_flight_message->message_type);
+ g_warning("Unexpected message: %u", priv->in_flight_message->message_type);
break;
}
smartcard_message_complete_in_flight(smartcard_channel);
diff --git a/src/decode-glz.c b/src/decode-glz.c
index b7dd2e6..d5b72ab 100644
--- a/src/decode-glz.c
+++ b/src/decode-glz.c
@@ -92,7 +92,7 @@ static void glz_decoder_window_resize(SpiceGlzDecoderWindow *w)
struct glz_image **new_images;
int i, new_slot;
- SPICE_DEBUG("%s: array resize %d -> %d", __FUNCTION__,
+ SPICE_DEBUG("%s: array resize %u -> %u", __FUNCTION__,
w->nimages, w->nimages * 2);
new_images = g_new0(struct glz_image*, w->nimages * 2);
for (i = 0; i < w->nimages; i++) {
@@ -367,7 +367,7 @@ static void decode_header(GlibGlzDecoder *d)
d->image.id = decode_64(d);
d->image.win_head_dist = decode_32(d);
- SPICE_DEBUG("%s: %dx%d, id %" PRId64 ", ref %" PRId64,
+ SPICE_DEBUG("%s: %ux%u, id %" PRIu64 ", ref %" PRIu64,
__FUNCTION__,
d->image.width, d->image.height, d->image.id,
d->image.id - d->image.win_head_dist);
diff --git a/src/spice-channel.c b/src/spice-channel.c
index 61ab15c..c6e548d 100644
--- a/src/spice-channel.c
+++ b/src/spice-channel.c
@@ -568,7 +568,7 @@ void spice_msg_in_hexdump(SpiceMsgIn *in)
{
SpiceChannelPrivate *c = in->channel->priv;
- fprintf(stderr, "--\n<< hdr: %s serial %" PRIu64 " type %d size %d sub-list %d\n",
+ fprintf(stderr, "--\n<< hdr: %s serial %" PRIu64 " type %u size %u sub-list %u\n",
c->name, spice_header_get_in_msg_serial(in),
spice_header_get_msg_type(in->header, c->use_mini_header),
spice_header_get_msg_size(in->header, c->use_mini_header),
@@ -581,7 +581,7 @@ void spice_msg_out_hexdump(SpiceMsgOut *out, unsigned char *data, int len)
{
SpiceChannelPrivate *c = out->channel->priv;
- fprintf(stderr, "--\n>> hdr: %s serial %" PRIu64 " type %d size %d sub-list %d\n",
+ fprintf(stderr, "--\n>> hdr: %s serial %" PRIu64 " type %u size %u sub-list %u\n",
c->name,
spice_header_get_out_msg_serial(out),
spice_header_get_msg_type(out->header, c->use_mini_header),
@@ -1185,7 +1185,7 @@ static gboolean spice_channel_recv_auth(SpiceChannel *channel)
}
if (link_res != SPICE_LINK_ERR_OK) {
- CHANNEL_DEBUG(channel, "link result: reply %d", link_res);
+ CHANNEL_DEBUG(channel, "link result: reply %u", link_res);
spice_channel_failed_authentication(channel, FALSE);
return FALSE;
}
@@ -1209,7 +1209,7 @@ void spice_channel_up(SpiceChannel *channel)
{
SpiceChannelPrivate *c = channel->priv;
- CHANNEL_DEBUG(channel, "channel up, state %d", c->state);
+ CHANNEL_DEBUG(channel, "channel up, state %u", c->state);
if (SPICE_CHANNEL_GET_CLASS(channel)->channel_up)
SPICE_CHANNEL_GET_CLASS(channel)->channel_up(channel);
@@ -1272,7 +1272,7 @@ static void spice_channel_send_link(SpiceChannel *channel)
*(uint32_t *)p = GUINT32_TO_LE(g_array_index(c->caps, uint32_t, i));
p += sizeof(uint32_t);
}
- CHANNEL_DEBUG(channel, "channel type %d id %d num common caps %d num caps %d",
+ CHANNEL_DEBUG(channel, "channel type %d id %d num common caps %u num caps %u",
c->channel_type,
c->channel_id,
c->common_caps->len,
@@ -1298,11 +1298,11 @@ static gboolean spice_channel_recv_link_hdr(SpiceChannel *channel)
goto error;
}
- CHANNEL_DEBUG(channel, "Peer version: %d:%d",
+ CHANNEL_DEBUG(channel, "Peer version: %u:%u",
GUINT32_FROM_LE(c->peer_hdr.major_version),
GUINT32_FROM_LE(c->peer_hdr.minor_version));
if (c->peer_hdr.major_version != c->link_hdr.major_version) {
- g_warning("major mismatch (got %d, expected %d)",
+ g_warning("major mismatch (got %u, expected %u)",
c->peer_hdr.major_version, c->link_hdr.major_version);
goto error;
}
@@ -1567,7 +1567,7 @@ static gboolean spice_channel_perform_auth_sasl(SpiceChannel *channel)
if (c->has_error)
goto error;
if (len > SASL_MAX_MECHLIST_LEN) {
- g_critical("mechlistlen %d too long", len);
+ g_critical("mechlistlen %u too long", len);
goto error;
}
@@ -1602,11 +1602,11 @@ restart:
goto restart;
}
- CHANNEL_DEBUG(channel, "Server start negotiation with mech %s. Data %d bytes %p '%s'",
+ CHANNEL_DEBUG(channel, "Server start negotiation with mech %s. Data %u bytes %p '%s'",
mechname, clientoutlen, clientout, clientout);
if (clientoutlen > SASL_MAX_DATA_LEN) {
- g_critical("SASL negotiation data too long: %d bytes",
+ g_critical("SASL negotiation data too long: %u bytes",
clientoutlen);
goto error;
}
@@ -1635,7 +1635,7 @@ restart:
if (c->has_error)
goto error;
if (len > SASL_MAX_DATA_LEN) {
- g_critical("SASL negotiation data too long: %d bytes",
+ g_critical("SASL negotiation data too long: %u bytes",
len);
goto error;
}
@@ -1653,7 +1653,7 @@ restart:
if (c->has_error)
goto error;
- CHANNEL_DEBUG(channel, "Client start result complete: %d. Data %d bytes %p '%s'",
+ CHANNEL_DEBUG(channel, "Client start result complete: %d. Data %u bytes %p '%s'",
complete, len, serverin, serverin);
/* Loop-the-loop...
@@ -1688,7 +1688,7 @@ restart:
g_clear_pointer(&serverin, g_free);
- CHANNEL_DEBUG(channel, "Client step result %d. Data %d bytes %p '%s'", err, clientoutlen, clientout, clientout);
+ CHANNEL_DEBUG(channel, "Client step result %d. Data %u bytes %p '%s'", err, clientoutlen, clientout, clientout);
/* Previous server call showed completion & we're now locally complete too */
if (complete && err == SASL_OK)
@@ -1709,13 +1709,13 @@ restart:
if (c->has_error)
goto error;
- CHANNEL_DEBUG(channel, "Server step with %d bytes %p", clientoutlen, clientout);
+ CHANNEL_DEBUG(channel, "Server step with %u bytes %p", clientoutlen, clientout);
spice_channel_read(channel, &len, sizeof(guint32));
if (c->has_error)
goto error;
if (len > SASL_MAX_DATA_LEN) {
- g_critical("SASL negotiation data too long: %d bytes", len);
+ g_critical("SASL negotiation data too long: %u bytes", len);
goto error;
}
@@ -1733,7 +1733,7 @@ restart:
if (c->has_error)
goto error;
- CHANNEL_DEBUG(channel, "Client step result complete: %d. Data %d bytes %p '%s'",
+ CHANNEL_DEBUG(channel, "Client step result complete: %d. Data %u bytes %p '%s'",
complete, len, serverin, serverin);
/* This server call shows complete, and earlier client step was OK */
@@ -1754,9 +1754,9 @@ restart:
goto error;
}
ssf = *(const int *)val;
- CHANNEL_DEBUG(channel, "SASL SSF value %d", ssf);
+ CHANNEL_DEBUG(channel, "SASL SSF value %u", ssf);
if (ssf < 56) { /* 56 == DES level, good for Kerberos */
- g_critical("negotiation SSF %d was not strong enough", ssf);
+ g_critical("negotiation SSF %u was not strong enough", ssf);
goto error;
}
}
@@ -1807,7 +1807,7 @@ static gboolean spice_channel_recv_link_msg(SpiceChannel *channel)
c->peer_hdr.size - c->peer_pos);
c->peer_pos += rc;
if (c->peer_pos != c->peer_hdr.size) {
- g_critical("%s: %s: incomplete link reply (%d/%d)",
+ g_critical("%s: %s: incomplete link reply (%d/%u)",
c->name, __FUNCTION__, rc, c->peer_hdr.size);
goto error;
}
@@ -1821,7 +1821,7 @@ static gboolean spice_channel_recv_link_msg(SpiceChannel *channel)
c->tls = TRUE;
return FALSE;
default:
- g_warning("%s: %s: unhandled error %d",
+ g_warning("%s: %s: unhandled error %u",
c->name, __FUNCTION__, c->peer_msg->error);
goto error;
}
@@ -1840,13 +1840,13 @@ static gboolean spice_channel_recv_link_msg(SpiceChannel *channel)
g_array_set_size(c->remote_common_caps, num_common_caps);
for (i = 0; i < num_common_caps; i++, caps++) {
g_array_index(c->remote_common_caps, uint32_t, i) = GUINT32_FROM_LE(*caps);
- CHANNEL_DEBUG(channel, "got common caps %u:0x%X", i, GUINT32_FROM_LE(*caps));
+ CHANNEL_DEBUG(channel, "got common caps %d:0x%X", i, GUINT32_FROM_LE(*caps));
}
g_array_set_size(c->remote_caps, num_channel_caps);
for (i = 0; i < num_channel_caps; i++, caps++) {
g_array_index(c->remote_caps, uint32_t, i) = GUINT32_FROM_LE(*caps);
- CHANNEL_DEBUG(channel, "got channel caps %u:0x%X", i, GUINT32_FROM_LE(*caps));
+ CHANNEL_DEBUG(channel, "got channel caps %d:0x%X", i, GUINT32_FROM_LE(*caps));
}
if (!spice_channel_test_common_capability(channel,
@@ -2683,7 +2683,7 @@ gboolean spice_channel_open_fd(SpiceChannel *channel, int fd)
c = channel->priv;
if (c->state > SPICE_CHANNEL_STATE_CONNECTING) {
- g_warning("Invalid channel_connect state: %d", c->state);
+ g_warning("Invalid channel_connect state: %u", c->state);
return true;
}
@@ -2772,7 +2772,7 @@ void spice_channel_disconnect(SpiceChannel *channel, SpiceChannelEvent reason)
{
SpiceChannelPrivate *c;
- CHANNEL_DEBUG(channel, "channel disconnect %d", reason);
+ CHANNEL_DEBUG(channel, "channel disconnect %u", reason);
g_return_if_fail(SPICE_IS_CHANNEL(channel));
g_return_if_fail(channel->priv != NULL);
@@ -2810,7 +2810,7 @@ static gboolean test_capability(GArray *caps, guint32 cap)
c = g_array_index(caps, guint32, word_index);
ret = (c & (1 << (cap % 32))) != 0;
- SPICE_DEBUG("test cap %d in 0x%X: %s", cap, c, ret ? "yes" : "no");
+ SPICE_DEBUG("test cap %u in 0x%X: %s", cap, c, ret ? "yes" : "no");
return ret;
}
diff --git a/src/spice-client-glib-usb-acl-helper.c b/src/spice-client-glib-usb-acl-helper.c
index 03a0317..80cdced 100644
--- a/src/spice-client-glib-usb-acl-helper.c
+++ b/src/spice-client-glib-usb-acl-helper.c
@@ -286,7 +286,7 @@ static void stdin_read_complete(GObject *src, GAsyncResult *res, gpointer data)
NULL, stdin_read_complete, NULL);
break;
default:
- FATAL_ERROR("Unexpected extra input in state %d: %s\n", state, s);
+ FATAL_ERROR("Unexpected extra input in state %u: %s\n", state, s);
}
g_free(s);
}
diff --git a/src/spice-gstaudio.c b/src/spice-gstaudio.c
index 12e6d66..1d96c77 100644
--- a/src/spice-gstaudio.c
+++ b/src/spice-gstaudio.c
@@ -395,7 +395,7 @@ static void playback_mute_changed(GObject *object, GParamSpec *pspec, gpointer d
return;
g_object_get(object, "mute", &mute, NULL);
- SPICE_DEBUG("playback mute changed to %u", mute);
+ SPICE_DEBUG("playback mute changed to %d", mute);
if (GST_IS_BIN(p->playback.sink))
e = gst_bin_get_by_interface(GST_BIN(p->playback.sink), GST_TYPE_STREAM_VOLUME);
@@ -457,7 +457,7 @@ static void record_mute_changed(GObject *object, GParamSpec *pspec, gpointer dat
return;
g_object_get(object, "mute", &mute, NULL);
- SPICE_DEBUG("record mute changed to %u", mute);
+ SPICE_DEBUG("record mute changed to %d", mute);
if (GST_IS_BIN(p->record.src))
e = gst_bin_get_by_interface(GST_BIN(p->record.src), GST_TYPE_STREAM_VOLUME);
diff --git a/src/spice-gtk-session.c b/src/spice-gtk-session.c
index bbcbeeb..6cdae87 100644
--- a/src/spice-gtk-session.c
+++ b/src/spice-gtk-session.c
@@ -183,7 +183,7 @@ static void spice_gtk_session_sync_keyboard_modifiers_for_channel(SpiceGtkSessio
SpiceInputsChannel* inputs,
gboolean force)
{
- gint guest_modifiers = 0, client_modifiers = 0;
+ guint32 guest_modifiers = 0, client_modifiers = 0;
g_return_if_fail(SPICE_IS_INPUTS_CHANNEL(inputs));
@@ -511,7 +511,7 @@ static GtkClipboard* get_clipboard_from_selection(SpiceGtkSessionPrivate *s,
} else if (selection == VD_AGENT_CLIPBOARD_SELECTION_PRIMARY) {
return s->clipboard_primary;
} else {
- g_warning("Unhandled clipboard selection: %d", selection);
+ g_warning("Unhandled clipboard selection: %u", selection);
return NULL;
}
}
@@ -864,7 +864,7 @@ static gboolean clipboard_grab(SpiceMainChannel *main, guint selection,
}
}
if (!found) {
- g_warning("clipboard: couldn't find a matching type for: %d",
+ g_warning("clipboard: couldn't find a matching type for: %u",
types[n]);
}
}
diff --git a/src/spice-pulse.c b/src/spice-pulse.c
index fd5fe91..5248bc3 100644
--- a/src/spice-pulse.c
+++ b/src/spice-pulse.c
@@ -644,7 +644,7 @@ static void playback_mute_changed(GObject *object, GParamSpec *pspec, gpointer d
pa_operation *op;
g_object_get(object, "mute", &mute, NULL);
- SPICE_DEBUG("playback mute changed %u", mute);
+ SPICE_DEBUG("playback mute changed %d", mute);
if (!p->playback.stream ||
pa_stream_get_index(p->playback.stream) == PA_INVALID_INDEX)
@@ -687,7 +687,7 @@ static void record_mute_changed(GObject *object, GParamSpec *pspec, gpointer dat
pa_operation *op;
g_object_get(object, "mute", &mute, NULL);
- SPICE_DEBUG("record mute changed %u", mute);
+ SPICE_DEBUG("record mute changed %d", mute);
if (!p->record.stream ||
pa_stream_get_device_index(p->record.stream) == PA_INVALID_INDEX)
@@ -996,7 +996,7 @@ static void spice_pulse_complete_async_task(struct async_task *task, const gchar
complete_task(task->pulse, task, err_msg);
if (p->results != NULL) {
p->results = g_list_remove(p->results, task);
- SPICE_DEBUG("Number of async task is %d", g_list_length(p->results));
+ SPICE_DEBUG("Number of async task is %u", g_list_length(p->results));
}
free_async_task(task);
}
@@ -1205,7 +1205,7 @@ static void pulse_stream_restore_info_async(gboolean is_playback,
}
p->results = g_list_append(p->results, task);
- SPICE_DEBUG ("Number of async task is %d", g_list_length(p->results));
+ SPICE_DEBUG ("Number of async task is %u", g_list_length(p->results));
return;
fail:
diff --git a/src/spice-session.c b/src/spice-session.c
index e02d684..8b3cdd0 100644
--- a/src/spice-session.c
+++ b/src/spice-session.c
@@ -1694,7 +1694,7 @@ void spice_session_start_migrating(SpiceSession *session,
g_warn_if_fail(ring_get_length(&s->channels) == ring_get_length(&m->channels));
- SPICE_DEBUG("migration channels left:%d (in migration:%d)",
+ SPICE_DEBUG("migration channels left:%u (in migration:%u)",
ring_get_length(&s->channels), ring_get_length(&m->channels));
s->migration_left = spice_session_get_channels(session);
}
@@ -1931,7 +1931,7 @@ void spice_session_disconnect(SpiceSession *session)
s = session->priv;
- SPICE_DEBUG("session: disconnecting %d", s->disconnecting);
+ SPICE_DEBUG("session: disconnecting %u", s->disconnecting);
if (s->disconnecting != 0)
return;
diff --git a/src/spice-util.c b/src/spice-util.c
index 6b1f8e5..7788921 100644
--- a/src/spice-util.c
+++ b/src/spice-util.c
@@ -266,7 +266,7 @@ const gchar* spice_yes_no(gboolean value)
G_GNUC_INTERNAL
guint16 spice_make_scancode(guint scancode, gboolean release)
{
- SPICE_DEBUG("%s: %s scancode %d",
+ SPICE_DEBUG("%s: %s scancode %u",
__FUNCTION__, release ? "release" : "", scancode);
if (release) {
diff --git a/src/spice-widget-egl.c b/src/spice-widget-egl.c
index 1557fdc..0698af3 100644
--- a/src/spice-widget-egl.c
+++ b/src/spice-widget-egl.c
@@ -655,10 +655,11 @@ gboolean spice_egl_update_scanout(SpiceDisplay *display,
attrs[10] = EGL_LINUX_DRM_FOURCC_EXT;
attrs[11] = format;
attrs[12] = EGL_NONE;
- SPICE_DEBUG("fd:%d stride:%d y0:%d %dx%d format:0x%x (%c%c%c%c)",
+ SPICE_DEBUG("fd:%d stride:%u y0:%d %ux%u format:0x%x (%c%c%c%c)",
scanout->fd, scanout->stride, scanout->y0top,
scanout->width, scanout->height, format,
- format & 0xff, (format >> 8) & 0xff, (format >> 16) & 0xff, format >> 24);
+ (int)format & 0xff, (int)(format >> 8) & 0xff,
+ (int)(format >> 16) & 0xff, (int)format >> 24);
d->egl.image = eglCreateImageKHR(d->egl.display,
EGL_NO_CONTEXT,
diff --git a/src/spice-widget.c b/src/spice-widget.c
index 9a4b204..d0fa912 100644
--- a/src/spice-widget.c
+++ b/src/spice-widget.c
@@ -304,7 +304,7 @@ void spice_display_widget_update_monitor_area(SpiceDisplay *display)
if (c->surface_id != 0) {
g_warning("FIXME: only support monitor config with primary surface 0, "
- "but given config surface %d", c->surface_id);
+ "but given config surface %u", c->surface_id);
goto whole;
}
@@ -808,7 +808,7 @@ static void try_keyboard_grab(SpiceDisplay *display)
status = gdk_keyboard_grab(gtk_widget_get_window(widget), FALSE,
GDK_CURRENT_TIME);
if (status != GDK_GRAB_SUCCESS) {
- g_warning("keyboard grab failed %d", status);
+ g_warning("keyboard grab failed %u", status);
d->keyboard_grab_active = false;
} else {
d->keyboard_grab_active = true;
@@ -966,7 +966,7 @@ static GdkGrabStatus do_pointer_grab(SpiceDisplay *display)
GDK_CURRENT_TIME);
if (status != GDK_GRAB_SUCCESS) {
d->mouse_grab_active = false;
- g_warning("pointer grab failed %d", status);
+ g_warning("pointer grab failed %u", status);
} else {
d->mouse_grab_active = true;
g_signal_emit(display, signals[SPICE_DISPLAY_MOUSE_GRAB], 0, true);
@@ -1424,7 +1424,7 @@ static gboolean key_event(GtkWidget *widget, GdkEventKey *key)
}
#endif
- SPICE_DEBUG("%s %s: keycode: %d state: %d group %d modifier %d",
+ SPICE_DEBUG("%s %s: keycode: %d state: %u group %d modifier %d",
__FUNCTION__, key->type == GDK_KEY_PRESS ? "press" : "release",
key->hardware_keycode, key->state, key->group, key->is_modifier);
@@ -1863,7 +1863,7 @@ static gboolean button_event(GtkWidget *widget, GdkEventButton *button)
SpiceDisplayPrivate *d = display->priv;
int x, y;
- SPICE_DEBUG("%s %s: button %d, state 0x%x", __FUNCTION__,
+ SPICE_DEBUG("%s %s: button %u, state 0x%x", __FUNCTION__,
button->type == GDK_BUTTON_PRESS ? "press" : "release",
button->button, button->state);
@@ -2255,7 +2255,7 @@ static void update_mouse_mode(SpiceChannel *channel, gpointer data)
GdkWindow *window = gtk_widget_get_window(GTK_WIDGET(display));
g_object_get(channel, "mouse-mode", &d->mouse_mode, NULL);
- SPICE_DEBUG("mouse mode %d", d->mouse_mode);
+ SPICE_DEBUG("mouse mode %u", d->mouse_mode);
switch (d->mouse_mode) {
case SPICE_MOUSE_MODE_CLIENT:
diff --git a/src/spicy-screenshot.c b/src/spicy-screenshot.c
index ab1f387..68f9335 100644
--- a/src/spicy-screenshot.c
+++ b/src/spicy-screenshot.c
@@ -89,7 +89,7 @@ static void invalidate(SpiceChannel *channel,
rc = write_ppm_32();
break;
default:
- fprintf(stderr, "unsupported spice surface format %d\n", d_format);
+ fprintf(stderr, "unsupported spice surface format %u\n", d_format);
rc = -1;
break;
}
@@ -105,7 +105,7 @@ static void main_channel_event(SpiceChannel *channel, SpiceChannelEvent event,
case SPICE_CHANNEL_OPENED:
break;
default:
- g_warning("main channel event: %d", event);
+ g_warning("main channel event: %u", event);
g_main_loop_quit(mainloop);
}
}
diff --git a/src/spicy-stats.c b/src/spicy-stats.c
index 1bed12e..8ca4cc1 100644
--- a/src/spicy-stats.c
+++ b/src/spicy-stats.c
@@ -36,7 +36,7 @@ static void main_channel_event(SpiceChannel *channel, SpiceChannelEvent event,
case SPICE_CHANNEL_OPENED:
break;
default:
- g_warning("main channel event: %d", event);
+ g_warning("main channel event: %u", event);
g_main_loop_quit(mainloop);
}
}
diff --git a/src/spicy.c b/src/spicy.c
index d984f62..8fc6845 100644
--- a/src/spicy.c
+++ b/src/spicy.c
@@ -1207,7 +1207,7 @@ static void main_channel_event(SpiceChannel *channel, SpiceChannelEvent event,
break;
default:
/* TODO: more sophisticated error handling */
- g_warning("unknown main channel event: %d", event);
+ g_warning("unknown main channel event: %u", event);
/* connection_disconnect(conn); */
break;
}
diff --git a/src/usb-device-manager.c b/src/usb-device-manager.c
index da7e596..325533d 100644
--- a/src/usb-device-manager.c
+++ b/src/usb-device-manager.c
@@ -1970,7 +1970,7 @@ spice_usb_device_manager_can_redirect_device(SpiceUsbDeviceManager *self,
gchar *spice_usb_device_get_description(SpiceUsbDevice *device, const gchar *format)
{
#ifdef USE_USBREDIR
- int bus, address, vid, pid;
+ guint16 bus, address, vid, pid;
gchar *description, *descriptor, *manufacturer = NULL, *product = NULL;
g_return_val_if_fail(device != NULL, NULL);
diff --git a/src/usbutil.c b/src/usbutil.c
index 6cd8148..6dcb4de 100644
--- a/src/usbutil.c
+++ b/src/usbutil.c
@@ -106,7 +106,7 @@ static gchar *spice_usbutil_get_sysfs_attribute(int bus, int address,
if (stat(filename, &stat_buf) != 0)
return NULL;
- snprintf(filename, sizeof(filename), "/sys/dev/char/%d:%d/%s",
+ snprintf(filename, sizeof(filename), "/sys/dev/char/%u:%u/%s",
major(stat_buf.st_rdev), minor(stat_buf.st_rdev), attribute);
if (!g_file_get_contents(filename, &contents, NULL, NULL))
return NULL;
commit 67011b2c8ba6a50249dd64af83514658a5b18560
Author: Marc-André Lureau <marcandre.lureau at gmail.com>
Date: Mon May 30 18:39:02 2016 +0200
build-sys: remove -Wmissing-declarations
The warning doesn't show up anymore.
Signed-off-by: Marc-André Lureau <marcandre.lureau at gmail.com>
Acked-by: Pavel Grunt <pgrunt at redhat.com>
diff --git a/configure.ac b/configure.ac
index 3c90e9e..8f1e91b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -581,11 +581,7 @@ SPICE_CHECK_LZ4
dnl ===========================================================================
dnl check compiler flags
-# We want to enable these, but need to sort out the
-# decl mess with src/generated_*.c
-dontwarn="-Wmissing-declarations"
-
-SPICE_COMPILE_WARNINGS([$dontwarn])
+SPICE_COMPILE_WARNINGS
SPICE_CFLAGS="$SPICE_CFLAGS $WARN_CFLAGS"
More information about the Spice-commits
mailing list