Andrew Morton wrote:
On Tue, 20 Feb 2007 17:35:16 +1100
NeilBrown <[EMAIL PROTECTED]> wrote:

+               for (i = conf->raid_disks ; i-- ;  ) {

That statement should be dragged out, shot, stomped on then ceremonially
incinerated.

What's wrong with doing

        for (i = 0; i < conf->raid_disks; i++) {

in a manner which can be understood without alcoholic fortification?

I don't find either hard to read, but you suggestion isn't equivalent, since it increments rather than decrements the index.
I admit I probably would write it the same way Neil did...

--
bill davidsen <[EMAIL PROTECTED]>
 CTO TMR Associates, Inc
 Doing interesting things with small computers since 1979

-
To unsubscribe from this list: send the line "unsubscribe linux-raid" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to