note
pjf
<blockquote><i>why not just make this a patch for Fatal triggered by an option on the use line</i></blockquote>
<p>
This is an excellent idea, and one I really should have considered and mentioned in my original post. I'm very happy with the idea of a patch to Fatal; it keeps everything together, and it certainly simplifies the code.
</p><p>
Downsides of patching Fatal are:
</p>
<ul><li>We'd be changing the behaviour of a core module, which means waiting until 5.10.1 before it's deployed, or making <tt>Fatal</tt> dual-lifed (core + CPAN).</li>
<li>I'd need to get sign-off from p5p, Lionel, or both. That shouldn't be a problem for a good idea, but it does have additional administrative overhead.</li>
<li>We're altering the interface of a very well-known core module to do something that it's never done before. There may be surprises there.</li>
<li>The calling syntax doesn't look like a pragma, but the more I look at it the less I mind. Using <tt>:lexical</tt> I hope makes it pretty obvious what's going on.</li>
</ul>
<p>
The advantages are that it goes in the core for everyone to enjoy, my ability to reuse code in Fatal becomes much much easier, and the functionality goes in the place where I imagine most people will expect it to be.
</p><p>
I'll open discussions with p5p to see their thoughts. Of course, I still very much appreciate feedback on the original proposal as well.
</p><p>
Many thanks,
</p>
<!-- Node text goes above. Div tags should contain sig only -->
<div class="pmsig"><div class="pmsig-116014">
<i>Paul Fenwick</i><br>
<a href="http://perltraining.com.au/">Perl Training Australia</a>
</div></div>
673060
673068