[PATCH] Documentation for the prepare_lock_surface event description, is incorrect

[Bryce Harrington]

On Tue, Jul 14, 2015 at 09:20:42AM +0300, Pekka Paalanen wrote:
> On Mon, 13 Jul 2015 13:04:38 -0700
> Bryce Harrington <bryce at osg.samsung.com> wrote:
> 
> > On Mon, Jul 13, 2015 at 09:12:24AM +0300, Pekka Paalanen wrote:
> > > On Fri, 10 Jul 2015 15:31:03 -0700
> > > Bryce Harrington <bryce at osg.samsung.com> wrote:
> > > 
> > > > On Fri, Jul 10, 2015 at 09:33:46AM -0400, Christopher Michael wrote:
> > > > > Thanks for the review. I've included an updated patch which fixes
> > > > > the second 'shell' mentioned in the descrption to be 'client'.
> > > > > 
> > > > > 
> > > > > Documentation for the prepare_lock_surface event description is
> > > > > incorrect. The summary says "Tell the client..." however the full-text
> > > > > description says "tell the shell..."
> > > > > 
> > > > > Signed-off-by: Chris Michael <cp.michael at samsung.com>
> > > > Reviewed-by: Bryce Harrington <bryce at osg.samsung.com>
> > > 
> > > Acked-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> > 
> > Thanks, pushed:
> > To ssh://git.freedesktop.org/git/wayland/weston
> >    29f09fe..504c4c5  master -> master
> > 
> > Chris, btw one note below:
> > 
> > > Thanks,
> > > pq
> > > 
> > > 
> > > > > ---
> > > > >  protocol/desktop-shell.xml | 4 ++--
> > > > >  1 file changed, 2 insertions(+), 2 deletions(-)
> > > > > 
> > > > > diff --git a/protocol/desktop-shell.xml b/protocol/desktop-shell.xml
> > > > > index fb0b748..dd81574 100644
> > > > > --- a/protocol/desktop-shell.xml
> > > > > +++ b/protocol/desktop-shell.xml
> > > > > @@ -44,9 +44,9 @@
> > > > > 
> > > > >      <event name="prepare_lock_surface">
> > > > >        <description summary="tell the client to create, set the lock
> > > > > surface">
> > 
> > Looks like your email program may have wrapped the above line.  I
> > unwrapped it to get git to accept it.
> 
> Yeah... this was not a correctly formatted patch email.
> Take a look at what actually ended up pushed:
> http://cgit.freedesktop.org/wayland/weston/commit/?id=504c4c56fccfe483fbc0abb436b44618c82f9058
> 
> All the actual changes are missing.

Re-pushed with the changes.

Bryce