Hi!
> case $mount_times in
> -"1" )
> +"1"|"100")
> mount_times=1
> ;;
> -"2" )
> - mount_times=100
> - ;;
> * )
Hmm, after the change the mount_times is always set to 1 (forget to
remove mount_times=1?).
Also we may only want to check that the parameter is a numeric so that
we can tune the test parameters later (and for other parameters that can
be tuned such as number of subgroup as well).
The portable way to do that seems to be case statement:
case $mount_times in
''|*[!0-9]*) usage;;
*) ;;
esac
Otherwise the patch looks good.
--
Cyril Hrubis
chru...@suse.cz
------------------------------------------------------------------------------
_______________________________________________
Ltp-list mailing list
Ltp-list@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ltp-list
