On Wed, Oct 26, 2016 at 10:08:09AM +0100, Frediano Ziglio wrote:
> This was introduced with 96e94c6f32d4345ea25ef7a474fbd92de03b38ad
> (Convert RedChannel hierarchy to GObject).
> The type for "core-interface" property should be
> SpiceCoreInterfaceInternal, not SpiceCoreInterface.
>
> Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx>
> ---
> server/dummy-channel.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> diff --git a/server/dummy-channel.c b/server/dummy-channel.c
> index c43221b..61a8d95 100644
> --- a/server/dummy-channel.c
> +++ b/server/dummy-channel.c
> @@ -50,7 +50,8 @@ static void dummy_watch_update_mask(SpiceWatch *watch, int event_mask)
> {
> }
>
> -static SpiceWatch *dummy_watch_add(int fd, int event_mask, SpiceWatchFunc func, void *opaque)
> +static SpiceWatch *dummy_watch_add(const SpiceCoreInterfaceInternal *iface,
> + int fd, int event_mask, SpiceWatchFunc func, void *opaque)
> {
> return NULL; // apparently allowed?
> }
> @@ -60,7 +61,7 @@ static void dummy_watch_remove(SpiceWatch *watch)
> }
>
> // TODO: actually, since I also use channel_client_dummym, no need for core. Can be NULL
Feel free to fix the "dummym" typo there in that commit as well. Looks
like passing a NULL core-interface would fix this bug too, do you know
why this is a TODO and not done now? Just curious, I'm fine with
Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
for now.
Christophe
> -static const SpiceCoreInterface dummy_core = {
> +static const SpiceCoreInterfaceInternal dummy_core = {
> .watch_update_mask = dummy_watch_update_mask,
> .watch_add = dummy_watch_add,
> .watch_remove = dummy_watch_remove,
> --
> 2.7.4
>
> _______________________________________________
> Spice-devel mailing list
> Spice-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel