<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Oct 21, 2013 at 5:54 PM, Alexander Monakov <span dir="ltr"><<a href="mailto:amonakov@ispras.ru" target="_blank">amonakov@ispras.ru</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
On Thu, 17 Oct 2013, Carl Worth wrote:<br>
<br>
> This enables replay of a trace captured on ChromeOS.<br>
><br>
> Note: Replaying on EGL is not supported, (will trigger an abort),<br>
> since, as far as I know, EGL does not provide an equivalent function.<br>
<br>
</div>My preference would be to issue a warning and continue replaying; this is what<br>
happens on GLX if the underlying driver does not implement this function.<br>
<br>
Other than that, I looked at the patch and it looks good to me.<br>
<br>
I'll note that this function, like glFinish and glFlush, is almost, but not<br>
quite, a candidate for "this call concludes drawing a frame" flag.<br></blockquote></div></div><div class="gmail_extra"><br></div><div class="gmail_extra">Yes, I also think that a warning would be more consistent with what we handle elsewhere. We tend to only abort on context/device initialization, as that tends to lead to crashes anyway.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Otherwise it also looks good to me.</div><div class="gmail_extra"><br></div><div class="gmail_extra">BTW, if anybody cares, it might be possible to implement this on other platforms via glRead/WritePixels.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Jose</div></div>