<div class="gmail_quote">2011/9/22 Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Thu, Sep 22, 2011 at 10:33 AM, Ruben Van Boxem<br>
<<a href="mailto:vanboxem.ruben@gmail.com">vanboxem.ruben@gmail.com</a>> wrote:<br>
> 2011/6/30 Ruben Van Boxem <<a href="mailto:vanboxem.ruben@gmail.com">vanboxem.ruben@gmail.com</a>><br>
</div><div class="im">> - include/__config/type_traits: add a define _LIBCXX_HAS_FEATURE_IS_BASE_OF<br>
> for MSVC and GCC, which both have this, but not __has_feature, which is<br>
> Clang-only. Modify <type_traits> accordingly.<br>
<br>
</div>I noticed that you're testing _MSC_VER 1400, which is VS 2005.<br>
However, I couldn't find any documentation of is_base_of for anything<br>
before Visual Studio 2008 (_MSC_VER 1500).  Unfortunately, I don't<br>
have VS 2005 to test for sure, but this may require a second look.  Or<br>
I could be wrong.  ;-)<br></blockquote><div><br>MSVS2005 page detailing __is_base_of. the 2003 page doesn't have this page, so I assume it does not support it.<br><a href="http://msdn.microsoft.com/en-us/library/ms177194%28v=vs.80%29.aspx">http://msdn.microsoft.com/en-us/library/ms177194%28v=vs.80%29.aspx</a><br>
 </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im"><br>
> All this should work for both MinGW(-w64)/GCC and Microsoft/MSVC<br>
> headers/libraries. Note the *_l functions are only available on recent<br>
> Windows versions, which should be good enough for now. Especially the locale<br>
> stuff (up till now!) should work for more than only plain "C" locale.<br>
<br>
</div>It's more dependent on the MSVCRT version on the system than the OS.<br>
I've seen the _l versions of the APIs with VS 2005, and the claimed<br>
support (assuming the proper redist is installed) is as far back as<br>
Windows 95.<br></blockquote><div><br>Ok, good to know. After checking MSDN again, I did find all these functions in VS2005 doc pages, so that could mean XP has decent support (if kept up to date). I never understood how Windows, msvcrt.dll and the redistributable packages worked...<br>
 </div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Overall, I think the patch looks good though.<br></blockquote><div><br>Great news, thanks!<br><br>Ruben <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<font color="#888888"><br>
~Aaron<br>
</font></blockquote></div><br>