Michael Graham wrote:
> As for implementation, is there an easy way to do this without requiring
> Class::ISA? My patch didn't deal at all with the order of classes in
> the hierarchy. It sorted callbacks based on numeric position only. If
> two callbacks were installed at the same position, then they would get
> run in an arbitrary order (at that position).
[snip]
> AFAIK, Class::ISA is a core module on 5.8.x, but not on 5.6.x or
> earlier.
[snip]
> It looks like Class::ISA handles multiple inheritance fine. Without
> Class::ISA, I imagine it would be bit of a pain to build the inheritance
> tree when multiple inheritance is involved.
I would much rather have the extra requirement that is stable enough to
make it into the core rather than try and impelement this ourselves,
especially with multiple inheritance in the picture.
I'm also not sure I like the runtime requirement either. While it does
provide protection from this additional dependency it causes extra
headache for someone who may not know the dependency exists. Maybe if
it's listed as an optional dependency it wouldn't be so bad, so that
CPAN could still follow it if the user wants...
--
Michael Peters
Developer
Plus Three, LP
---------------------------------------------------------------------
Web Archive: http://www.mail-archive.com/suppressed/
http://marc.theaimsgroup.com/?l=cgiapp&r=1&w=2
To unsubscribe, e-mail: suppressed
For additional commands, e-mail: suppressed
Mail converted by mhonarc 2.6.15
This archive provided courtesy of JSW4.NET, Internet Hosting Services for Small Business.