linux-kernel - Re: [PATCH v2 02/06] staging: crypto: skein: rename camelcase vars

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: <20140519134418.23f917fe@chukar.edge2.net>
Date:	Mon, 19 May 2014 13:44:18 -0600
From:	Jake Edge <jake@....net>
To:	Dan Carpenter <dan.carpenter@...cle.com>
Cc:	Anton Saraev <antonysaraev@...il.com>, devel@...verdev.osuosl.org,
	gregkh@...uxfoundation.org, jason@...edaemon.net,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 02/06] staging: crypto: skein: rename camelcase vars

On Mon, 19 May 2014 22:21:12 +0300 Dan Carpenter wrote:
> So, Jake, you're right, but if he redoes patch 2/6 then he has to redo
> the rest as well.  The bad names were bad in the original code already,
> so normally we let people fix things up in follow on patches in that
> case.
> 
> Is that ok with you?

Sure, that seems fine ... I only had fairly minor quibbles, the vast
majority of it is good stuff ...

Not sure it's useful (or even the right way to go about it), but, if it
is, Greg (or Jason or whoever) can add my:

Reviewed-by: Jake Edge <jake@....net>

to the patch set ...

jake

-- 
Jake Edge - LWN - jake@....net - http://lwn.net
--
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/