Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SUREFIRE-2058] Corrupted STDOUT by directly writing to native stream in forked JVM 1 with UTF-8 console logging #518

Merged
merged 3 commits into from Apr 26, 2022
Merged

[SUREFIRE-2058] Corrupted STDOUT by directly writing to native stream in forked JVM 1 with UTF-8 console logging #518

merged 3 commits into from Apr 26, 2022

Commits on Apr 25, 2022

  1. [SUREFIRE-2058] Add readString unit test covering cases with overflow… 

    …ing output buffer

- shouldReadStringOverflowOnNewLine - ends up with 1 single byte (LF) remaining on input buffer
- shouldReadStringOverflowOn4BytesEncodedSymbol - causing an infinite loop with 4 bytes left on input buffer
    @zoltanmeze
    zoltanmeze committed Apr 25, 2022
    Copy the full SHA
    a771bbb View commit details
    Browse the repository at this point in the history

  2. [SUREFIRE-2058] Flip and clear output char buffer after each chunk read 

    Overflow can happen even when output buffer has still some remaining space left
    @zoltanmeze
    zoltanmeze committed Apr 25, 2022
    Copy the full SHA
    ef373d7 View commit details
    Browse the repository at this point in the history

  3. [SUREFIRE-2058] Add static import for emptyMap and remove explicit ty… 

    …pe arguments
    @zoltanmeze
    zoltanmeze committed Apr 25, 2022
    Copy the full SHA
    2a72800 View commit details
    Browse the repository at this point in the history