<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 3, 2015 at 3:26 PM, Kuba Brecka <span dir="ltr"><<a href="mailto:kuba.brecka@gmail.com" target="_blank">kuba.brecka@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Landed in r231162.<br>
<span class=""><br>
> LGTM. Do you plan to inherit WinSymbolizer from SymbolizerTool?<br>
<br>
<br>
</span>Yes, although it's not as straight-forward as I'd like. I planned to move the implementation of `POSIXSymbolizer` into just `Symbolizer`, which would no longer have any subclasses. The `ChooseSymbolizer` would then simply have a `if (SANITIZER_WINDOWS)` branch in it, and we wouldn't need `PlatformInit` at all.<br></blockquote><div><br></div><div>Right, I also think we should do that, if possible (maybe, re-implement ChooseSymbolizer() on Windows to avoid having #ifdefs) and get rid of PlatformInit. In that case Symbolizer constructor would just initialize symbolozer_tool_ with the result of platform-specific ChooseSymbolizer().</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Speaking of Windows, I found out that we currently don't run the testcases from the main `TestCases/` directory on Windows (we treat them as if they were in `TestCases/Posix/`), is that intentional?  If yes, what's the point of having a separate `Posix` subdirectory?<br></blockquote><div><br></div><div>That's right. I believe we would eventually run all TestCases/ under Windows as well, but for now we're just not ready for the switch.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://reviews.llvm.org/D8029" target="_blank">http://reviews.llvm.org/D8029</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">Alexey Samsonov<br><a href="mailto:vonosmas@gmail.com" target="_blank">vonosmas@gmail.com</a></div></div>
</div></div>