<div dir="ltr">This also fixes the attach of a valid buffer after attaching a NULL one. Currently the surface is showing the wrong buffer.</div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/6/11 Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri,  7 Jun 2013 16:52:44 +0300<br>
Ander Conselvan de Oliveira <<a href="mailto:ander.conselvan.de.oliveira@intel.com">ander.conselvan.de.oliveira@intel.com</a>><br>
wrote:<br>
<div class="im"><br>
> Checking for gs->num_images for determining the previous buffer type<br>
> when attaching is not reliable. The number of images is never cleared<br>
> in the SHM path, so after a switch from an EGL buffer to SHM, every<br>
> following attach of an SHM buffer will happen with gs->num_images > 0,<br>
> and the code will assume the previous buffer was an EGL one.<br>
><br>
> Fix this by adding a buffer_type field to gl_surface_state.<br>
> ---<br>
>  src/gl-renderer.c |   21 +++++++++++++++++----<br>
>  1 file changed, 17 insertions(+), 4 deletions(-)<br>
<br>
</div>I like tracking buffer type explicitly, rather than inferring from<br>
num_images.<br>
<br>
This series looks ok and seems to work ok from what little testing I<br>
did.<br>
<br>
<br>
Thanks,<br>
pq<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org">wayland-devel@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/mailman/listinfo/wayland-devel</a><br>
</div></div></blockquote></div><br></div>