diff mbox

coding-style: fix mismatch of jump label name

Message ID 1478105854-11867-1-git-send-email-yamada.masahiro@socionext.com
State Accepted
Commit beab6cb20c1dff4a0fb1ee9d14ff5f826ccd2c84
Headers show

Commit Message

Masahiro Yamada Nov. 2, 2016, 4:57 p.m. UTC
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(-)

-- 
1.9.1

Comments

Jean Delvare Nov. 2, 2016, 7:26 p.m. UTC | #1
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
Jonathan Corbet Nov. 8, 2016, 2:05 a.m. UTC | #2
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
diff mbox

Patch

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: