https://bugs.ruby-lang.org/https://bugs.ruby-lang.org/favicon.ico?17113305112012-11-07T23:38:07ZRuby Issue Tracking SystemRuby master - Bug #7302: r37497 changes rb_enumeratorize without NEWShttps://bugs.ruby-lang.org/issues/7302?journal_id=325642012-11-07T23:38:07Zmarcandre (Marc-Andre Lafortune)marcandre-ruby-core@marc-andre.ca
<ul></ul><p>Oh, please excuse my ignorance. I'll revert <code>rb_enumeratorize</code> to what it was and create <code>rb_enumeratorize_with_size</code> instead, as you suggest.</p>
<p>I thought that functions in intern.h were internal to CRuby... So, which headers are considered part of the Ruby CAPI?</p> Ruby master - Bug #7302: r37497 changes rb_enumeratorize without NEWShttps://bugs.ruby-lang.org/issues/7302?journal_id=325852012-11-08T03:50:55Znaruse (Yui NARUSE)naruse@airemix.jp
<ul></ul><p>marcandre (Marc-Andre Lafortune) wrote:</p>
<blockquote>
<p>Oh, please excuse my ignorance. I'll revert <code>rb_enumeratorize</code> to what it was and create <code>rb_enumeratorize_with_size</code> instead, as you suggest.</p>
<p>I thought that functions in intern.h were internal to CRuby... So, which headers are considered part of the Ruby CAPI?</p>
</blockquote>
<p>intern.h was meant to internal CAPI as you thought.<br>
But it is install to system and includes many essential CAPIs like rb_ary_<em>, rb_hash_</em>, and so on.<br>
Therefore people used it as if it is public, and we must treat it as public while they are still unofficial.</p>
<p>Because of this failure, we now created true internal header file, internal.h.<br>
(some says those header files should be restructured, but it is still not worked)</p> Ruby master - Bug #7302: r37497 changes rb_enumeratorize without NEWShttps://bugs.ruby-lang.org/issues/7302?journal_id=325872012-11-08T04:47:12Zmarcandre (Marc-Andre Lafortune)marcandre-ruby-core@marc-andre.ca
<ul><li><strong>Status</strong> changed from <i>Assigned</i> to <i>Closed</i></li></ul><p>naruse (Yui NARUSE) wrote:</p>
<blockquote>
<p>Therefore people used it as if it is public, and we must treat it as public while they are still unofficial.</p>
</blockquote>
<p>Thank you for the explanation.</p>
<blockquote>
<p>Because of this failure, we now created true internal header file, internal.h.<br>
(some says those header files should be restructured, but it is still not worked)</p>
</blockquote>
<p>Count me in the "some" :-)</p>
<p>Fixed. Thanks for checking my commits.</p>