New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Better overload APIs needed #12129
Comments
From @andkgit bisect commit 50853fa Make overloaded classes inherit fallback sample fail report http://www.cpantesters.org/cpan/report/7d4efbec-a3ba-11e1-b045-c602f4b14d39 perl -V Summary of my perl5 (revision 5 version 17 subversion 0) configuration: Characteristics of this binary (from libperl): -- |
From @cpansproutAlready? :-) -- Father Chrysostomos |
The RT System itself - Status changed from 'new' to 'open' |
From @cpansproutOn Tue May 22 22:48:22 2012, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Well, it turns out that most* of CPAN, not just Mouse, is breaking I also forgot to change ExtUtils::ParseXS to account. I’ve just noticed that, whereas overload.pm says: In the current perlxs.pod says: If you do not set any FALLBACK value when using OVERLOAD, So, what is described as an implementation artefact (something to be So, should ExtUtils::ParseXS follow suit and allow fallback to be I would prefer the former, but I could implement it either way. This * Exaggeration. -- Father Chrysostomos |
From @cpansproutOn Wed May 23 00:01:46 2012, sprout wrote:
I’ve just fixed it in commit 3866ea3, but I don’t particularly like it. -- Father Chrysostomos |
From @cpansproutOn Wed May 23 00:01:46 2012, sprout wrote:
Actually, it didn’t affect the fix, but the question remains. -- Father Chrysostomos |
From @rgarciaOn 23 May 2012 15:03, Father Chrysostomos via RT
2 functions probably: for registering, and for checking if there's |
From @ikegamiOn Wed, May 23, 2012 at 3:01 AM, Father Chrysostomos via RT <
This is a false dichotomy. Set by the base class is still set. |
From @cpansproutOn Wed May 23 06:26:40 2012, rgs@consttype.org wrote:
Just to put this in perspective, these are affected: PP: XS:
I was only thinking in terms of what CPAN modules already do. All the In any case, the registration interface could be: hv_amagic_register(HV *stash, const char *op, SV *value) with a NULL op for registering and a NULL value for the ‘no overload’ I’m suggesting amagic rather than overload in the name, because there For enquiry, which is best? hv_amagic(HV *stash, const char *op) Maybe we should use get and set, for parallelism. -- Father Chrysostomos |
From @jkeenanOn Wed May 23 13:41:26 2012, sprout wrote:
Can we get an update on the status of this ticket? Thank you very much. |
From @cpansproutOn Sat Jun 29 18:30:37 2013, jkeenan wrote:
I think we still need APIs for finding out the overload fallback value -- Father Chrysostomos |
From @jkeenanOn Sat Jun 29 18:58:01 2013, sprout wrote:
At http://www.cpantesters.org/distro/M/Mouse.html, Mouse 2.1.0 is passing on 5.19* and earlier versions on multiple platforms. So the original rationale for this ticket appears to be moot. Father C, could you either rename this ticket to reflect what issues still need discussion? Or perhaps close this and open up a fresh ticket? Thank you very much. |
Migrated from rt.perl.org#113050 (status was 'open')
Searchable as RT113050$
The text was updated successfully, but these errors were encountered: