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

["Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>]

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

> [gregorr - Tue Jul 20 02:34:37 2004]:
> 
> The number of options has become quite immense.

Yes.

>  This patch divides 
> the options into 3 levels: Vital, situational, and rarely changed.

Good idea.  But how does it work with the option categories?

AFAIK the option categories are only used at the client...

> Typing "show" will cause it to show only vital options, and then tell 
> you how to show the others.  

Good, but "show 2" is not a good line.  It should be:

  show => show vital only
  show vital => show vital only
  show situational => show situational only
  show rare => show rare ones only
  show all => show all

Additionally, you should be able to give another parameter limiting the
category.  So

  show geographic
  show all geographic
  show geographic all <= maybe

> This not only makes the options list a 
> bit more manageable, but also easier for newbies to the server, since 
> they don't have to wade through tons of options to find what they're 
> looking for. 

As for the code, aside from the comments above you should use an
enumeration rather than values 0,1,2.

jason