[Freeciv-Dev] (PR#9434) Patch: Option levels
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
<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
|
|