2020-03-16 19:53:05

by Gabriel Ravier

Subject: [PATCH] gpio-hammer: Apply scripts/Lindent and retain good changes

"retain good changes" means that I left the help string split up instead
of having this weird thing where it tries to merge together the last three
lines and it looks **really** bad

Signed-off-by: Gabriel Ravier <[email protected]>
---
tools/gpio/gpio-hammer.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tools/gpio/gpio-hammer.c b/tools/gpio/gpio-hammer.c
index 0e0060a6eb34..51a65f8149ea 100644
--- a/tools/gpio/gpio-hammer.c
+++ b/tools/gpio/gpio-hammer.c
@@ -77,7 +77,7 @@ int hammer_device(const char *device_name, unsigned int *lines, int nlines,

fprintf(stdout, "[%c] ", swirr[j]);
j++;
- if (j == sizeof(swirr)-1)
+ if (j == sizeof(swirr) - 1)
j = 0;

fprintf(stdout, "[");
--
2.24.1

2020-03-17 13:51:18

by Bartosz Golaszewski

Subject: Re: [PATCH] gpio-hammer: Apply scripts/Lindent and retain good changes

pon., 16 mar 2020 o 20:50 Gabriel Ravier <[email protected]> napisał(a):
>
> "retain good changes" means that I left the help string split up instead
> of having this weird thing where it tries to merge together the last three
> lines and it looks **really** bad
>
> Signed-off-by: Gabriel Ravier <[email protected]>
> ---
> tools/gpio/gpio-hammer.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/gpio/gpio-hammer.c b/tools/gpio/gpio-hammer.c
> index 0e0060a6eb34..51a65f8149ea 100644
> --- a/tools/gpio/gpio-hammer.c
> +++ b/tools/gpio/gpio-hammer.c
> @@ -77,7 +77,7 @@ int hammer_device(const char *device_name, unsigned int *lines, int nlines,
>
> fprintf(stdout, "[%c] ", swirr[j]);
> j++;
> - if (j == sizeof(swirr)-1)
> + if (j == sizeof(swirr) - 1)
> j = 0;
>
> fprintf(stdout, "[");
> --
> 2.24.1
>

Patch applied, thanks!

Bartosz