<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On 8 Mar 2012, at 20:50, Zack Rusin wrote:</div><blockquote type="cite"><div><font class="Apple-style-span" color="#000000"><br></font>Cool. FWIW, ApiTrace has already been made to work on Android by the LunarG <br>folks. I'm ccing Chia-I Wu to see if he wants to say anything about it.<br></div></blockquote><div><br></div>Great!</div><div><br><blockquote type="cite"><div>From my side, I really dislike the #if android's and the cmake patches are <br>very ugly. Also importing GL headers to compile a GL application used to debug <br>GL applications seems at best crazy.<br>But I have very little time for fun projects nowadays and I'm sure Jose will <br>do a proper review when he'll be back from his vacations.<br></div></blockquote></div><br><div>I also dislike ifdefs, but for such small changes I'm not sure which is worse - ifdefing the small changes, or putting said functions in Android-specific files.</div><div><br></div><div>I tried to make the cmake changes as uninvasive as possible. Can you suggest how to clean it up - I'm not sure what your specific concerns are.</div><div><br></div><div>When I spoke to Jose about the GL headers being imported he suggested that this way was the best way to deal with the issue of apitrace supporting several different GL APIs for tracing at runtime, but no way at choose at compile time which ones to compile in. I originally looked into getting apitrace to only build certain APIs like GLES2 at compile time but it proved to be a huge undertaking and I was on a time budget… Do you have any other possibilities for how to deal with this problem?</div><div><br></div><div>Thanks,</div><div><br></div><div>George</div></body></html>