* [PATCH] coding-style: fix mismatch of jump label name
@ 2016-11-02 16:57 Masahiro Yamada
2016-11-02 19:26 ` Jean Delvare
2016-11-08 2:05 ` Jonathan Corbet
0 siblings, 2 replies; 3+ messages in thread
From: Masahiro Yamada @ 2016-11-02 16:57 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Mauro Carvalho Chehab, linux-kernel
Cc: Jonathan Corbet, Jean Delvare, Markus Elfring, Greg Kroah-Hartman,
Masahiro Yamada
Commit 865a1caa4b6b ("CodingStyle: Clarify and complete chapter 7")
renamed the label "out_buffer" to "out_free_buffer", but missed to
change this line.
Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---
Note:
This patch is based on linux-next.
The Documentation/CodingStyle was moved to
Documentation/process/coding-style.rst
Documentation/process/coding-style.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/process/coding-style.rst b/Documentation/process/coding-style.rst
index 968808be..3e79051 100644
--- a/Documentation/process/coding-style.rst
+++ b/Documentation/process/coding-style.rst
@@ -475,7 +475,7 @@ The rationale for using gotos is:
...
}
result = 1;
- goto out_buffer;
+ goto out_free_buffer;
}
...
out_free_buffer:
--
1.9.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] coding-style: fix mismatch of jump label name
2016-11-02 16:57 [PATCH] coding-style: fix mismatch of jump label name Masahiro Yamada
@ 2016-11-02 19:26 ` Jean Delvare
2016-11-08 2:05 ` Jonathan Corbet
1 sibling, 0 replies; 3+ messages in thread
From: Jean Delvare @ 2016-11-02 19:26 UTC (permalink / raw)
To: Masahiro Yamada
Cc: Mauro Carvalho Chehab, Mauro Carvalho Chehab, linux-kernel,
Jonathan Corbet, Markus Elfring, Greg Kroah-Hartman
Hi Masahiro-san,
On Thu, 3 Nov 2016 01:57:34 +0900, Masahiro Yamada wrote:
> Commit 865a1caa4b6b ("CodingStyle: Clarify and complete chapter 7")
> renamed the label "out_buffer" to "out_free_buffer", but missed to
> change this line.
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
>
> Note:
> This patch is based on linux-next.
> The Documentation/CodingStyle was moved to
> Documentation/process/coding-style.rst
>
> Documentation/process/coding-style.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/process/coding-style.rst b/Documentation/process/coding-style.rst
> index 968808be..3e79051 100644
> --- a/Documentation/process/coding-style.rst
> +++ b/Documentation/process/coding-style.rst
> @@ -475,7 +475,7 @@ The rationale for using gotos is:
> ...
> }
> result = 1;
> - goto out_buffer;
> + goto out_free_buffer;
> }
> ...
> out_free_buffer:
Oops, my bad. Good catch.
Reviewed-by: Jean Delvare <jdelvare@suse.de>
Feel free to add a Fixes: tag, although I doubt this kind of
documentation patch is often backported, so it probably doesn't really
matter.
Thanks,
--
Jean Delvare
SUSE L3 Support
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] coding-style: fix mismatch of jump label name
2016-11-02 16:57 [PATCH] coding-style: fix mismatch of jump label name Masahiro Yamada
2016-11-02 19:26 ` Jean Delvare
@ 2016-11-08 2:05 ` Jonathan Corbet
1 sibling, 0 replies; 3+ messages in thread
From: Jonathan Corbet @ 2016-11-08 2:05 UTC (permalink / raw)
To: Masahiro Yamada
Cc: Mauro Carvalho Chehab, Mauro Carvalho Chehab, linux-kernel,
Jean Delvare, Markus Elfring, Greg Kroah-Hartman
On Thu, 3 Nov 2016 01:57:34 +0900
Masahiro Yamada <yamada.masahiro@socionext.com> wrote:
> Commit 865a1caa4b6b ("CodingStyle: Clarify and complete chapter 7")
> renamed the label "out_buffer" to "out_free_buffer", but missed to
> change this line.
Applied to the docs tree, thanks.
jon
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-11-08 2:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-11-02 16:57 [PATCH] coding-style: fix mismatch of jump label name Masahiro Yamada
2016-11-02 19:26 ` Jean Delvare
2016-11-08 2:05 ` Jonathan Corbet
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox