On Mon, 11 Apr 2016 17:30:21 -0500
Yong Bakos <[email protected]> wrote:

> Jason,
> See below, this one patch affects the argument name of the
> zwp_fullscreen_shell_v1#capability event param name.
> 
> yong
> 
> 
> > On Apr 11, 2016, at 5:23 PM, Yong Bakos <[email protected]> wrote:
> > 
> > From: Yong Bakos <[email protected]>
> > 
> > Signed-off-by: Yong Bakos <[email protected]>
> > ---
> > unstable/fullscreen-shell/fullscreen-shell-unstable-v1.xml | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/unstable/fullscreen-shell/fullscreen-shell-unstable-v1.xml 
> > b/unstable/fullscreen-shell/fullscreen-shell-unstable-v1.xml
> > index 93f95b8..b32dd88 100644
> > --- a/unstable/fullscreen-shell/fullscreen-shell-unstable-v1.xml
> > +++ b/unstable/fullscreen-shell/fullscreen-shell-unstable-v1.xml
> > @@ -85,7 +85,7 @@
> >     wl_display.sync request immediately after binding to ensure that they
> >     receive all the capability events.
> >       </description>
> > -      <arg name="capabilty" type="uint"/>
> > +      <arg name="capability" type="uint"/>
> >     </event>
> > 
> >     <enum name="present_method">

Hi,

for C language bindings, I cannot see argument names making any
difference. We should usually refrain from changing them, but in this
case it is such an obvious typo that I'm ok with changing it. So have a
Reviewed-by: Pekka Paalanen <[email protected]>

The fun question is, do any other language bindings actually care about
the names as API?

This is yet another undocumented conundrum about the Wayland XML
language.


Thanks,
pq

Attachment: pgpEnd10tRoJ4.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to