Re: [PATCH] Remove unnecessary declarations fromDocumentation/accounting/getdelays.c

From: Jesper Juhl
Date: Sun Nov 25 2012 - 17:07:13 EST


On Sat, 24 Nov 2012, Anthony G. Basile wrote:

> From: "Anthony G. Basile" <basile@xxxxxxxxxxxxxxxxxx>
>
> stime and utime are declared __u64 but are never used. On a glibc system
> this is harmless lint, but on a uClibc system, because of the difference
> in they way header files stack, including stdio.h brings in time.h and
> this causes a name collision with stime. Since these are useless anyhow,
> we remove them.
>
> Signed-off-by: Anthony G. Basile <basile@xxxxxxxxxxxxxxxxxx>
> ---
> Documentation/accounting/getdelays.c | 1 -
> 1 files changed, 0 insertions(+), 1 deletions(-)
>
> diff --git a/Documentation/accounting/getdelays.c b/Documentation/accounting/getdelays.c
> index 6f706ac..f8ebcde 100644
> --- a/Documentation/accounting/getdelays.c
> +++ b/Documentation/accounting/getdelays.c
> @@ -51,7 +51,6 @@ int dbg;
> int print_delays;
> int print_io_accounting;
> int print_task_context_switch_counts;
> -__u64 stime, utime;
>
> #define PRINTF(fmt, arg...) { \
> if (dbg) { \
>

Seems fine to me...

--
Jesper Juhl <jj@xxxxxxxxxxxxx> http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/