use warnings;
use strict;
-my $mro_recursor_stack;
+# For the love of everything that is crab-like: DO NOT reach into this
+# The entire thing is really fragile and should not be screwed with
+# unless absolutely and unavoidably necessary
+our $__describe_class_query_cache;
BEGIN {
package # hide from pause
require Digest::MD5;
require Math::BigInt;
+ my $cur_class;
+ no strict 'refs';
+
# the non-assign-unless-there-is-a-hash is deliberate
- ( $mro_recursor_stack->{cache} || {} )->{$_[0]}{gen} ||= (
+ ( $__describe_class_query_cache->{'!internal!'} || {} )->{$_[0]}{gen} ||= (
Math::BigInt->new( '0x' . ( Digest::MD5::md5_hex( join "\0", map {
- ( $mro_recursor_stack->{cache} || {} )->{$_}{methlist} ||= do {
+ ( $__describe_class_query_cache->{'!internal!'} || {} )->{$_}{methlist} ||= (
+
+ $cur_class = $_
- my $class = $_;
- no strict 'refs';
+ and
# RV to be hashed up and turned into a number
join "\0", (
- $class,
+ $cur_class,
map
{(
# stringification should be sufficient, ignore names/refaddr entirely
$_,
- attributes::get( $_ ),
+ do {
+ my @attrs;
+ local $@;
+ local $SIG{__DIE__} if $SIG{__DIE__};
+ # attributes::get may throw on blessed-false crefs :/
+ eval { @attrs = attributes::get( $_ ); 1 }
+ or warn "Unable to determine attributes of coderef $_ due to the following error: $@";
+ @attrs;
+ },
)}
map
{(
# skip dummy C::C3 helper crefs
- ! ( ( $Class::C3::MRO{$class} || {} )->{methods}{$_} )
+ ! ( ( $Class::C3::MRO{$cur_class} || {} )->{methods}{$_} )
and
(
- ref(\ "${class}::"->{$_} ) ne 'GLOB'
+ ref(\ "${cur_class}::"->{$_} ) ne 'GLOB'
or
- defined( *{ "${class}::"->{$_} }{CODE} )
+ defined( *{ "${cur_class}::"->{$_} }{CODE} )
)
)
- ? ( \&{"${class}::$_"} )
+ ? ( \&{"${cur_class}::$_"} )
: ()
}
- keys %{ "${class}::" }
- );
- }
+ keys %{ "${cur_class}::" }
+ )
+ )
} (
@{
- ( $mro_recursor_stack->{cache} || {} )->{$_[0]}{linear_isa}
+ ( $__describe_class_query_cache->{'!internal!'} || {} )->{$_[0]}{linear_isa}
||=
mro::get_linear_isa($_[0])
},
((
- ( $mro_recursor_stack->{cache} || {} )->{$_[0]}{is_universal}
+ ( $__describe_class_query_cache->{'!internal!'} || {} )->{$_[0]}{is_universal}
||=
mro::is_universal($_[0])
) ? () : @{
- ( $mro_recursor_stack->{cache} || {} )->{UNIVERSAL}{linear_isa}
+ ( $__describe_class_query_cache->{'!internal!'} || {} )->{UNIVERSAL}{linear_isa}
||=
mro::get_linear_isa("UNIVERSAL")
} ),
# to be outfits with *COPY PASTED* pieces of lib/DBIx/Class/Storage/*
# in their production codebases. There is no point in breaking these
# if whatever they used actually continues to work
- my $warned;
my $sigh = sub {
-
- require Carp;
- my $cluck = "The @{[ (caller(1))[3] ]} constant is no more - adjust your code" . Carp::longmess();
-
- warn $cluck unless $warned->{$cluck}++;
+ DBIx::Class::_Util::emit_loud_diag(
+ skip_frames => 1,
+ msg => "The @{[ (caller(1))[3] ]} constant is no more - adjust your code"
+ );
0;
};
our @EXPORT_OK = qw(
sigwarn_silencer modver_gt_or_eq modver_gt_or_eq_and_lt
fail_on_internal_wantarray fail_on_internal_call
- refdesc refcount hrefaddr set_subname describe_class_methods
- scope_guard detected_reinvoked_destructor
+ refdesc refcount hrefaddr set_subname get_subname describe_class_methods
+ scope_guard detected_reinvoked_destructor emit_loud_diag
true false
is_exception dbic_internal_try visit_namespaces
quote_sub qsub perlstring serialize deep_clone dump_value uniq
$visited_count;
}
-# FIXME In another life switch this to a polyfill like the one in namespace::clean
+# FIXME In another life switch these to a polyfill like the ones in namespace::clean
+sub get_subname ($) {
+ my $gv = B::svref_2object( $_[0] )->GV;
+ wantarray
+ ? ( $gv->STASH->NAME, $gv->NAME )
+ : ( join '::', $gv->STASH->NAME, $gv->NAME )
+ ;
+}
sub set_subname ($$) {
# fully qualify name
$dump_str;
}
+my $seen_loud_screams;
+sub emit_loud_diag {
+ my $args = { ref $_[0] eq 'HASH' ? %{$_[0]} : @_ };
+
+ unless ( defined $args->{msg} and length $args->{msg} ) {
+ emit_loud_diag(
+ msg => "No 'msg' value supplied to emit_loud_diag()"
+ );
+ exit 70;
+ }
+
+ my $msg = "\n$0: $args->{msg}";
+
+ # when we die - we usually want to keep doing it
+ $args->{emit_dups} = !!$args->{confess}
+ unless exists $args->{emit_dups};
+
+ local $Carp::CarpLevel =
+ ( $args->{skip_frames} || 0 )
+ +
+ $Carp::CarpLevel
+ +
+ # hide our own frame
+ 1
+ ;
+
+ my $longmess = Carp::longmess();
+
+ # different object references will thwart deduplication without this
+ ( my $key = "${msg}\n${longmess}" ) =~ s/\b0x[0-9a-f]+\b/0x.../gi;
+
+ return $seen_loud_screams->{$key} if
+ $seen_loud_screams->{$key}++
+ and
+ ! $args->{emit_dups}
+ ;
+
+ $msg .= $longmess
+ unless $msg =~ /\n\z/;
+
+ print STDERR "$msg\n"
+ or
+ print STDOUT "\n!!!STDERR ISN'T WRITABLE!!!:$msg\n";
+
+ return $seen_loud_screams->{$key}
+ unless $args->{confess};
+
+ # increment *again*, because... Carp.
+ $Carp::CarpLevel++;
+
+ # not $msg - Carp will reapply the longmess on its own
+ Carp::confess($args->{msg});
+}
+
+
###
### This is *NOT* boolean.pm - deliberately not using a singleton
###
1;
}
or
- Carp::cluck(
- "Execution of scope guard $_[0] resulted in the non-trappable exception:\n\n$@"
+ DBIx::Class::_Util::emit_loud_diag(
+ emit_dups => 1,
+ msg => "Execution of scope guard $_[0] resulted in the non-trappable exception:\n\n$@\n "
);
}
}
and
length( my $class = ref $e )
) {
- carp_unique( sprintf(
- "Objects of external exception class '%s' stringify to '' (the "
+ carp_unique(
+ "Objects of external exception class '$class' stringify to '' (the "
. 'empty string), implementing the so called null-object-pattern. '
. 'Given Perl\'s "globally cooperative" exception handling using this '
. 'class of exceptions is extremely dangerous, as it may (and often '
. 'does) result in silent discarding of errors. DBIx::Class tries to '
. 'work around this as much as possible, but other parts of your '
. 'software stack may not be even aware of the problem. Please submit '
- . 'a bugreport against the distribution containing %s',
-
- ($class) x 2,
- ));
+ . "a bugreport against the distribution containing '$class'",
+ );
$not_blank = 1;
}
for keys %$destruction_registry;
if (! length ref $_[0]) {
- printf STDERR '%s() expects a blessed reference %s',
- (caller(0))[3],
- Carp::longmess,
- ;
+ emit_loud_diag(
+ emit_dups => 1,
+ msg => (caller(0))[3] . '() expects a blessed reference'
+ );
return undef; # don't know wtf to do
}
elsif (! defined $destruction_registry->{ my $addr = refaddr($_[0]) } ) {
return 0;
}
else {
- carp_unique ( sprintf (
+ emit_loud_diag( msg => sprintf (
'Preventing *MULTIPLE* DESTROY() invocations on %s - an *EXTREMELY '
. 'DANGEROUS* condition which is *ALMOST CERTAINLY GLOBAL* within your '
. 'application, affecting *ALL* classes without active protection against '
}
{
- # FIXME - should be a private my(), but I'm too uncertain whether
- # all bases are covered
- our $describe_class_query_cache;
sub describe_class_methods {
- my ($class, $requested_mro) = @_;
+ my $args = (
+ ref $_[0] eq 'HASH' ? $_[0]
+ : ( @_ == 1 and ! length ref $_[0] ) ? { class => $_[0] }
+ : { @_ }
+ );
- croak "Expecting a class name"
+ my ($class, $requested_mro) = @{$args}{qw( class use_mro )};
+
+ croak "Expecting a class name either as the sole argument or a 'class' option"
if not defined $class or $class !~ $module_name_rx;
$requested_mro ||= mro::get_mro($class);
# mro::set_mro() does not bump pkg_gen - WHAT THE FUCK?!
my $query_cache_key = "$class|$requested_mro";
- my $stack_cache_key =
+ my $internal_cache_key =
( mro::get_mro($class) eq $requested_mro )
? $class
: $query_cache_key
# we use the cache for linear_isa lookups on new MRO as well - it adds
# a *tiny* speedup, and simplifies the code a lot
#
- local $mro_recursor_stack->{cache} = {}
- unless $mro_recursor_stack->{cache};
+ local $__describe_class_query_cache->{'!internal!'} = {}
+ unless $__describe_class_query_cache->{'!internal!'};
my $my_gen = 0;
$my_gen += get_real_pkg_gen($_) for ( my @full_ISA = (
@{
- $mro_recursor_stack->{cache}{$stack_cache_key}{linear_isa}
+ $__describe_class_query_cache->{'!internal!'}{$internal_cache_key}{linear_isa}
||=
mro::get_linear_isa($class, $requested_mro)
},
((
- $mro_recursor_stack->{cache}{$class}{is_universal}
+ $__describe_class_query_cache->{'!internal!'}{$class}{is_universal}
||=
mro::is_universal($class)
) ? () : @{
- $mro_recursor_stack->{cache}{UNIVERSAL}{linear_isa}
+ $__describe_class_query_cache->{'!internal!'}{UNIVERSAL}{linear_isa}
||=
mro::get_linear_isa("UNIVERSAL")
}),
));
- my $slot = $describe_class_query_cache->{$query_cache_key} ||= {};
+ my $slot = $__describe_class_query_cache->{$query_cache_key} ||= {};
unless ( ($slot->{cumulative_gen}||0) == $my_gen ) {
- # remove ourselves from ISA
- shift @full_ISA;
-
# reset
%$slot = (
class => $class,
- isa => [
- @{ $mro_recursor_stack->{cache}{$stack_cache_key}{linear_isa} }
- [ 1 .. $#{$mro_recursor_stack->{cache}{$stack_cache_key}{linear_isa}} ]
+ isa => { map { $_ => 1 } @full_ISA },
+ linear_isa => [
+ @{ $__describe_class_query_cache->{'!internal!'}{$internal_cache_key}{linear_isa} }
+ [ 1 .. $#{$__describe_class_query_cache->{'!internal!'}{$internal_cache_key}{linear_isa}} ]
],
mro => {
type => $requested_mro,
cumulative_gen => $my_gen,
);
+ # remove ourselves from ISA
+ shift @full_ISA;
+
# ensure the cache is populated for the parents, code below can then
# efficiently operate over the query_cache directly
describe_class_methods($_) for reverse @full_ISA;
# what describe_class_methods for @full_ISA produced above
( map { values %{
- $describe_class_query_cache->{$_}{methods_defined_in_class} || {}
+ $__describe_class_query_cache->{$_}{methods_defined_in_class} || {}
} } map { "$_|" . mro::get_mro($_) } reverse @full_ISA ),
# our own non-cleaned subs + their attributes
) ? {
via_class => $class,
name => $_,
- attributes => {
- map { $_ => 1 } attributes::get( \&{"${class}::${_}"} )
- },
+ attributes => { map { $_ => 1 } do {
+ my @attrs;
+ local $@;
+ local $SIG{__DIE__} if $SIG{__DIE__};
+ # attributes::get may throw on blessed-false crefs :/
+ eval { @attrs = attributes::get( \&{"${class}::${_}"} ); 1 }
+ or warn "Unable to determine attributes of the \\&${class}::$_ method due to following error: $@";
+ @attrs;
+ } },
}
: ()
} keys %{"${class}::"} )