Fix iterators in Java API (#3000)
authorAndres Noetzli <andres.noetzli@gmail.com>
Thu, 16 May 2019 00:18:48 +0000 (00:18 +0000)
committerAina Niemetz <aina.niemetz@gmail.com>
Thu, 16 May 2019 00:18:48 +0000 (17:18 -0700)
commit62d361071ea54c9b7cba882313ab4dedef6f1286
tree2e721a2fb77769a2824955a2b47fdf8b86203da3
parentfc8907afc08d7b418471a537f9c23e9964df82df
Fix iterators in Java API (#3000)

Fixes #2989. SWIG 3 seems to have an issue properly resolving
`T::const_iterator::value_type` if that type itself is a `typedef`.
This is for example the case in the `UnsatCore` class, which `typedef`s
`const_iterator` to `std::vector<Expr>::const_iterator`. As a
workaround, this commit changes the `JavaIteratorAdapter` class to take
two template parameters, one of which is the `value_type`. The commit
also adds a compile-time assertion that `T::const_iterator::value_type`
can be converted to `value_type` to avoid nasty surprises. A nice
side-effect of this solution is that explicit `typemap`s are not
necessary anymore, so they are removed. Additionally, the commit adds a
`toString()` method for the Java API of `UnsatCore` and adds examples
that show and test the iteration over the unsat core and the statistics.
Iterating over `Statistics` now returns instances of `Statistic` instead
of `Object[]`, which is a bit cleaner and requires less glue code.
examples/api/java/CMakeLists.txt
examples/api/java/Statistics.java [new file with mode: 0644]
examples/api/java/UnsatCores.java [new file with mode: 0644]
src/bindings/java/CMakeLists.txt
src/bindings/java_iterator_adapter.h
src/expr/datatype.i
src/expr/expr.i
src/proof/unsat_core.i
src/smt/command.i
src/util/statistics.i