[PATCH weston v3 03/17] compositor-wayland: rename wayland_output_init_from_config

Pekka Paalanen ppaalanen at gmail.com
Fri May 6 12:50:05 UTC 2016


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

> On May 5, 2016, at 6:12 PM, Yong Bakos <junk at humanoriented.com> wrote:
> > 
> > Hi Benoit,
> > 
> > On May 5, 2016, at 3:45 PM, Benoit Gschwind <gschwind at gnu-log.net> wrote:  
> >> 
> >> Rename wayland_output_init_from_config to wayland_output_config_init.
> >> 
> >> Signed-off-by: Benoit Gschwind <gschwind at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 811 bytes
Desc: OpenPGP digital signature
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20160506/77592ae2/attachment.sig>


More information about the wayland-devel mailing list