[Spice-devel] [PATCH spice-gtk] spice-widget: Remove useless return statement

[Frediano Ziglio]

Signed-off-by: Frediano Ziglio <***@redhat.com>
---
src/spice-widget.c | 2 --
1 file changed, 2 deletions(-)

diff --git a/src/spice-widget.c b/src/spice-widget.c
index 312c640a..9bb42211 100644
--- a/src/spice-widget.c
+++ b/src/spice-widget.c
@@ -2994,8 +2994,6 @@ static void channel_new(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *di
return;
}
#endif
-
- return;
}

static void channel_destroy(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display)

[Christophe Fergeau]

Post by Frediano Ziglio
---
src/spice-widget.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/spice-widget.c b/src/spice-widget.c
index 312c640a..9bb42211 100644
--- a/src/spice-widget.c
+++ b/src/spice-widget.c
@@ -2994,8 +2994,6 @@ static void channel_new(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *di
return;
}
#endif
-
- return;
}
static void channel_destroy(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display)
--
2.17.2
_______________________________________________
Spice-devel mailing list
https://lists.freedesktop.org/mailman/listinfo/spice-devel

[Eduardo Lima (Etrunko)]

Post by Frediano Ziglio
---
src/spice-widget.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/spice-widget.c b/src/spice-widget.c
index 312c640a..9bb42211 100644
--- a/src/spice-widget.c
+++ b/src/spice-widget.c
@@ -2994,8 +2994,6 @@ static void channel_new(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *di
return;

Isn't this one here useless as well?

Post by Frediano Ziglio
}
#endif
-
- return;
}
static void channel_destroy(SpiceSession *s, SpiceChannel *channel, SpiceDisplay *display)

[Frediano Ziglio]

Post by Eduardo Lima (Etrunko)

Post by Frediano Ziglio
---
src/spice-widget.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/src/spice-widget.c b/src/spice-widget.c
index 312c640a..9bb42211 100644
--- a/src/spice-widget.c
+++ b/src/spice-widget.c
@@ -2994,8 +2994,6 @@ static void channel_new(SpiceSession *s, SpiceChannel
*channel, SpiceDisplay *di
return;

Isn't this one here useless as well?

Didn't notice. Yes, currently useless too, I think is here in a sort
of coherent

if (this) {
handle this
return;
}
if (that) {
handle that
return;
}
....

Post by Eduardo Lima (Etrunko)

Post by Frediano Ziglio
}
#endif
-
- return;
}
static void channel_destroy(SpiceSession *s, SpiceChannel *channel,
SpiceDisplay *display)

Frediano