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

acc: make mark_t::operator bool() explicit

Browse source 
This avoids a few conversion problems, and also made the bug of
sbacc (fixed by 37fc948be4) obvious.

Reported by Thomas Medioni.

* spot/twa/acc.hh (mark_t::operator bool): Make it explicit.
* spot/twaalgos/remfin.cc: Adjust.
This commit is contained in:
Alexandre Duret-Lutz 2017-03-03 13:26:33 +01:00
parent 37fc948be4
commit cf5d2c2b32
3 changed files with 9 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
NEWS
View file

@ -30,6 +30,12 @@ New in spot 2.3.1.dev (not yet released)
anymore and should be replaced by the newly-introduced --format=%x
for consistency with tools producing automata.
Backward-incompatible changes:
- spot::acc_cond::mark_t::operator bool() has been marked as
explicit. The implicit converion to bool (and, via bool, to int)
was a source of bugs.
New in spot 2.3.1 (2017-02-20)

2
spot/twa/acc.hh
View file

@ -98,7 +98,7 @@ namespace spot
return id >= o.id;
}
operator bool() const
explicit operator bool() const
{
return id != 0;
}

4
spot/twaalgos/remfin.cc
View file

@ -260,7 +260,7 @@ namespace spot
i -= f;
for (auto s: states)
{
bool acc = aut->state_acc_sets(s) & i;
bool acc{aut->state_acc_sets(s) & i};
for (auto& t: aut->out(s))
res->new_acc_edge(s, t.dst, t.cond, acc);
}
@ -282,7 +282,7 @@ namespace spot
acc_cond::mark_t acc = aut->state_acc_sets(s);
if (acc.has(r))
continue;
bool jacc = acc & inf_alone;
bool jacc{acc & inf_alone};
bool cacc = fin_alone.has(r) || acc.has(r + 1);
for (auto& t: aut->out(s))
{