PrintF is likely broken on x86
Open
area:stdlib
kind:bug
status:help-wanted
- Dominant language
- C#
- Stars
- 451
- Forks
- 49
- Avg merge
- 13h 59m
- Merged PRs (30d)
- 6
Description
See this code: https://github.com/ForNeVeR/Cesium/blob/baff13b0401dac42eaf95e89889129081d390e2f/Cesium.Runtime/StdIoFunctions.cs#L219-L227
`varargs` is casted to a pointer to `nint` or `nuint`.
These types have different sizes on different platforms, while our current `vararg` convention is that we pass an array of 8-byte-sized items (doubles, longs or pointers).
We should also figure out whether it's currently called correctly.
Contributor guide
Assessment
This issue has not been assessed yet.