On Tue, Jul 23, 2013 at 12:15 AM, Father Chrysostomos via RT <perlbug-followup@perl.org> wrote: ... > Is this another case where a function was mistakenly added to the API > (like stashpv_hvname_match, added in ed221c5717 and removed in > c947b31cf14), whereas it should simply have been exported (X in embed.fnc)? > > The only use of this function *anywhere* in publicly-viewable code (OK, > just CPAN and the perl core) is the HvPLACEHOLDERS macro in hv.h. > > Only one CPAN module, Data::Alias, uses HvPLACEHOLDERS, and it is like > this: HvPLACEHOLDERS(a2)++. > > So it looks as though it is safe to change > > ApoR |I32* |hv_placeholders_p |NN HV *hv > > to > > XpoR |SSize_t*|hv_placeholders_p |NN HV *hv > > Comments? No objection -- Reini Urban http://cpanel.net/ http://www.perl-compiler.org/Thread Previous | Thread Next