<br><br><div class="gmail_quote">On Wed, Feb 1, 2012 at 3:43 PM, David Cournapeau <span dir="ltr">&lt;<a href="mailto:cournape@gmail.com">cournape@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5">On Wed, Feb 1, 2012 at 9:01 PM, Charles R Harris<br>
&lt;<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>&gt; wrote:<br>
&gt; Hi All,<br>
&gt;<br>
&gt; Two things here.<br>
&gt;<br>
&gt; 1) Some macros for threading and the iterator now require a trailing<br>
&gt; semicolon. This change will be reverted before the 1.7 release so that scipy<br>
&gt; 0.10 will compile, but because it is desirable in the long term it would be<br>
&gt; helpful if folks maintaining c extensions using numpy would try compiling<br>
&gt; them against current development and adding the semicolon where needed. The<br>
&gt; added semicolon will be backward compatible with earlier versions of numpy.<br>
&gt;<br>
&gt; 2) It is proposed to deprecate all of the macros in the old_defines.h file<br>
&gt; and require the use of their replacements. Numpy itself will have made this<br>
&gt; change after pull-189 is merged and getting rid of the surplus macros will<br>
&gt; help clean up the historical detritus that has built up over the years,<br>
&gt; easing maintenance, clarifying code, and making the eventual transition to<br>
&gt; 2.0 a bit easier. There is a sed script in the tools directory as part of<br>
&gt; the pull request that can be used to make the needed substitutions.<br>
&gt;<br>
&gt; Thoughts?<br>
<br>
</div></div>Long needed cleanup, thanks for taking care of this.<br>
<br>
Is there a need to review anything, or has this already been merged ?<br>
<br></blockquote><div><br>It hasn&#39;t been merged yet. The main question has been if/how we should deprecate the old macros. The announcement also serves notice ;)<br><br>I&#39;ve got a patch doing this for scipy put haven&#39;t put it up yet.<br>
<br>Chuck <br></div><br></div>