On Mon, 5 Jul 2021 at 23:02, Zebediah Figura zfigura@codeweavers.com wrote:
libs/vkd3d-shader/dxbc.c | 374 +++++++++++---------------------------- 1 file changed, 105 insertions(+), 269 deletions(-)
Conceptually I mostly like this, but it does change quite a bit of code at once; there would certainly be ways to split this, like first implementing the old helpers on top of the new ones, and then gradually replacing them. This also removes some compiler error messages. It may be fine to move/merge those (e.g. we could check for out of memory at the end of vkd3d_shader_serialize_root_signature()), but I don't think we want to remove them entirely.