This could be split, one patch implementing GetVariableByIndex() / GetVariableByName() (even separately) with a stub ShaderReflectionVariable and one patch fleshing ShaderReflectionVariable out. Not a huge deal though.
+static const struct ID3D10ShaderReflectionVariableVtbl d3d10_shader_reflection_variable_vtbl = +{
- /* ID3D10ShaderReflectionVariable methods */
- d3d10_shader_reflection_variable_GetDesc,
- d3d10_shader_reflection_variable_GetType,
+};
That comment doesn't seem particularly useful.