1. 21 Jul, 2020 6 commits
    • Alexandre Duret-Lutz's avatar
      org: run a spell checker on the documentation · f3b8bf8e
      Alexandre Duret-Lutz authored
      * doc/org/autcross.org, doc/org/autfilt.org, doc/org/citing.org,
      doc/org/compile.org, doc/org/concepts.org, doc/org/csv.org,
      doc/org/dstar2tgba.org, doc/org/genaut.org, doc/org/genltl.org,
      doc/org/hierarchy.org, doc/org/hoa.org, doc/org/index.org,
      doc/org/install.org, doc/org/ltl2tgba.org, doc/org/ltl2tgta.org,
      doc/org/ltlcross.org, doc/org/ltlfilt.org, doc/org/ltlgrind.org,
      doc/org/ltlsynt.org, doc/org/oaut.org, doc/org/randaut.org,
      doc/org/randltl.org, doc/org/satmin.org, doc/org/tut.org,
      doc/org/tut01.org, doc/org/tut02.org, doc/org/tut03.org,
      doc/org/tut04.org, doc/org/tut10.org, doc/org/tut11.org,
      doc/org/tut12.org, doc/org/tut20.org, doc/org/tut21.org,
      doc/org/tut22.org, doc/org/tut23.org, doc/org/tut24.org,
      doc/org/tut30.org, doc/org/tut31.org, doc/org/tut50.org,
      doc/org/tut51.org, doc/org/tut52.org, doc/org/tut90.org,
      doc/org/upgrade2.org: Run ispell-buffer on all these.
      * bin/autfilt.cc, python/spot/__init__.py: Fix typos in
      help texts noticed while spell-checking the org files.
      f3b8bf8e
    • Alexandre Duret-Lutz's avatar
      org: fix python execution with in-tree source and Swig4 · 0fbc83e9
      Alexandre Duret-Lutz authored
      * doc/org/.dir-locals.el.in, doc/org/init.el.in: Set the
      SPOT_UNINSTALLED envvar, as we already do in the test suite.
      0fbc83e9
    • Alexandre Duret-Lutz's avatar
      ltlcross: completely fix #420 · 457e130e
      Alexandre Duret-Lutz authored
      Reported by Salomon Sickert.
      
      * bin/ltlcross.cc: Also call determinize_unknown_acceptance() for
      positive automata.
      * tests/core/ltlcross3.test: Add another test case.
      * NEWS: Mention the fix.
      457e130e
    • Alexandre Duret-Lutz's avatar
      Merge branch 'master' into next · 1a0c8a44
      Alexandre Duret-Lutz authored
      1a0c8a44
    • Alexandre Duret-Lutz's avatar
      d61d6570
    • Alexandre Duret-Lutz's avatar
      Release Spot 2.9.2 · 66a6fbdc
      Alexandre Duret-Lutz authored
      * configure.ac, NEWS, doc/org/setup.org: Set version to 2.9.2.
      66a6fbdc
  2. 20 Jul, 2020 4 commits
  3. 17 Jul, 2020 2 commits
    • Etienne Renault's avatar
      Fixes #419 · 4db6a342
      Etienne Renault authored
      * configure.ac: Here.
      4db6a342
    • Alexandre Duret-Lutz's avatar
      formula: make operator bool explicit · 9daa4e60
      Alexandre Duret-Lutz authored
      * spot/tl/formula.hh (formula::operator bool): Make it explicit.  When
      compiling in C++20 mode with g++ 10.1, this bool operator was used
      instead of the comparison operators while looking up a
      std::pair<formula,formula> in a hash map, causing many test suite
      failures.  This problem does not occur with clang++ 10.0, so it
      might just be a bug in g++ 10.1.  But having explicit operator bool
      is good practice anyway.
      9daa4e60
  4. 16 Jul, 2020 5 commits
    • Alexandre Duret-Lutz's avatar
      C++20: work around g++ 10.1 bug #95242 · a8959ecf
      Alexandre Duret-Lutz authored
      With GCC 10.1 all comparisons that default to the <=> C++20 comparison
      operator emit a spurious zero-as-null-pointer-constant warning, which
      is an error in our configuration.  This is due to an implementation
      choice in the libstdc++ library, so it also causes warning from
      clang++ 10.0 when using the same libstdc++ library.  A fix for GCC
      PR95242 was committed in g++ (not in libstdc++), so while g++ 10.2
      might be fixed, clang++ will need a similar fix.
      
      I've seen those failures on Arch linux with gcc 10.1.0-2 and clang
      10.0.0-3.  On Debian sid, g++ 10.1.0-4 seems to already include the
      fix.
      
      * m4/gccwarn.m4: Include a string comparison in the test code so that
      -Wzero-as-null-pointer-constant is not enabled if it would produce
      warnings on such statements.
      a8959ecf
    • Alexandre Duret-Lutz's avatar
      C++20: fix warnings reported by g++ 10.1 · 368acaad
      Alexandre Duret-Lutz authored
      * spot/tl/formula.hh (formula::operator bool): Mark as noexcept.
      * bin/common_trans.hh, bin/common_trans.cc: Use std::atomic instead of
      volatile.
      368acaad
    • Alexandre Duret-Lutz's avatar
      [buddy] build in C++17 mode by default · a7ff5b4e
      Alexandre Duret-Lutz authored
      * configure.ac: Activate C++17, and replace --enable-c++17 by
      --enable-c++20.
      a7ff5b4e
    • Alexandre Duret-Lutz's avatar
      build in C++17 mode by default · 4f230976
      Alexandre Duret-Lutz authored
      * configure.ac: Activate C++17, and replace --enable-c++17 by
      --enable-c++20.
      * NEWS: Mention the news.
      * .gitlab-ci.yml: Use C++20 for the former C++17 builds.
      * HACKING, README, doc/org/compile.org, doc/org/concepts.org,
      doc/org/index.org, doc/org/install.org, doc/org/tut.org,
      doc/org/upgrade2.org, spot/misc/escape.hh: Adjust mentions
      of C++14.
      4f230976
    • Alexandre Duret-Lutz's avatar
      a770727e
  5. 15 Jul, 2020 5 commits
  6. 14 Jul, 2020 1 commit
  7. 13 Jul, 2020 17 commits
    • Alexandre Duret-Lutz's avatar
      run: fix reduce on automata with Fin · f2403c91
      Alexandre Duret-Lutz authored
      Reported by Florian Renkin.
      
      * spot/twaalgos/emptiness.cc (reduce): If the automaton uses Fin
      acceptance, check the reduced cycle and revert to the original cycle
      if necessary.
      * tests/python/intrun.py: New file.
      * tests/Makefile.am: Add it.
      * spot/twaalgos/emptiness.hh: Improve documentation.
      f2403c91
    • Alexandre Duret-Lutz's avatar
      genem: replace one recursive call by a loop · 9caba8bf
      Alexandre Duret-Lutz authored
      * spot/twaalgos/genem.cc: In the spot29 implementation for the generic
      case, when Fin(fo)=true and Fin(fo)=false have to be tested
      separately, the second test can be done by a loop instead of a
      recursion, to avoid unnecessary processing of the acceptance
      condition.  Suggested by Jan Strejček.
      9caba8bf
    • Alexandre Duret-Lutz's avatar
      address a new g++-10 warnings · a3769dfd
      Alexandre Duret-Lutz authored
      * spot/twa/twa.hh (set_named_prop): Declare the lambda as noexcept.
      * spot/twaalgos/couvreurnew.cc (acss_states): Likewise.
      a3769dfd
    • Alexandre Duret-Lutz's avatar
      swig: search for swig4.0 · e20bae66
      Alexandre Duret-Lutz authored
      * configure.ac: Use swig4.0 when available.
      * HACKING: Update.
      e20bae66
    • Alexandre Duret-Lutz's avatar
      ltldo: improve error messages · 4cfa2538
      Alexandre Duret-Lutz authored
      Use ltldo:... instead of error:... and warning:... and also improve
      the diagnostic displayed after a translation failure to mention the
      tool and formula.
      
      Incidentally, this fixes a spurious test case failure observed by
      Philipp Schlehuber on CentOS7.7 where glibc 2.17 is installed.  With
      this system, when posix_spawn() starts a binary that does not exist,
      it returns success and let the child die with exit code 127.  On more
      recent glibc, posix_spawn() manages to return execve()'s errno, as if
      the child had not been created.  We handle those two different ways to
      fail, but before this patch one used to print "error:..." and the
      other "ltldo:...".
      
      * bin/ltldo.cc: Display the program_name in error message.  Display
      the command name and formula on translation failure.
      * tests/core/ltldo.test: Adjust test case.
      * NEWS: Mention the fix.
      4cfa2538
    • Alexandre Duret-Lutz's avatar
      3368b4b9
    • Alexandre Duret-Lutz's avatar
      sccinfo: fix doc · f16bc8a5
      Alexandre Duret-Lutz authored
      * spot/twaalgos/sccinfo.hh (scc_info_options::NONE): Fix doxygen doc.
      f16bc8a5
    • Alexandre Duret-Lutz's avatar
      twa: get rid of set_num_sets_() · 9e075e73
      Alexandre Duret-Lutz authored
      * spot/twa/twa.hh (set_num_sets_): Remove, and adjust all uses.
      This fixes #414.
      9e075e73
    • Alexandre Duret-Lutz's avatar
      ltlsynt: use wdba-minimize=2 and ba-simul=0 · 37d0b0d0
      Alexandre Duret-Lutz authored
      * bin/ltlsynt.cc: Here.
      * tests/core/ltlsynt.test: Add extra test case.
      * NEWS: Mention ltlsynt -x and related defaults.
      37d0b0d0
    • Florian Renkin's avatar
      ltlsynt: Change default options · 56c8d690
      Florian Renkin authored and Alexandre Duret-Lutz's avatar Alexandre Duret-Lutz committed
      * bin/ltlsynt.cc: Change default options.
      * tests/core/ltlsynt.test: Add test.
      56c8d690
    • Florian Renkin's avatar
      ltlsynt: Add more elements in csv · 8ac24acb
      Florian Renkin authored and Alexandre Duret-Lutz's avatar Alexandre Duret-Lutz committed
      * bin/ltlsynt.cc: Add the number of states of the dpa
      and of the parity game in the csv.
      8ac24acb
    • Florian Renkin's avatar
      ltlsynt: Add -x option for translation · 7c09f64c
      Florian Renkin authored and Alexandre Duret-Lutz's avatar Alexandre Duret-Lutz committed
      * bin/ltlsynt.cc: ltlsynt can use extra options for translator.
      7c09f64c
    • Alexandre Duret-Lutz's avatar
      work around diagnostic changes in Bison 3.6 · e06f8a3e
      Alexandre Duret-Lutz authored
      Bison <3.6 used to complain about "$undefined", while Bison >=3.6 now
      write "invalid token".
      
      * tests/core/parseaut.test, tests/core/parseerr.test,
      tests/core/sugar.test: Adjust expected diagnostics to match Bison pre
      and post 3.6.
      e06f8a3e
    • Alexandre Duret-Lutz's avatar
      ltlsynt: add --algo=ps · 16540869
      Alexandre Duret-Lutz authored
      * bin/ltlsynt.cc: Implement this.
      * tests/core/ltlsynt.test: Add a test case.
      * NEWS: Mention it.
      16540869
    • Alexandre Duret-Lutz's avatar
      simplify_acc: perform unit-propagation earlier · b434ac7f
      Alexandre Duret-Lutz authored
      Closes #405.   This shows no difference on the test suite,
      but that is thanks to the previous patch: without it, an
      example in automata.ipynb would have an extra edge.
      
      * spot/twaalgos/cleanacc.cc (simplify_acceptance): Call
      unit_propagation() before simplify_complementary_marks_here() and
      fuse_marks_here(), because that is simpler to perform.
      b434ac7f
    • Alexandre Duret-Lutz's avatar
      remfin: do not clone transitions that are accepting in main · b762f542
      Alexandre Duret-Lutz authored
      * spot/twaalgos/remfin.cc (default_strategy): Detect transitions
      from the main copy that are completely accepting and that do not
      need to be repeated in the clones.
      * tests/python/remfin.py: Add a test case.
      * tests/core/ltl2dstar4.test: Improve expected results.
      * NEWS: Mention the change.
      b762f542
    • Alexandre Duret-Lutz's avatar
      improve fuse_marks_here by detecting more patterns · c005041e
      Alexandre Duret-Lutz authored
      This remove some restrictions that prevented fuse_marks_here from
      simplifying certain patterns, as noted in the first comment of
      issue #405.
      
      * spot/twaalgos/cleanacc.cc (find_interm_rec, find_fusable): Remove
      some unnecessary restrictions to singleton marks, and replace the hack
      put one non-singleton mark at the beginning of the singleton list by a
      sort.
      * tests/python/simplacc.py: Add two test cases.
      * tests/python/automata.ipynb, tests/core/remfin.test: Improve
      expected results.
      * NEWS: Mention the bug.
      c005041e