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.

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

testprocess: More debug logging #11007

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Conversation

smcv
Copy link
Contributor

@smcv smcv commented Sep 30, 2024

Attempting to debug #11006.

  • testprocess: Update iteration count as (presumably) intended

  • testprocess: Show log messages as (presumably) intended

    The result of the % operator will be in the range 0 to 31, so it would
    never be 32, but it can be 0.

  • testprocess: Log a message when all data has been written

    This will make it more obvious when writing has finished and we are only
    waiting for reading.

  • testprocess: Log on every iteration

  • testprocess: Improve debug logging

The result of the `%` operator will be in the range 0 to 31, so it would
never be 32, but it can be 0.

Helps: libsdl-org#11006
Signed-off-by: Simon McVittie <[email protected]>
This will make it more obvious when writing has finished and we are only
waiting for reading.

Helps: libsdl-org#11006
Signed-off-by: Simon McVittie <[email protected]>
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.

1 participant