<br><br><div class="gmail_quote">On Wed, Sep 1, 2010 at 9:07 PM, Charles R Harris <span dir="ltr"><<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>Hi Jason,<br><br><div class="gmail_quote"><div class="im">On Tue, Aug 31, 2010 at 2:56 PM, Jason McCampbell <span dir="ltr"><<a href="mailto:jmccampbell@enthought.com" target="_blank">jmccampbell@enthought.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204, 204, 204);padding-left:1ex">
Hi Chuck (and anyone else interested),<div><br></div><div>I updated the refactoring page on the NumPy developer wiki (seems to be down or I'd paste in the link).  It certainly isn't complete, but there are a lot more details about the data structures and memory handling and an outline of some additional topics that needs to be filled in.</div>


<div><br></div></blockquote></div><div> <br>I note that there are some C++ style comments in the code which will cause errors on some platforms, so I hope you are planning on removing them at some point. Also,<br></div></div>
</blockquote><div><br></div><div>Mostly the C++ comments are there for specific things we need to fix before it's complete (easier to search for).  Likely a few are attributable to "muscle memory" in my fingers as well, but all will be removed as we button it up.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;"><div class="gmail_quote"><div><br>if (yes) foo;<br>
<br>is very bad style. There is a lot of that in old code like that that still needs to be cleaned up, but I also see some in the new code. It would be best to get it right to start with.<br></div></div></blockquote><div>
<br></div><div>Agreed.  In the code I have edited I typically re-write it as "if (NULL != yes) foo;" but a lot of code has been copied in wholesale and we haven't always updated that code.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="gmail_quote"><div><snip><br><br>Chuck<br>
</div></div>
<br>_______________________________________________<br>
NumPy-Discussion mailing list<br>
<a href="mailto:NumPy-Discussion@scipy.org">NumPy-Discussion@scipy.org</a><br>
<a href="http://mail.scipy.org/mailman/listinfo/numpy-discussion" target="_blank">http://mail.scipy.org/mailman/listinfo/numpy-discussion</a><br>
<br></blockquote></div><br>