Re: [PATCH v2 3/4] staging: rtl8192u: remove redundant code

From: Lorenzo Stoakes
Date: Thu Jan 01 2015 - 05:12:16 EST


On 1 January 2015 at 04:58, Sudip Mukherjee <sudipm.mukherjee@xxxxxxxxx> wrote:
> On Thu, Jan 01, 2015 at 10:19:56AM +0530, Sudip Mukherjee wrote:
>> but this is just removal of unnecessary braces. this is not removal of redundant code.

Hi Sudip,

Again in the same theme as the whitespace changes I felt I could
fairly reasonably package up a set of similar changes which remove
redundant *symbols* of one kind or another, which though they might
not constitute significant reduction in redundant code could
reasonably come under that header.

Again I'm more than happy to edit the message to describe this change
more correctly and/or split further though, as before, I'm wary of
splitting too much as to cause irritation to reviewers.

> and, missed writing in the last mail.
> your [PATCH v2 2/4] and [PATCH v2 3/4] are same.

Apologies for this, I simply batched up the whole set once again
without checking what would remain the same or not. I don't make this
mistake for the v3 of patch 2 and will be careful to avoid this in
future!

Best, Lorenzo
--
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/