On Thu, 5 May 2016 18:27:27 -0500
Yong Bakos <j...@humanoriented.com> wrote:

> On May 5, 2016, at 6:12 PM, Yong Bakos <j...@humanoriented.com> wrote:
> > 
> > Hi Benoit,
> > 
> > On May 5, 2016, at 3:45 PM, Benoit Gschwind <gschw...@gnu-log.net> wrote:  
> >> 
> >> Rename wayland_output_init_from_config to wayland_output_config_init.
> >> 
> >> Signed-off-by: Benoit Gschwind <gschw...@gnu-log.net>
> >> ---
> >> src/compositor-wayland.c | 4 ++--
> >> 1 file changed, 2 insertions(+), 2 deletions(-)
> >> 
> >> diff --git a/src/compositor-wayland.c b/src/compositor-wayland.c
> >> index b51467d..2822a09 100644
> >> --- a/src/compositor-wayland.c
> >> +++ b/src/compositor-wayland.c
> >> @@ -1092,7 +1092,7 @@ err_name:
> >> }
> >> 
> >> static void
> >> -wayland_output_init_from_config(struct 
> >> weston_wayland_backend_output_config *output,
> >> +wayland_output_config_init(struct weston_wayland_backend_output_config 
> >> *output,
> >>                            struct weston_config_section *config_section,
> >>                            int option_width, int option_height,
> >>                            int option_scale)  
> > 
> > The indentation of the subsequent parameters should probably be adjusted to 
> > match the first.  
> 
> Of course, please just ignore me if the goal here was to keep the patch
> very specific to just the rename (per Pekka's feedback).

Hi,

well, part of a rename is reflowing and re-aligning the lines, so it
should be done where needed.


Thanks,
pq

Attachment: pgpx0MJ5pMIWn.pgp
Description: OpenPGP digital signature

_______________________________________________
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to