[Freeciv-Dev] Re: documentation patch to units.ruleset

[Raahul Kumar <raahul_da_man@xxxxxxxxx>]

--- "Per I. Mathisen" <Per.Inge.Mathisen@xxxxxxxxxxx> wrote:
> This patch adds documentation of unit flags to units.ruleset.  Removes the
> need to read the source to find out what they do. Please take a look at it
> and see if I got some things wrong.
> 

Not wrong exactly. Improved, yes.

> Yours,
> Per
> 
> "My mother never saw the irony in calling me a son-of-a-bitch."
>  -- Jack Nicholson
> > Index: units.ruleset
> ===================================================================
> RCS file: /home/freeciv/CVS/freeciv/data/default/units.ruleset,v
> retrieving revision 1.29
> diff -u -2 -r1.29 units.ruleset
> --- units.ruleset     2001/12/11 16:16:42     1.29
> +++ units.ruleset     2002/03/10 22:57:17
> @@ -48,8 +48,44 @@
>  ; fuel          = number of turns, for air units

It is not just number of turns, for barbarian units it is the lifetime. Also,
have to mention that when air units run out of fuel they crash.



> +; "IgTer"    - ignore terrain like explorers

Better as treat every tile as 1/3 move cost.

> +; "Carrier"  - can transport air units, but not land units

Make it clear that air units include missiles.






> +; "Partial_Invis" - visible only to adjancent units; does not hide
> transported

This is new one on me. I didn't know there was a partial_invis flag.


> +; "Diplomat" - can do diplomat actions

Probably mention dipl chance here.

> +; "Spy"              - can do even more diplomat actions

Mention what those actions are.




> +; "No_Land_Attack" - (sea only) cannot attack targets on land

Is it impossible to make an air unit with on land_attack? I've never used
fighters to attack land units.



__________________________________________________
Do You Yahoo!?
Try FREE Yahoo! Mail - the world's greatest free email!
http://mail.yahoo.com/