Changes between Version 8 and Version 9 of Commentary/Rts/Conventions
- Timestamp:
- 07/08/11 18:45:57 (23 months ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Commentary/Rts/Conventions
v8 v9 1 2 3 1 = The GHC Commentary: Coding Style Guidelines for RTS C code = 4 2 … … 8 6 {{{rts/}}} and {{{includes/}}}. See [wiki:Commentary/CodingStyle Coding Style Guidelines] for code in {{{compiler/}}}. 9 7 10 NB. This page is pretty old. 11 12 NB. These are just suggestions. They're not set in stone. Some of 8 These are just suggestions. They're not set in stone. Some of 13 9 them are probably misguided. If you disagree with them, feel free to 14 10 modify this document (and make your commit message reasonably 15 informative) or mail someone (eg. [mailto:glasgow-haskell-users@haskell.org The GHC mailing list]) 11 informative) or mail someone 12 (eg. [mailto:glasgow-haskell-users@haskell.org The GHC mailing list]) 16 13 17 14 == References == … … 22 19 * The C99 standard. One reasonable reference is [http://home.tiscalinet.ch/t_wolf/tw/c/c9x_changes.html here]. 23 20 24 * Writing Solid Code, Microsoft Press. (Highly recommended.)25 26 * Autoconf documentation. See also21 * Writing Solid Code, Microsoft Press. (Highly recommended.) 22 23 * Autoconf documentation. See also 27 24 [http://peti.gmd.de/autoconf-archive/ The autoconf macro archive] 28 25 and [http://www.cyclic.com/cyclic-pages/autoconf.html Cyclic Software's description]. … … 402 399 == Source-control issues == 403 400 404 * Don't be tempted to re indent or reorganise large chunks of code -401 * Don't be tempted to re-indent or re-organise large chunks of code - 405 402 it generates large diffs in which it's hard to see whether anything 406 403 else was changed, and causes extra conflicts when moving patches to 407 404 another branch. 408 405 409 If you must re indent or reorganise, don't include any functional406 If you must re-indent or re-organise, don't include any functional 410 407 changes that commit and give advance warning that you're about to do 411 408 it in case anyone else is changing that file.
