[PATCH weston] Include space in 'if ('

Bryce Harrington bryce at osg.samsung.com
Wed Jul 13 01:46:44 UTC 2016


On Wed, Jul 13, 2016 at 10:23:42AM +1000, Peter Hutterer wrote:
> On Tue, Jul 12, 2016 at 04:59:05PM -0700, Bryce Harrington wrote:
> > Signed-off-by: Bryce Harrington <bryce at osg.samsung.com>
> > ---
> 
> Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>
> but seriously, imo you should push things like this directly, I'm not sure
> how many eyeballs whitespace changes really need :)

Yeah you're right, I debated it.  ;-)
Thanks for the r-b, will push with the other patch.

Bryce
 
> Cheers,
>    Peter
> 
> >  compositor/main.c                 | 2 +-
> >  ivi-shell/ivi-layout-transition.c | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> > diff --git a/compositor/main.c b/compositor/main.c
> > index 8400d70..1f75ae0 100644
> > --- a/compositor/main.c
> > +++ b/compositor/main.c
> > @@ -1499,7 +1499,7 @@ load_wayland_backend(struct weston_compositor *c,
> >  	int ret = 0;
> >  
> >  	ret = load_wayland_backend_config(c, argc, argv, wc, &config);
> > -	if(ret < 0) {
> > +	if (ret < 0) {
> >  		weston_wayland_backend_config_release(&config);
> >  		return ret;
> >  	}
> > diff --git a/ivi-shell/ivi-layout-transition.c b/ivi-shell/ivi-layout-transition.c
> > index 04b62a5..4913db4 100644
> > --- a/ivi-shell/ivi-layout-transition.c
> > +++ b/ivi-shell/ivi-layout-transition.c
> > @@ -439,7 +439,7 @@ ivi_layout_transition_move_resize_view(struct ivi_layout_surface *surface,
> >  		transition_move_resize_view_destroy,
> >  		duration);
> >  
> > -	if(transition && layout_transition_register(transition))
> > +	if (transition && layout_transition_register(transition))
> >  		return;
> >  	layout_transition_destroy(transition);
> >  }
> > -- 
> > 1.9.1
>  


More information about the wayland-devel mailing list