develooper Front page | perl.perl5.porters | Postings from May 2011

Re: [PATCH] export store_cop_label

Thread Previous | Thread Next
From:
Joshua ben Jore
Date:
May 26, 2011 11:19
Subject:
Re: [PATCH] export store_cop_label
Message ID:
BANLkTikf-j28XE=HdthDHPUpkn1+L0H_ug@mail.gmail.com
On Thu, May 26, 2011 at 1:26 AM, Jan Dubois <jand@activestate.com> wrote:
> On Wed, 25 May 2011, Joshua ben Jore wrote:
>> On Wed, Apr 13, 2011 at 12:48 PM, Jan Dubois <jand@activestate.com> wrote:
>>> B::C is not in ext/ inside the core, so "EX" does not apply. If you
>>> want to use it from CPAN, is has to be "A". From the core point of
>>> view, nothing on CPAN is "more special" than everything else.
>>>
>>> Claiming upfront that you intend to ignore the spirit of the rules
>>> anyways just makes it so much harder to get your patch accepted, so
>>> please don't do that!
>>
>> There's utility in exposing things but not making promises about
>> stable APIs. If that's the point to this change, then I find sympathy
>> with it.
>
> I'm not sure what you are arguing for here:
>
> b) ALL internal non-static functions should be exported in case some
>   module finds utility in them.  But unless the function is marked
>   as part of the public API we reserve the right to change at will.

I tend to think Reini ought to be able to get his internal non-static
function exported because he has actual utility in it. It isn't part
of the public supported API so it doesn't have any promises about
being stable between patch releases. We absolutely shouldn't
prematurely document and fix it's behavior into stability but there's
little point to preventing something useful from getting access to it.

Josh

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About