lpereira edited PR #8716.
lpereira edited PR #8716:
The original idea of the filtering here was to avoid passing characters such as the NUL byte, whitespaces, or other things that could cause issues to tools that will consume this data. Since it's impractical to update the list of allowed characters every time a language that
compiles to Wasm generates symbols slightly differently, allow all graphic ASCII characters instead.(Tests don't need updating.)
lpereira commented on PR #8716:
Alright, pushed a new version that only tests for a bad character if it's not a graphic ASCII character.
alexcrichton submitted PR review:
Sorry yeah I don't want to cause too much trouble/headache with this. My gut is that something will go wrong if random emoji is fed to
objdump
and/or mixing in things like right-to-left and left-to-right characters and things like that, but I've got no concrete evidence and everything might handle it all just fine.In any case I think adjustments along the lines of this definitely make sense and are fine to land without trying to investigate deeply how tooling reacts to these symbols (since it's probably all fine anyway)
alexcrichton merged PR #8716.
Last updated: Dec 23 2024 at 12:05 UTC