On Sun, Mar 15, 2009 at 12:42:19PM -0000, Ville M. Vainio wrote:
> On Sun, Mar 15, 2009 at 2:27 PM, Gael Varoquaux
> <email address hidden> wrote:
> > This is simply _horrible_: module-level monkey patching. In other words:
> > huge side effects. I don't think this solution is acceptabe, and I'd like
> > this bug reopened, admitedly with a milestone set for later than 0.10.
> I agree that it's horrible. A proper solution would be to implement
> the crash handler as a hook (and have current crash handler as the
> default handler if hook raises TryNext).
On Sun, Mar 15, 2009 at 12:42:19PM -0000, Ville M. Vainio wrote:
> On Sun, Mar 15, 2009 at 2:27 PM, Gael Varoquaux
> <email address hidden> wrote:
> > This is simply _horrible_: module-level monkey patching. In other words:
> > huge side effects. I don't think this solution is acceptabe, and I'd like
> > this bug reopened, admitedly with a milestone set for later than 0.10.
> I agree that it's horrible. A proper solution would be to implement
> the crash handler as a hook (and have current crash handler as the
> default handler if hook raises TryNext).
That sounds good.
Gaël