<p dir="ltr"><br>
Am 28.11.2013 17:07 schrieb "Stefan Dösinger" <<a href="mailto:stefandoesinger@gmail.com">stefandoesinger@gmail.com</a>>:<br>
><br>
><br>
> Am 28.11.2013 um 15:56 schrieb Henri Verbeet <<a href="mailto:hverbeet@gmail.com">hverbeet@gmail.com</a>>:<br>
><br>
> > On 28 November 2013 10:48, Stefan Dösinger <<a href="mailto:stefan@codeweavers.com">stefan@codeweavers.com</a>> wrote:<br>
> >> /* Read the framebuffer back into the surface */<br>
> >> -static void read_from_framebuffer(struct wined3d_surface *surface, void *dest, UINT pitch)<br>
> >> +static void read_from_framebuffer(struct wined3d_surface *surface,<br>
> >> +        const struct wined3d_bo_address *data, UINT pitch)<br>
> > It seems a bit silly to pass these as parameters when the only caller<br>
> > gets them from the surface.<br>
> It keeps the switch between resource->allocatedMemory and PBO in a central place. My goal is to remove resource->allocatedMemory and have surface_get_data and surface_map choose between PBO, heap_memory, user-memory set by surface_set_memory and the DIB.<br>

><br>
I guess you meant that I can call surface_get_memory in the function itself...<br>
</p>