On Sat, Dec 03, 2011 at 03:00:22PM +0100, Hans de Goede wrote: > There is only one caller of smartcard_message_complete_in_flight() and > that caller already checks that priv->in_flight_message != NULL, as > it dereferences the priv->in_flight_message pointer.
Today this code will never be triggered, but one never knows what will
happen tomorrow after some refactoring. This check documents one assertion
when we get into this function, which never hurts. We can make it into a
single g_return_if_fail if that would make things clearer, but I'd prefer
to keep it.
Christophe
>
> Signed-off-by: Hans de Goede <hdego...@redhat.com>
> ---
> gtk/channel-smartcard.c | 5 -----
> 1 files changed, 0 insertions(+), 5 deletions(-)
>
> diff --git a/gtk/channel-smartcard.c b/gtk/channel-smartcard.c
> index f376e53..04815a3 100644
> --- a/gtk/channel-smartcard.c
> +++ b/gtk/channel-smartcard.c
> @@ -264,11 +264,6 @@ smartcard_message_new(VSCMsgType msg_type, SpiceMsgOut
> *msg_out)
> static void
> smartcard_message_complete_in_flight(SpiceSmartcardChannel *channel)
> {
> - if (channel->priv->in_flight_message == NULL) {
> - g_return_if_fail(g_queue_is_empty(channel->priv->message_queue));
> - return;
> - }
> -
> smartcard_message_free(channel->priv->in_flight_message);
> channel->priv->in_flight_message =
> g_queue_pop_head(channel->priv->message_queue);
> if (channel->priv->in_flight_message != NULL)
> --
> 1.7.7.4
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/spice-devel
pgp3VhsNnvAC8.pgp
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/spice-devel
