On Wed Apr 10 13:02:56 2024 +0000, Giovanni Mascellani wrote:
I wouldn't remove the FIXME() here. If anything I'd replace it with something stronger (`vkd3d_unreachable()`), but if for some bad reason we end up here in spite of the validation we did before I'd like to know.
The parser is allowed to continue even if values like these are invalid, so it is reachable. This is a common pattern where nothing prevents continued parsing, so we can potentially find other issues before the parser errors out.