[PATCH] Add vty command for setting Access control classes

[Peter Stuge]

Alexander Chemeris wrote:
> > The command looks pretty horrible. Do you think that there could be a
> > better user interface than a single command with 16 parameters?
> >
> > I'd suggest something like: rach access control class (0..15) (0|1)
> 
> I like our way more, as it's a single line vs many-many lines. A
> single line doesn't occupy so much screen space and is easier to
> manipulate from VTY or over a slow satellite connection (happen
> frequently here). Also it gives you the whole picture in a single
> glance.
> 
> Does it sound like a valid reason?

No not the least. Compare:

rach access control classes 0 1 1 0 0 1 0 0 1 0 0 0 1 1 0

with:

rach access control class 1 1
rach access control class 2 1
rach access control class 5 0
rach access control class 5 1
rach access control class 8 1
rach access control class 13 1
rach access control class 14 1

I know which I find clearer, and easier to manipulate with a minimum
of unwanted side effects.

"The connectivity to my sites suck" really can't motivate such a bad
user interface. Use mosh and/or train operators to deal with higher
latency connections.


//Peter


PS. Did you notice the error in my 'class' commands?
PPS. Did you notice the error in your 'classes' command?