A new patch is attached.<br><br>Marek<br><br><div class="gmail_quote">On Mon, Mar 7, 2011 at 4:16 PM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">On 03/05/2011 09:26 PM, Marek Olšák wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
RenderTexture doesn't have to be called in invalidate_rb, I guess.<br>
---<br>
</blockquote>
<br></div>
The patch looks good but there's one simple optimization that could be made. If glRenderBufferStorage() is called with a new width/height but the internalFormat stays the same, we wouldn't have to invalidate the parent FBO(s).<br>
<br>
I think changing the size of a renderbuffer would be more common than channging its format.<br><font color="#888888">
<br>
-Brian<br>
</font></blockquote></div><br>