test/streams_tests.cpp compilation on SunOS / illumos
#29907
On systems where int8_t is defined as char, the {S,Uns}erialize(Stream&, signed char) functions become undefined.
This PR resolves the issue by testing {S,Uns}erialize(Stream&, int8_t) instead.
No behavior change on systems where int8_t is defined as signed char, which is the case for most other systems.
Fixes #29884.
An alternative approach is mentioned in #29884 (comment) as well.
On systems where `int8_t` is defined as `char`, the
`{S,Uns}erialize(Stream&, signed char)` functions become undefined.
This change resolves the issue by testing
`{S,Uns}erialize(Stream&, int8_t)` instead.
No behavior change on systems where `int8_t` is defined as
`signed char`, which is the case for most other systems.
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
For detailed information about the code coverage, see the test coverage report.
See the guideline for information on the review process.
If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.
lgtm ACK 976e5d8f7b2bc77cb1443b8bf0f38cb07db70e9b
int8_t is also what real, non-test code should use.
