[Spice-devel] [PATCH spice-gtk 5/7] Fix many -Werror=format warnings
Pavel Grunt
pgrunt at redhat.com
Wed Jun 1 06:43:18 UTC 2016
On Mon, 2016-05-30 at 19:31 +0200, Marc-André Lureau wrote:
> Signed-off-by: Marc-André Lureau <marcandre.lureau at gmail.com>
Acked-by: Pavel Grunt <pgrunt at redhat.com>
> ---
> src/channel-base.c | 2 +-
> src/channel-cursor.c | 2 +-
> src/channel-display-gst.c | 4 +--
> 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 +-
> 24 files changed, 87 insertions(+), 86 deletions(-)
>
> 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 f162078..7a33c49 100644
> --- a/src/channel-display-gst.c
> +++ b/src/channel-display-gst.c
> @@ -369,7 +369,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 */
> @@ -405,7 +405,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 c555f75..27144b5 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,
> @@ -2680,7 +2680,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;
> }
>
> @@ -2769,7 +2769,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);
> @@ -2807,7 +2807,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
> SpiceInputs
> Channel* 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;
More information about the Spice-devel
mailing list