<div dir="ltr">On Thu, Jul 18, 2013 at 4:32 PM, Nigel Stewart <span dir="ltr"><<a href="mailto:nigels.com@gmail.com" target="_blank">nigels.com@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello all,<br>
<br>
Another batch of Regal-related apitrace patches for your scrutiny and opinion:<br></blockquote><div><br></div><div>I'll go over them as soon as possible. I really haven't had much time in or out of work for apitrace maintenance over the last weeks... :-/</div>
<div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
1. Conditionally compile-out os::log with -DTRACE_OS_LOG=0<br>
<a href="https://github.com/apitrace/apitrace/pull/164" target="_blank">https://github.com/apitrace/apitrace/pull/164</a><br>
Allows Regal to handle the apitrace logging.<br>
<br>
2. glretrace: GLX and WGL support for ES2/EGL traces.<br>
<a href="https://github.com/apitrace/apitrace/pull/165" target="_blank">https://github.com/apitrace/apitrace/pull/165</a><br>
egltrace isn't happy with my NVIDIA driver, but glretrace works.<br>
<br>
3. glimports: Check for iOS and skip OpenGL and CGL as appropriate.<br>
<a href="https://github.com/apitrace/apitrace/pull/167" target="_blank">https://github.com/apitrace/apitrace/pull/167</a><br>
Needed for the iOS build of Regal<br>
<br>
4. Opt-out of backtrace support with -DTRACE_BACKTRACE=0<br>
<a href="https://github.com/apitrace/apitrace/pull/168" target="_blank">https://github.com/apitrace/apitrace/pull/168</a><br>
I didn't spend much time with the backtrace functionality, but my<br>
initial feeling was that it was too platform-specific for Regal purposes.<br>
Also, I'm concerned about jeopardizing the ease of building Regal.<br></blockquote><div><br></div><div>I'm a bit concerned about 1 and 4.  I don't want to have  -DFOO -DBOO for every bit of functionality that Regal builds wants or not. I'd rather have only one  -DREGAL_BUILD (or something like that) that does everything that needs to be done for regal builds. This way it is clear what are the differences for Regal.</div>
<div><br></div><div>In short, it is as if Regal builds were a different (sub)platform (which sort of is true anyway).</div><div><br></div><div>Jose</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
I think the risk of side-effects for the vanilla apitrace builds are minimal,<br>
these don't add any value to apitrace except to harmonize with Regal<br>
using it as an optional (statically linked) layer.<br>
<br>
I'd like to announce the Regal support for apitrace around Siggraph timeframe,<br>
next week.  The Regal branch of apitrace isn't perfectly aligned with upstream,<br>
but progress is being made here.<br>
<br>
<a href="https://github.com/p3/regal" target="_blank">https://github.com/p3/regal</a><br>
<br>
Thanks in advance,<br>
<br>
- Nigel<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On Fri, Jul 5, 2013 at 12:23 PM, Nigel Stewart <<a href="mailto:nigels.com@gmail.com">nigels.com@gmail.com</a>> wrote:<br>
> Hello all,<br>
><br>
> Cross-posting here on the mailing list of additional opinions and eyeballs.<br>
><br>
> 1. Resolve C4267 MS compiler warnings<br>
> <a href="https://github.com/apitrace/apitrace/pull/156" target="_blank">https://github.com/apitrace/apitrace/pull/156</a><br>
> Use explicit c-casting to appease the MS compiler.<br>
><br>
> 2. Support for Mac OS X 10.5 Leopard<br>
> <a href="https://github.com/apitrace/apitrace/pull/157" target="_blank">https://github.com/apitrace/apitrace/pull/157</a><br>
> CGLShareGroupObj and IOSurfaceRef were added 10.6 SDK.<br>
> Still builds for 10.5 with some modest workarounds.<br>
><br>
> 3. Replace dynamic_cast with virtual functions<br>
> <a href="https://github.com/apitrace/apitrace/pull/158" target="_blank">https://github.com/apitrace/apitrace/pull/158</a><br>
> apitrace can be built with without RTTI if dynamic_cast is avoided.<br>
><br>
> These originate from our effort to integrate apitrace into Regal as a<br>
> dispatch layer.  These patches bring the projects into closer<br>
> alignment.<br>
> <a href="https://github.com/p3/regal" target="_blank">https://github.com/p3/regal</a><br>
><br>
> - Nigel<br>
_______________________________________________<br>
apitrace mailing list<br>
<a href="mailto:apitrace@lists.freedesktop.org">apitrace@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/apitrace" target="_blank">http://lists.freedesktop.org/mailman/listinfo/apitrace</a><br>
</div></div></blockquote></div><br></div></div>