On 9/24/07, Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Mon, Sep 24, 2007 at 07:27:56PM +0200, Johannes Berg wrote: > > On Mon, 2007-09-24 at 13:24 -0400, Luis R. Rodriguez wrote: > > > > > * ath5k_hw_set_rfgain_opt() is defined once and used once, can we > > > instead make it inline? If we really are not going to re-use this > > > later why do we put it into a routine on its own? > > > > Why mark it inline then? The compiler will inline it anyway if it's only > > used once. And I haven't looked at the code but making it an own routine > > could possibly make the code more readable, especially if lots of > > indentation is involved.` > > Seconded. Splitting things up into logically sound subroutines is > good programming style. Then that comment can be ignored, thanks. Luis - To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
