Skip to content

Reject failed input streams - #1486

Open
henriquejsza wants to merge 4 commits into
jbeder:masterfrom
henriquejsza:fix/408-stream-errors
Open

Reject failed input streams#1486
henriquejsza wants to merge 4 commits into
jbeder:masterfrom
henriquejsza:fix/408-stream-errors

Conversation

@henriquejsza

Copy link
Copy Markdown

Fixes #408

  • reject failed input streams before and during parsing
  • preserve empty and EOF-only streams
  • add regression coverage for Load and LoadAll

Tests:

  • CMake C++11 static build and full test suite
  • Clang C++23 shared build and full CTest
  • GCC C++20 ASan/UBSan suite
  • Bazel build :all and test test

@henriquejsza

Copy link
Copy Markdown
Author

CI follow-up: the six Windows x64 MinGW jobs built successfully but hung in CTest when launched under Git Bash. In that shell, /mingw64/bin precedes the compiler-matching C:\mingw64\bin; prepending /c/mingw64/bin only for the affected matrix entries makes the full suite complete. This was verified in diagnostic run 32585105051, where the job passed in 1m40s. Commit f1c2b7f applies only that PATH adjustment; the refreshed upstream workflow is currently awaiting maintainer approval.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Load and LoadAll ignore failbit and badbit on istream

1 participant