Skip to content

Commit

Permalink
Updates ion-c-sys to pick up fix for binary literals. (#228)
Browse files Browse the repository at this point in the history
* Removes skip-list for amazon-ion/ion-c#235.
* Updates `ion-c` and `ion-tests` submodules to HEAD.
* Fixes URL in ion-c submodule.
  • Loading branch information
almann authored Apr 28, 2021
1 parent 18fe3ec commit ed085dd
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 8 deletions.
2 changes: 1 addition & 1 deletion .gitmodules
Original file line number Diff line number Diff line change
Expand Up @@ -3,4 +3,4 @@
url = https://github.com/amzn/ion-tests.git
[submodule "ionc"]
path = ion-c-sys/ion-c
url = https://github.com/amzn/ion-c
url = https://github.com/amzn/ion-c.git
2 changes: 1 addition & 1 deletion ion-c-sys/ion-c
Submodule ion-c updated from f595da to a52407
2 changes: 1 addition & 1 deletion ion-tests
6 changes: 1 addition & 5 deletions tests/loader_test_vectors.rs
Original file line number Diff line number Diff line change
Expand Up @@ -40,11 +40,7 @@ const LOAD_ONE_EQUIVS_SKIP_LIST: &[&str] = &[
];

/// Files that should only be skipped in equivalence file testing
const EQUIVS_SKIP_LIST: &[&str] = &[
// ion-c seems to have a problem with negative binary literals (amzn/ion-c#235)
"ion-tests/iontestdata/good/equivs/binaryInts.ion",
"ion-tests/iontestdata/good/equivs/intsWithUnderscores.ion",
];
const EQUIVS_SKIP_LIST: &[&str] = &[];

/// Files that should only be skipped in non-equivalence file testing
const NON_EQUIVS_SKIP_LIST: &[&str] = &[
Expand Down

0 comments on commit ed085dd

Please sign in to comment.