On Mon, Mar 14, 2016 at 1:41 PM, Henri Verbeet hverbeet@gmail.com wrote:
On 14 March 2016 at 00:03, Józef Kucia jkucia@codeweavers.com wrote:
Signed-off-by: Józef Kucia jkucia@codeweavers.com
dlls/wined3d/device.c | 18 +----------------- dlls/wined3d/surface.c | 22 ++-------------------- dlls/wined3d/view.c | 40 ++++++++++++++++++++++++++++++++++++++++ dlls/wined3d/wined3d_private.h | 3 +++ 4 files changed, 46 insertions(+), 37 deletions(-)
I'm not sure how I feel about this change. There doesn't seem to be a particularly compelling reason to change this, although I suppose there's no strong reason not to either. I guess it depends on where you're going with this.
This change allows to make surface_color_fill() static and internal to surface.c. If we want we may also get rid of surface_depth_fill() and surface_color_fill() as they are merely simple wrappers around wined3d_rendertarget_view_clear().
However, the main idea behind this change is to put the implementation of render target view clears in wined3d_rendertarget_view_clear() and call it from a wined3d_cs_op_handler for WINED3D_CS_OP_CLEAR_RENDERTARGET_VIEW. wined3d_device_clear_rendertarget_view() would just emit WINED3D_CS_OP_CLEAR_RENDERTARGET_VIEW.