Putting cyrus-future code in CVS. Also, code style

Matt Selsky selsky at columbia.edu
Mon Jun 28 11:59:54 EDT 2010


We should document the rules for the prevalent style and put it in the wiki.  Who can do that? (time and access-wise)



On Jun 28, 2010, at 9:30 AM, Ken Murchison <murch at andrew.cmu.edu> wrote:

> I think the style is mixed because of different authors and different editors.  Is isn't written down anywhere, but I agree that we should just stick with the most prevalent existing style.
> 
> 
> Bron Gondwana wrote:
>> Hi All,
>> I'm going to start pushing all my cyrus-future code in to
>> CVS over the next couple of days - so expect pretty
>> massive commits!
>> Also, a question about coding style.  Is it codified somewhere?
>> There are a few different styles throughout the code, but there
>> seems to be a fairly consistent style in most of the code.
>> Some of the utilities in particular are dodgy - 2 character
>> indents, 8 spaces instead of a tab, etc.  I also did some tidy
>> up on telemetry.c to coerce some types so there aren't warnings
>> on the syslog statement, and I noticed lots of whitespace inside
>> ifs and tab-aligned variable declarations.  Inconsistent code
>> style shits me, so I changed them to match the most common
>> style throughout the code.
>> My question is - should we start being a bit more strict about
>> both coding style and variable naming?  It makes the code heaps
>> more maintainable, and that's going to matter going forwards.
>> I've been doing bits and pieces as I touch blocks of code, but
>> not too much because it interferes with seeing the real changes.
>> Regards,
>> Bron.
> 
> -- 
> Kenneth Murchison
> Systems Programmer
> Carnegie Mellon University
> 


More information about the Cyrus-devel mailing list