Re: [dev] [PATCH] add break;s for last cases in switch statements

From: Roberto E. Vargas Caballero <k0ga_AT_shike2.com>
Date: Sun, 27 Apr 2014 10:58:36 +0200

Applied thanks!!!.



On Sat, Apr 26, 2014 at 11:50:37PM +0200, Markus Teich wrote:
> ---
>
> Heyho,
>
> here is the patch. For cases ending with return; I did not add a break since it
> serves the same functionallity in this context.
>
> --Markus
>
>
> st.c | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/st.c b/st.c
> index d2261e2..49df792 100644
> --- a/st.c
> +++ b/st.c
> _AT_@ -1228,6 +1228,7 @@ ttynew(void) {
> opt_io, strerror(errno));
> }
> }
> + break;
> }
> }
>
> _AT_@ -1673,6 +1674,7 @@ tdefcolor(int *attr, int *npar, int l) {
> default:
> fprintf(stderr,
> "erresc(38): gfx attr %d unknown\n", attr[*npar]);
> + break;
> }
>
> return idx;
> _AT_@ -2387,6 +2389,7 @@ tputc(char *c, int len) {
> * strhandle();
> */
> }
> + break;
> }
> return;
> }
> _AT_@ -2550,6 +2553,7 @@ tputc(char *c, int len) {
> fprintf(stderr, "erresc: unknown sequence ESC 0x%02X '%c'\n",
> (uchar) ascii, isprint(ascii)? ascii:'.');
> term.esc = 0;
> + break;
> }
> }
> /*
> --
> 1.8.3.2
>

-- 
Roberto E. Vargas Caballero
Received on Sun Apr 27 2014 - 10:58:36 CEST

This archive was generated by hypermail 2.3.0 : Sun Apr 27 2014 - 11:00:12 CEST