Hi Ben,<br><br>Can you please remove the _MSC_VER &gt;= 1600 check?<br><br>_MSC_VER 1600 is set for Visual Studio 2010, which means<br>that Visual Studio 2012 will get a lot of errors because _MSC_VER<br>is defined as 1700.<br>
<br>Cheers,<br>Cristian.<br><br><div class="gmail_quote">On Wed, Mar 6, 2013 at 5:53 PM, Ben Allison <span dir="ltr">&lt;<a href="mailto:benski@winamp.com" target="_blank">benski@winamp.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks, Erik.<br>
<br>
This is the delta that should fix everything up.<br>
I hope that #include &quot;share/compat.h&quot; is acceptable from the test_streams<br>
binary.  If not, you can probably change it to FLAC/ordinals.h instead<br>
<br>
Also don&#39;t forget to commit my earlier VERSION=&quot;1.3.0&quot; patch.  You can add<br>
the &quot;pre1&quot; in a normal text editor without risk.  I didn&#39;t include those<br>
changes in the big patch you committed already.<br>
<span class="HOEnZb"><font color="#888888">-Ben<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
&gt; Ben Allison wrote:<br>
&gt;<br>
&gt;&gt; Erik, et al.<br>
&gt;&gt;<br>
&gt;&gt; This fixes the entire library and does it more cleanly.  It puts most of<br>
&gt;&gt; the guts into share/compat.h<br>
&gt;<br>
&gt; Thanks Ben.<br>
&gt;<br>
&gt; I applied a slightly tweaked version of the patch. The patch had to be<br>
&gt; tweaked because it broke the Linux build :-). Specifically, the file<br>
&gt; include/FLAC/ordinals.h is a public header file and should not include<br>
&gt; the project internal header &quot;share/compat.h&quot;. I also moved some of the<br>
&gt; #ifdef stuff to &quot;share/compat.h&quot;.<br>
&gt;<br>
&gt; What is in git now may not build on windows but should be really, really<br>
&gt; close. I&#39;d appreciate it if you could test it and report the error<br>
&gt; messages. It might take a couple of iterations but we should get it<br>
&gt; working for all the platforms.<br>
&gt;<br>
&gt; Cheers,<br>
&gt; Erik<br>
&gt; --<br>
&gt; ----------------------------------------------------------------------<br>
&gt; Erik de Castro Lopo<br>
&gt; <a href="http://www.mega-nerd.com/" target="_blank">http://www.mega-nerd.com/</a><br>
&gt; _______________________________________________<br>
&gt; flac-dev mailing list<br>
&gt; <a href="mailto:flac-dev@xiph.org">flac-dev@xiph.org</a><br>
&gt; <a href="http://lists.xiph.org/mailman/listinfo/flac-dev" target="_blank">http://lists.xiph.org/mailman/listinfo/flac-dev</a><br>
&gt;<br>
</div></div><br>_______________________________________________<br>
flac-dev mailing list<br>
<a href="mailto:flac-dev@xiph.org">flac-dev@xiph.org</a><br>
<a href="http://lists.xiph.org/mailman/listinfo/flac-dev" target="_blank">http://lists.xiph.org/mailman/listinfo/flac-dev</a><br>
<br></blockquote></div><br>