Re: [PATCH 6/8] d3dx9: Factor out free_parameter_data() function.

[Matteo Bruni]

2017-04-20 13:26 GMT+02:00 Paul Gofman <gofmanp(a)gmail.com>:
> +static void free_parameter(struct d3dx_parameter *param, BOOL element, BOOL child)
> +{
> +    unsigned int i;
> +
> +    TRACE("Free parameter %p, name %s, type %s, child %s\n", param, param->name,
> +            debug_d3dxparameter_type(param->type), child ? "yes" : "no");

Again, given that it's a chance for some cleanup, please add the
period at the end of the message. Also I'd change the "child" part,
either just using %#x or child ? "child" : "" (or something else, the
point being that the current "child yes" vs "child no" looks pretty
awkward).
Maybe you can also trace the "element" parameter.
> +    if (param->param_eval)
> +    {
> +        d3dx_free_param_eval(param->param_eval);
> +        param->param_eval = NULL;

Is setting the field to NULL necessary? Same for the others below.