Skip to content

Conversation

liny01-nbsa
Copy link
Contributor

I have the same issue, and my screen reader is needed for an automation tool, so I can't disable it. With this code. I have tested it works for after the change.

PS: The code is done w/ Copilot's help.

I have the same issue, and my screen reader is needed for an automation tool, so I can't disable it. With this code. I have tested it works for after the change.

PS: The code is done w/ Copilot's help.
@mame
Copy link
Owner

mame commented Mar 26, 2025

Thanks, looks good! Could you please add a test as well?

$stdout << "\xff"

It would be good enough to change "\xff" in the above line to "dummy\ndummy\ndummy\xff" or something.

@liny01-nbsa
Copy link
Contributor Author

@mame , strange enough, I had some issue running the tests locally. The last case had a few exceptions when I run on my laptop. I prepared a change as you suggested. Let me know if you'd like to make any updates. Thanks!

@mame mame merged commit 3768ffe into mame:main Mar 28, 2025
1 check passed
@mame
Copy link
Owner

mame commented Mar 28, 2025

Thanks!

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.

2 participants