<br><br><div class="gmail_quote">On Thu, Feb 10, 2011 at 4:26 AM, Charles R Harris <span dir="ltr">&lt;<a href="mailto:charlesr.harris@gmail.com">charlesr.harris@gmail.com</a>&gt;</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;">
<br><br><div class="gmail_quote"><div class="im">On Wed, Feb 9, 2011 at 10:50 PM, David <span dir="ltr">&lt;<a href="mailto:david@silveregg.co.jp" target="_blank">david@silveregg.co.jp</a>&gt;</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,<br>
<br>
in npy3_compat.h, one function simple_capsule_dtor is defined as static<br>
but non-inline. AFAIK, there is no reason not to put inline (if<br>
supported by the compiler of course) for a static function defined in a<br>
header. Unless I hear someone justify it, I will change it,<br>
<br></blockquote></div><div><br>It&#39;s only used as an argument so it needs to be defined, there is no place that it can be inlined. The NpyCapsule stuff could probably be moved out of the header into a compatibility library somewhere but that would require the builds of numpy/scipy to all link with that library.<br>

<br></div></div></blockquote><div><br>That said, inline should work also, but one needs a bit more faith in the compiler...<br><br>Chuck <br></div><br></div>