diff options
| author | Dana Robinson <43805+derobins@users.noreply.github.com> | 2021-03-17 19:22:02 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-03-17 19:22:02 (GMT) |
| commit | 35d6091ab7c09ec5d450860ba9795e6abe430c8c (patch) | |
| tree | 253e1f47c0998cb215f22036e07ddaa2f38c84d4 /test/trefstr.c | |
| parent | af54fd532c470fa0bb76aa924dc695a86d3b0f9d (diff) | |
| download | hdf5-35d6091ab7c09ec5d450860ba9795e6abe430c8c.zip hdf5-35d6091ab7c09ec5d450860ba9795e6abe430c8c.tar.gz hdf5-35d6091ab7c09ec5d450860ba9795e6abe430c8c.tar.bz2 | |
Cleans up a couple of MSVC warnings in testhdf5 (#475)
* Fixes a few testhdf5 warnings raised in Visual Studio
Visual Studio is grumpier about treating pointers like integers than
gcc.
* Committing clang-format changes
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Diffstat (limited to 'test/trefstr.c')
| -rw-r--r-- | test/trefstr.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/test/trefstr.c b/test/trefstr.c index 74455da..a3f568a 100644 --- a/test/trefstr.c +++ b/test/trefstr.c @@ -265,7 +265,9 @@ test_refstr_wrap(void) /* Get pointer to raw string in ref-counted string */ s = H5RS_get_str(rs); CHECK_PTR(s, "H5RS_get_str"); - CHECK(s, buf, "wrapping"); + if (s == buf) + TestErrPrintf("%d: Should not have gotten the same pointer from reference-counted string!\n", + __LINE__); cmp = HDstrcmp(s, buf); if (cmp <= 0) TestErrPrintf("%d: string comparison incorrect!\n", __LINE__); @@ -375,7 +377,7 @@ test_refstr_acat(void) /* Append a large string to ref-counted string */ large_str = HDmalloc(1024); - CHECK(large_str, NULL, "HDmalloc"); + CHECK_PTR(large_str, "HDmalloc"); HDmemset(large_str, 'a', 1024); large_str[1023] = '\0'; ret = H5RS_acat(rs, large_str); @@ -386,7 +388,7 @@ test_refstr_acat(void) CHECK_PTR(s, "H5RS_get_str"); HDsprintf(buf, "%s", "foobar"); large_str2 = HDmalloc(1024 + 6); - CHECK(large_str2, NULL, "HDmalloc"); + CHECK_PTR(large_str2, "HDmalloc"); HDstrcpy(large_str2, "foobar"); HDmemset(&large_str2[6], 'a', 1024); large_str2[1029] = '\0'; |