On Tue, 28 Dec 2021 at 23:20, Zebediah Figura zfigura@codeweavers.com wrote:
struct vkd3d_shader_parser_ops { void (*parser_reset)(struct vkd3d_shader_parser *parser); void (*parser_destroy)(struct vkd3d_shader_parser *parser);
- void (*parser_read_metadata)(struct vkd3d_shader_parser *parser); void (*parser_read_instruction)(struct vkd3d_shader_parser *parser, struct vkd3d_shader_instruction *instruction); bool (*parser_is_end)(struct vkd3d_shader_parser *parser);
};
I don't feel strongly one way or another at this point, but should parser_read_metadata() perhaps just return the metadata to the caller, instead of storing it in the vkd3d_shader_parser structure?