2009/3/30 Stefan Dösinger <ste...@codeweavers.com>:
> -    hr = IWineD3DDevice_CreateBuffer(This->wined3d_device, &wined3d_desc,
> -            (IUnknown *)object, &object->wined3d_buffer);
> +    hr = IWineD3DDevice_CreateBuffer(This->wined3d_device, desc->ByteWidth,
> +            desc->Usage, 0, WINED3DFMT_UNKNOWN, WINED3DPOOL_MANAGED,·
> +            &object->wined3d_buffer, 0, (IUnknown *)object);
You're adding trailing space here. More importantly though, the
wined3d_desc variable is now essentially unused. I already mentioned
on IRC a couple of weeks ago that I don't think changing the
CreateBuffer() call like this is an improvement.

I also mentioned that I think the patch is too big, I really don't
want to fix regressions in this. The patch can be split in at least 4
separate patches without trying too hard.


Reply via email to