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

Lorenzo Stoakes lstoakes at gmail.com
Thu Jan 1 10:12:09 UTC 2015


On 1 January 2015 at 04:58, Sudip Mukherjee <sudipm.mukherjee at gmail.com> 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


More information about the devel mailing list