alarsyo/spot
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

ltlsynt: --ins is optional but not its argument

Browse source 
Fixes #481, reported by Michaël Cadilhac.

* bin/ltlsynt.cc (argp_option): Remove erroneous OPTION_ARG_OPTIONAL.
* tests/core/ltlsynt.test: Add test case.
This commit is contained in:
Alexandre Duret-Lutz 2021-10-09 20:50:38 +02:00
parent b9a39c5576
commit 0e5f2d4819
2 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
bin/ltlsynt.cc
View file

@ -63,9 +63,9 @@ static const argp_option options[] =
{ "outs", OPT_OUTPUT, "PROPS", 0, { "outs", OPT_OUTPUT, "PROPS", 0,
"comma-separated list of controllable (a.k.a. output) atomic" "comma-separated list of controllable (a.k.a. output) atomic"
" propositions", 0}, " propositions", 0},
{ "ins", OPT_INPUT, "PROPS", OPTION_ARG_OPTIONAL, { "ins", OPT_INPUT, "PROPS", 0,
"comma-separated list of controllable (a.k.a. output) atomic" "comma-separated list of controllable (a.k.a. output) atomic"
" propositions. If unspecified its the complement of \"outs\".", 0}, " propositions", 0},
/**************************************************/ /**************************************************/
{ nullptr, 0, nullptr, 0, "Fine tuning:", 10 }, { nullptr, 0, nullptr, 0, "Fine tuning:", 10 },
{ "algo", OPT_ALGO, "sd|ds|ps|lar|lar.old", 0, { "algo", OPT_ALGO, "sd|ds|ps|lar|lar.old", 0,

2
tests/core/ltlsynt.test
View file

@ -406,7 +406,7 @@ ltlsynt --outs=p0 -f '!XXF(p0 & (p0 M Gp0))' > out
diff out exp diff out exp
f='Fp0 U XX((p0 & F!p1) | (!p0 & Gp1))' f='Fp0 U XX((p0 & F!p1) | (!p0 & Gp1))'
ltlsynt --verbose --algo=ps --outs=p1 --ins=p0 -f "$f" -x"dpa-simul=1" 2>err ltlsynt --verbose --algo=ps --outs p1 --ins p0 -f "$f" -x"dpa-simul=1" 2>err
grep 'DPA has 13 states' err grep 'DPA has 13 states' err
ltlsynt -x dpa-simul=0 --verbose --algo=ps --outs=p1 --ins=p0 -f "$f" 2>err ltlsynt -x dpa-simul=0 --verbose --algo=ps --outs=p1 --ins=p0 -f "$f" 2>err
grep 'DPA has 29 states' err grep 'DPA has 29 states' err