# HG changeset patch # User Eric Warmenhoven # Date 997898988 0 # Node ID a9940cdb86eed483befe936ac0bdaa29a31164c1 # Parent e257c17864ebf044b1bd4cf0f89d0af8020278e1 [gaim-migrate @ 2154] coding style. mmm. Also got rid of po/ChangeLog, it's autogenerated and we don't use it. committer: Tailor Script diff -r e257c17864eb -r a9940cdb86ee HACKING --- a/HACKING Wed Aug 15 17:26:14 2001 +0000 +++ b/HACKING Wed Aug 15 18:09:48 2001 +0000 @@ -33,7 +33,36 @@ and send it off. This file was last modified by $Author: warmenhoven $ on -$Date: 2001-05-19 20:01:55 -0400 (Sat, 19 May 2001) $. +$Date: 2001-08-15 14:09:48 -0400 (Wed, 15 Aug 2001) $. + + +CODING STYLE +============ + +Coding styles are like assholes, everyone has one and no one likes +anyone elses. This is mine and if you want me to accept a patch from +you without getting annoyed you'll follow this coding style. :) + +It would probably just be easier for me to include CodingStyle from the +linux kernel source. + +Tab indents. I *HATE* 2-space indents, and I strongly dislike 8-space +indents. Use a tab character. I'm likely to refuse a patch if it has +2-space indents. + +K&R style for braces. Braces always go on the same line as the if, etc. +that they're associated with; the only exception is functions. Braces +for else statements should have both braces on the same line as the else +(i.e. "} else {"). + +No functionOrVariableNamesLikeThis. Save it for Java. Underscores are +your friend. "tmp" is an excellent variable name. Hungarian style will +not be tolerated. Go back to Microsoft. + +I have a 105-char wide Eterm. Deal with it. + +NO goto. I'm very likely to refuse a patch if it makes use of goto. If +you feel the need to use goto, you need to rethink your design and flow. PROGRAM FLOW diff -r e257c17864eb -r a9940cdb86ee po/ChangeLog