Complete.Org: Mailing Lists: Archives: freeciv-dev: July 2004:
[Freeciv-Dev] (PR#9434) Patch: Option levels
Home

[Freeciv-Dev] (PR#9434) Patch: Option levels

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: akaquinn@xxxxxxxxxxx
Subject: [Freeciv-Dev] (PR#9434) Patch: Option levels
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Thu, 22 Jul 2004 20:21:26 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=9434 >

> [gregorr - Tue Jul 20 04:05:10 2004]:
> 
> Err, sorry, didn't do -b, so that had a bunch of silly stuff that was 
> only whitespace changes.  This patch is the same but with -b. 

The sset_level field should come immediately after (or immediately
before) the sset_category field in the structure.

Also it would help if you followed the style guide.  Missing spaces,
C++-style comments, and overrun lines are easy for the maintainer to fix
when committing the patch but when there are a lot of them (as there
are) it gets annoying.

jason



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