linux-kernel - Re: [PATCH] This patch to printk.c fixes a few errors reported by checkpatch.pl

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <9a8748490801031542m1f640731m12c96c59df967416@mail.gmail.com>
Date:	Fri, 4 Jan 2008 00:42:58 +0100
From:	"Jesper Juhl" <jesper.juhl@...il.com>
To:	"Paolo Ciarrocchi" <paolo.ciarrocchi@...il.com>
Cc:	linux-kernel@...r.kernel.org, bunk@...nel.org
Subject: Re: [PATCH] This patch to printk.c fixes a few errors reported by checkpatch.pl

On 04/01/2008, Paolo Ciarrocchi <paolo.ciarrocchi@...il.com> wrote:
> This patch to printk.c fixes a few errors reported by checkpatch.pl
>
[...]
> -               for (a=console_drivers->next, b=console_drivers ;
> -                    a; b=a, a=b->next) {
> +               for (a = console_drivers->next, b = console_drivers ;

I would say that if you are modifying this line anyway you should make it read

               for (a = console_drivers->next, b = console_drivers;

(the change is the removal of the space before the ';' at the end of the line)

otherwise I think it looks sane enough as a small style cleanup.

-- 
Jesper Juhl <jesper.juhl@...il.com>
Don't top-post  http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please      http://www.expita.com/nomime.html
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/