linux-kernel - Re: [PATCH] drivers: staging: bcm: Removed a developer debug statement.

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]

Date:	Thu, 24 Oct 2013 01:02:45 +0300
From:	Dan Carpenter <dan.carpenter@...cle.com>
To:	Chuong Ngo <cngo.github@...il.com>
Cc:	gregkh@...uxfoundation.org, klmckinney1@...il.com,
	devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drivers: staging: bcm:  Removed a developer debug
 statement.

On Wed, Oct 23, 2013 at 03:11:20PM -0400, Chuong Ngo wrote:
> Removed a developer debug statement per the TODO list.  Additionally, removed braces for the if-statement to match coding style.

Line wrap the changelog at 72 characters.  You don't really need to
mention the removed braces because that's too obvious to bother
commenting on.

My guess is that Greg will probably be willing to apply this as is so
don't worry about resending unless he requests it.  (He normally seems
to let newbies get by with one line wrapping warning before he starts
making you redo it).

regards,
dan carpenter

--
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/