<br><br><div class="gmail_quote">On Wed, Feb 22, 2012 at 5:16 PM,  <span dir="ltr"><<a href="mailto:samsonov@google.com">samsonov@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 2012/02/22 13:04:16, timurrrr_at_google_com wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Alexander, Evgeniy,<br>
</blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Can you please review this patch?<br>
It adds some basic support for function interception on Windows.<br>
</blockquote>
<br>
IMO InitializeAllInterceptors() really does look horrible now :)<br>
I think we need to use functions like:<br>
InitializeSignalInterceptors()<br>
InitializePthreadInterceptors(<u></u>)<br>
InitializeGCDInterceptors()<br>
and so on. Probably these functions can be defined in their own .cc<br>
file, but even now they may simplify things.<br>
<br>
Also, what about using if (ASAN_WINDOWS)?<br></blockquote><div><br></div><div>omg, if (ASAN_WINDOWS) can be a problem as we'll still need to resolve symbols</div><div>that can be undefined on a given os.</div><div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks,<br>
Timur<br>
</blockquote>
<br>
<br>
<br>
<a href="http://codereview.appspot.com/5690065/" target="_blank">http://codereview.appspot.com/<u></u>5690065/</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov</div><div>Software Engineer, Moscow</div><div><a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a></div><br>