[PATCH 2/3] weston:Add wl_output name event

[Wang, Quanxian]

From: Wang, Quanxian
Sent: Wednesday, March 19, 2014 10:03 AM
To: 'Jason Ekstrand'
Cc: ppaalanen at gmail.com; wayland-devel at lists.freedesktop.org
Subject: RE: [PATCH 2/3] weston:Add wl_output name event



From: Jason Ekstrand [mailto:jason at jlekstrand.net]
Sent: Wednesday, March 19, 2014 8:55 AM
To: Wang, Quanxian
Cc: ppaalanen at gmail.com<mailto:ppaalanen at gmail.com>; wayland-devel at lists.freedesktop.org<mailto:wayland-devel at lists.freedesktop.org>
Subject: RE: [PATCH 2/3] weston:Add wl_output name event


On Mar 16, 2014 5:30 AM, "Wang, Quanxian" <quanxian.wang at intel.com<mailto:quanxian.wang at intel.com>> wrote:
>
>
>
>
>
> From: Jason Ekstrand [mailto:jason at jlekstrand.net<mailto:jason at jlekstrand.net>]
> Sent: Saturday, March 15, 2014 9:10 PM
>
> To: Wang, Quanxian
> Cc: ppaalanen at gmail.com<mailto:ppaalanen at gmail.com>; wayland-devel at lists.freedesktop.org<mailto:wayland-devel at lists.freedesktop.org>
> Subject: RE: [PATCH 2/3] weston:Add wl_output name event
>
>
>
>
> On Mar 15, 2014 4:57 AM, "Wang, Quanxian" <quanxian.wang at intel.com<mailto:quanxian.wang at intel.com>> wrote:
> >
> >
> >
> >
> >
> > From: Jason Ekstrand [mailto:jason at jlekstrand.net<mailto:jason at jlekstrand.net>]
> > Sent: Saturday, March 15, 2014 3:54 AM
> > To: Wang, Quanxian
> > Cc: ppaalanen at gmail.com<mailto:ppaalanen at gmail.com>; wayland-devel at lists.freedesktop.org<mailto:wayland-devel at lists.freedesktop.org>
> >
> > Subject: Re: [PATCH 2/3] weston:Add wl_output name event
> >
> >
> >
> >
> > On Mar 13, 2014 9:12 PM, "Quanxian Wang" <quanxian.wang at intel.com<mailto:quanxian.wang at intel.com>> wrote:
> > >
> > > Signed-off-by: Quanxian Wang <quanxian.wang at intel.com<mailto:quanxian.wang at intel.com>>
> > > ---
> > >  src/compositor.c | 3 +++
> > >  1 file changed, 3 insertions(+)
> > >
> > > diff --git a/src/compositor.c b/src/compositor.c
> > > index 98a4f6f..8e8964b 100644
> > > --- a/src/compositor.c
> > > +++ b/src/compositor.c
> > > @@ -3045,6 +3045,9 @@ bind_output(struct wl_client *client,
> > >         }
> > >
> > >         if (version >= 2)
> > > +               wl_output_send_name(resource, output->name);
> >
> > As with my comment on the protocol, wl_output_send_name should be version 3, not version 2.  That should also clear up Bryce's comment.
> >
> > [Wang, Quanxian] if it is 2, patch 3/3 is not needed because  currently shell only support version 2. Right?
>
> We should still have patch 3 since weston should always support the latest version of everything.
>
> [Wang, Quanxian]That is fine according to my testing. If no patch 3/3, desktop-shell will crash. I am not sure if more updates are needed. For example, in our shell code, we use such code  min(2, version) for wl_output interface, should we change it to minal(3,version) if merge this patch?

Of anything is crashing, chances are that something is wrong. Perhaps this is tied to adding events without bumping the version number?

[Wang, Quanxian] maybe, I will have a try and check if it is the bump issue which cause client crash. Thanks for your reminder. If it is, patch 3 is not needed. If we want to support latest version, we should bump wl_output to version 3 and add name listener.

[Wang, Quanxian] I have tested it. It works as your said. The crash happens without bumping the version number. I will send the patch series later with updates in shell. Thanks

--Jason Ekstrand

>
> >
> > Thanks,
> > --Jason Ekstrand
> >
> > > +
> > > +       if (version >= 2)
> > >                 wl_output_send_done(resource);
> > >  }
> > >
> > > --
> > > 1.8.1.2
> > >
> > > _______________________________________________
> > > wayland-devel mailing list
> > > wayland-devel at lists.freedesktop.org<mailto:wayland-devel at lists.freedesktop.org>
> > > http://lists.freedesktop.org/mailman/listinfo/wayland-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20140319/0b00653c/attachment.html>