Re: [PATCH 11/18] swim: dequeue in-flight request

From: Tejun Heo
Date: Sat May 16 2009 - 10:39:36 EST


Sergei Shtylyov wrote:
> And without duplication:

Similar response as the if/else one on the other thread. Is it really
any significantly better? The 'duplication' here is basically one
liner after the peek/fetch change and when the duplication is minimal,
I usually find it clearer to put the loop condition at the while
clause itself. If you think it's significantly better, please go
ahead and submit the patch but to me the change you're proposing is
basically cosmetic and not even a clearly better one at that.

Thanks.

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