Skip to content

Conversation

@xorgy
Copy link

@xorgy xorgy commented May 30, 2025

Fixes #257

@xorgy
Copy link
Author

xorgy commented May 30, 2025

I think this is a reasonable alternative to #261

@resuna
Copy link
Member

resuna commented May 30, 2025

Honestly I prefer the other fix, principle of least astonishment.

@xorgy
Copy link
Author

xorgy commented May 31, 2025

My realization that it's just printable ASCII with 0x20..0x3F after 0x40..5F... what possessed them to do this.
(i + 0x20) ^ 0b01100000.

@xorgy xorgy force-pushed the fix-compilation branch from 480dee6 to db8c160 Compare May 31, 2025 00:24
@xorgy
Copy link
Author

xorgy commented May 31, 2025

Alright, I've made it more tasteful for my own enjoyment. @resuna :+ )

@xorgy xorgy force-pushed the fix-compilation branch from db8c160 to fa20067 Compare May 31, 2025 00:34
@xorgy xorgy force-pushed the fix-compilation branch from fa20067 to 34d2205 Compare May 31, 2025 00:36
@skinkie
Copy link

skinkie commented Jul 8, 2025

And yet another alternative #271 ;-)

@gtjoseph
Copy link

Honestly I think you're overthinking this. #261 is the simplest fix.

@xorgy
Copy link
Author

xorgy commented Jul 27, 2025

I honestly don't care as long as one of them gets merged lol.

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.

Compilation will fail with GCC-15

4 participants