Skip to content
This repository has been archived by the owner on Apr 5, 2019. It is now read-only.

Commit

Permalink
tests: Check-in bindgen binding, config alignment, enable tests
Browse files Browse the repository at this point in the history
The cause of the SEGVs we were seeing in #22 were bindgen failing to
reflect the alignment attributes on structs (specifically gf_25519_t).

Until bindgen supports this, we have to check-in the generated bindings.

With the alignment fixed, the tests now pass!
  • Loading branch information
tarcieri committed Aug 15, 2018
1 parent fd7ef8d commit 83c6931
Show file tree
Hide file tree
Showing 7 changed files with 740 additions and 440 deletions.
18 changes: 3 additions & 15 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -28,23 +28,11 @@ be used yet. Check back later for updates.

### Known Issues

#### Test Suite

* [#22]: Test suite is failing due to SEGVs and is commented out

#### Travis CI

* [#24]: Sporadic SEGV on Travis C with clang when running `ristretto_gen_tables`
* [#19]: OS X is build-only (test suite not being run)

#### Appveyor

* [#21]: Windows is build-only (test suite not being run)

[#19]: https://github.com/Ristretto/libristretto255/issues/19
[#21]: https://github.com/Ristretto/libristretto255/issues/21
[#22]: https://github.com/Ristretto/libristretto255/issues/22
[#24]: https://github.com/Ristretto/libristretto255/issues/24
* [#24]: Sporadic SEGV on Travis C with clang when running
`ristretto_gen_tables`. This has been worked-around by enabling ASAN,
but is probably indicative of deeper problems.

## License

Expand Down
Loading

0 comments on commit 83c6931

Please sign in to comment.