Re: CVS commit: src/sys/arch/i386/stand/lib

[jnemeth%victoria.tc.ca@localhost (John Nemeth)]

On Apr 1,  5:45pm, Christos Zoulas wrote:
} In article <200812150529.mBF5TFfP005724%vtn1.victoria.tc.ca@localhost>,
} John Nemeth <jnemeth%victoria.tc.ca@localhost> wrote:
} >On May 6,  4:15pm, Thor Lancelot Simon wrote:
} >} On Sun, Dec 14, 2008 at 03:26:50PM -0800, John Nemeth wrote:
} >} > On Apr 1,  5:11am, Christos Zoulas wrote:
} >} [...]
} >} > } Log Message:
} >} > } ANSI and KNF from Anon Ymous
} >} > 
} >} >      Why are we doing stuff like this while in release mode?
} >} 
} >} There is a release branch.  Christos did not commit the change you
} >} quoted above to the release branch -- but you knew that, right?
} >
} >     Um, yes, I saw that.  However, cosmetic rototills are generally
} >frowned upon at release time.  Even on head, since they can complicate
} >pullups should they be necessary.
} 
} And you are picking on this, where the code is simple and the changes
} are merely syntactic, while there have been a huge amount of changes in
} the rest of the kernel? We cannot really stand still and wait for
} the release for 6 months or more. We just need to be responsible to

     I agree that we can't stand still (we've been slowed for too long
already).  In fact, I plan on adding a driver soon which hopefully can
be pulled up.  However, cosmetic changes are more like moving sideways
(although they can clear the way for forward changes) then moving
forwards and aren't generally a problem if they wait a significant
length of time.

}-- End of excerpt from Christos Zoulas