diff mbox

kconfig:lxdialog: remove duplicate code

Message ID 20121217143751.GA16993@udknight (mailing list archive)
State New, archived
Headers show

Commit Message

wang yanqing Dec. 17, 2012, 2:37 p.m. UTC
dialog.h has two line the same below:
extern char dialog_input_result[];
This patch remove one of them.

Signed-off-by: Wang YanQing <udknight@gmail.com>
---
 scripts/kconfig/lxdialog/dialog.h | 1 -
 1 file changed, 1 deletion(-)

Comments

Yann E. MORIN Dec. 17, 2012, 6:19 p.m. UTC | #1
Wang, All,

On Monday 17 December 2012 Wang YanQing wrote:
> dialog.h has two line the same below:
> extern char dialog_input_result[];
> This patch remove one of them.
> 
> Signed-off-by: Wang YanQing <udknight@gmail.com>

Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>

> ---
>  scripts/kconfig/lxdialog/dialog.h | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/scripts/kconfig/lxdialog/dialog.h b/scripts/kconfig/lxdialog/dialog.h
> index ee17a52..307022a 100644
> --- a/scripts/kconfig/lxdialog/dialog.h
> +++ b/scripts/kconfig/lxdialog/dialog.h
> @@ -221,7 +221,6 @@ int dialog_menu(const char *title, const char *prompt,
>  		const void *selected, int *s_scroll);
>  int dialog_checklist(const char *title, const char *prompt, int height,
>  		     int width, int list_height);
> -extern char dialog_input_result[];
>  int dialog_inputbox(const char *title, const char *prompt, int height,
>  		    int width, const char *init);
>  
> -- 
> 1.7.11.1.116.g8228a23
> 
>
wang yanqing Dec. 24, 2012, 4:48 a.m. UTC | #2
On Mon, Dec 17, 2012 at 07:19:07PM +0100, Yann E. MORIN wrote:
> Wang, All,
> 
> On Monday 17 December 2012 Wang YanQing wrote:
> > dialog.h has two line the same below:
> > extern char dialog_input_result[];
> > This patch remove one of them.
> > 
> > Signed-off-by: Wang YanQing <udknight@gmail.com>
> 
> Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Michal, All.

Hi Michal, could you consider pick up this obvious right
patch, and consider others' three patch I sented one week
ago? I just don't want delay them to 2013, and they can get
more test when they appear in next tree.
https://patchwork.kernel.org/patch/1887361/
https://patchwork.kernel.org/patch/1894341/
https://patchwork.kernel.org/patch/1887441/

Happy christmas!

Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kbuild" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Yann E. MORIN Dec. 25, 2012, 6:21 p.m. UTC | #3
Wang, All,

On Monday 24 December 2012 Wang YanQing wrote:
> On Mon, Dec 17, 2012 at 07:19:07PM +0100, Yann E. MORIN wrote:
> > Wang, All,
> > 
> > On Monday 17 December 2012 Wang YanQing wrote:
> > > dialog.h has two line the same below:
> > > extern char dialog_input_result[];
> > > This patch remove one of them.
> > > 
> > > Signed-off-by: Wang YanQing <udknight@gmail.com>
> > 
> > Reviewed-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> > Tested-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
> Michal, All.
> 
> Hi Michal, could you consider pick up this obvious right
> patch,

This code-duplication-removal patch could qualify for post-rc1.

> and consider others' three patch I sented one week
> ago? I just don't want delay them to 2013, and they can get
> more test when they appear in next tree.
> https://patchwork.kernel.org/patch/1887361/
> https://patchwork.kernel.org/patch/1894341/
> https://patchwork.kernel.org/patch/1887441/

It's too late for those: the merge window is closed, and they did not even
went into Michal's tree, so they definitely do not qualify for post-rc1.
We'll have to wait until the next merge window for inclusion.

Michal probably got caught by this shorter-than-usual merge window, and
missed the deadline. :-/

Wang, I've queued them here, and will push them again to Michal, after
the holidays season is over.

Merry Chrtistmas, and a Happy New Year to all! :-)

Regards,
Yann E. MORIN.
Yann E. MORIN Dec. 26, 2012, 12:09 a.m. UTC | #4
Wang, All,

On Tuesday 25 December 2012 Yann E. MORIN wrote:
> On Monday 24 December 2012 Wang YanQing wrote:
[--SNIP--]
> > and consider others' three patch I sented one week
> > ago? I just don't want delay them to 2013, and they can get
> > more test when they appear in next tree.
> > https://patchwork.kernel.org/patch/1887361/
> > https://patchwork.kernel.org/patch/1894341/
> > https://patchwork.kernel.org/patch/1887441/
[--SNIP--]
> Wang, I've queued them here, and will push them again to Michal, after
> the holidays season is over.

I've started gathering kconfig-related patches suitable for post-3.8 there:
    git://gitorious.org/linux-kconfig/linux-kconfig.git yem-kconfig-for-next

They are browseable there:
    https://www.gitorious.org/linux-kconfig/linux-kconfig/commits/yem-kconfig-for-next

I'll send a proper pull-request to Michal in a litle while.

Note: the autoritative tree for kconfig is Michal's. I'm only doing this
gathering of patches in an attempt to not forget any, and to make Michal's
life easier, if he finds it usefull to pull from there.

Branches in that tree are based off Michal's branches, and so will be
rebased accordingly.

Regards,
Yann E. MORIN.
diff mbox

Patch

diff --git a/scripts/kconfig/lxdialog/dialog.h b/scripts/kconfig/lxdialog/dialog.h
index ee17a52..307022a 100644
--- a/scripts/kconfig/lxdialog/dialog.h
+++ b/scripts/kconfig/lxdialog/dialog.h
@@ -221,7 +221,6 @@  int dialog_menu(const char *title, const char *prompt,
 		const void *selected, int *s_scroll);
 int dialog_checklist(const char *title, const char *prompt, int height,
 		     int width, int list_height);
-extern char dialog_input_result[];
 int dialog_inputbox(const char *title, const char *prompt, int height,
 		    int width, const char *init);