Complete.Org: Mailing Lists: Archives: freeciv-dev: January 2003:
[Freeciv-Dev] Re: (PR#2745) Change style guide to favor a++ instead of +
Home

[Freeciv-Dev] Re: (PR#2745) Change style guide to favor a++ instead of +

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxx
Cc: freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#2745) Change style guide to favor a++ instead of ++a
From: "Jason Short via RT" <rt@xxxxxxxxxxxxxx>
Date: Thu, 23 Jan 2003 00:32:09 -0800
Reply-to: rt@xxxxxxxxxxxxxx

Raimar Falke via RT wrote:

> Then split the patch and commit it in two parts.

Hrm, I missed the doc/ portion of the patch...

But I find the wording confusing.  You have:

   - Prefer the postfix operator if both (postfix and prefix) can be
     used. I.e. write "a++" instead of "++a".

I would write it as:

   - Use the postfix operator instead of the prefix operator when either
     will work.  That is, write "a++" instead of "++a".

although this could probably be better.

jason

Index: doc/CodingStyle
===================================================================
RCS file: /home/freeciv/CVS/freeciv/doc/CodingStyle,v
retrieving revision 1.5
diff -u -r1.5 CodingStyle
--- doc/CodingStyle     2002/12/22 18:02:16     1.5
+++ doc/CodingStyle     2003/01/23 08:28:51
@@ -165,6 +165,9 @@
     /* nothing */
   }
 
+- Use the postfix operator instead of the prefix operator when either will
+  work.  That is, write "a++" instead of "++a".
+
 - Order include files consistently:  First state all system include
   files with <> in alphabetic order, then all Freeciv include files
   with "", sorted by directory (common, server, ...) and then by

[Prev in Thread] Current Thread [Next in Thread]