[Freeciv-Dev] [patch #4689] CodingStyle update

2014-05-19 Thread Jacob Nevins
Update of patch #4689 (project freeciv):

  Status:  Ready For Test = Done   
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?4689

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #4689] CodingStyle update

2014-05-11 Thread Jacob Nevins
URL:
  http://gna.org/patch/?4689

 Summary: CodingStyle update
 Project: Freeciv
Submitted by: jtn
Submitted on: Sun 11 May 2014 12:34:26 BST
Category: docs
Priority: 5 - Normal
  Status: In Progress
 Privacy: Public
 Assigned to: jtn
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 2.4.3, 2.5.0, 2.6.0

___

Details:

After comments in patch #3901 I reviewed svn doc/CodingStyle against the wiki
version http://www.freeciv.org/wiki/Coding_Style, as they had diverged.

Mostly the wiki needed updating
http://www.freeciv.org/wiki/Coding_Style?diff=21630oldid=20599, but there
are a few updates on wiki that haven't made it back to svn (notably pepeto's
change about pointer declarations
http://www.freeciv.org/wiki/Coding_Style?diff=17090oldid=16310 from Sep
2010).

In addition, I've made a couple of cleanups of my own: notably, fixing an
example to use int foo(void) instead of int foo() in line with policy
elsewhere in the document.

Hopefully this shouldn't be controversial as there's no new policy here.




___

Reply to this item at:

  http://gna.org/patch/?4689

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #4689] CodingStyle update

2014-05-11 Thread Jacob Nevins
Update of patch #4689 (project freeciv):

  Status: In Progress = Ready For Test 

___

Additional Item Attachment:

File name: trunk-S2_5-S2_4-codingstyle.patch Size:3 KB


___

Reply to this item at:

  http://gna.org/patch/?4689

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #4689] CodingStyle update

2014-05-11 Thread Marko Lindqvist
Follow-up Comment #1, patch #4689 (project freeciv):

I agree with Pepeto's change, so ok for it to go to authoritative CodingStyle
document. :-)

___

Reply to this item at:

  http://gna.org/patch/?4689

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev