Switch several caller() invocations to explicit CORE::caller()
authorPeter Rabbitson <ribasushi@cpan.org>
Thu, 8 Oct 2015 19:18:49 +0000 (21:18 +0200)
committerPeter Rabbitson <ribasushi@cpan.org>
Thu, 15 Oct 2015 15:33:55 +0000 (17:33 +0200)
commit821edc0964a64b9d20b7d02c4a738b87e806f32d
treebecbd60792d3b5e34f272a93e7b0dedf51f5f240
parent07fadea8d16b657cc1ee572b8f64fa7a688be853
Switch several caller() invocations to explicit CORE::caller()

This not only fixes an obscure test failure due to an older Sub::Uplevel
(it would be too obnoxious to bump the dep just for the test case), but also
makes the entire codebase more robust in light of possible rogue/incomplete
caller() overrides

This was not a simple s/// job - each change was manually evaluated before
carrying out

Also while at it - fix the utterly annoying *UNKNOWN* eception site-marker:
it does not add any information and only confuses things. The heuristics in
Carp::Skip is supposed to be much much clearer, need to finish its tests...
Changes
lib/DBIx/Class/Carp.pm
lib/DBIx/Class/Exception.pm
lib/DBIx/Class/_Util.pm
t/lib/DBICTest/Util.pm
t/lib/DBICTest/Util/LeakTracer.pm
t/storage/txn_scope_guard.t