Looks good! <br><br><div class="gmail_quote">On Wed, Feb 8, 2012 at 7:38 AM,  <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">
kcc@:<br>
1) Moved auxiliary functions under __interception namespace.<br>
2) Why can't we for simplicity declare real_f inside __interception<br>
namespace as well? </blockquote><div>ok </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This may break if two different libraries using<br>
interception are linked together, but it's a mess anyway, as they'll<br>
have interceptors for the same functions.<br>
3) This CL is large already, so I'd prefer to resolve some FIXMEs after<br>
this CL is submitted, if you don't mind.<br></blockquote><div>ok </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
glider@ suggested we should remove all OS-specific<br>
details from the header "interception.h", so I moved them<br>
to OS-specific headers. In this way the user code will not include<br>
<dlfcn.h> or mach_override.h<div class="im"><br>
<br>
<br>
<a href="http://codereview.appspot.com/5642046/diff/1012/asan_interceptors.h" target="_blank">http://codereview.appspot.com/<u></u>5642046/diff/1012/asan_<u></u>interceptors.h</a><br>
File asan_interceptors.h (right):<br>
<br>
<a href="http://codereview.appspot.com/5642046/diff/1012/asan_interceptors.h#newcode34" target="_blank">http://codereview.appspot.com/<u></u>5642046/diff/1012/asan_<u></u>interceptors.h#newcode34</a><br>
asan_interceptors.h:34: size_t internal_strlen(const char *s);<br></div><div class="im">
On 2012/02/08 08:19:17, glider wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
BTW aren't these functions already declared in asan_internal.h?<br>
</blockquote>
<br></div>
Not yet.<br>
<br>
<a href="http://codereview.appspot.com/5642046/" target="_blank">http://codereview.appspot.com/<u></u>5642046/</a><br>
</blockquote></div><br>