On Sep 1, 12:29 pm, Joel VanderWerf <[email protected]> wrote: > Jeremy Evans wrote: > > We could expand Sequel::Error to store the underlying exception class > > (if any): > > > class Sequel::Error > > attr_accessor :underlying_exception > > end > > > That way all Sequel::Errors would have it, and you'd be able to match > > on the underlying exception, plus it wouldn't break assumptions about > > message being a string. What do you think of that? > > Actually, I tried that, calling it :wrapped_exception instead, and I > discarded it for some dumb reason (maybe to avoid adding an instance > variable).
wrapped_exception sounds better to me too: http://github.com/jeremyevans/sequel/commit/53a1428c4e3e37369cb811640fe4dbb977a39338 --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "sequel-talk" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [email protected] For more options, visit this group at http://groups.google.com/group/sequel-talk?hl=en -~----------~----~----~----~------~----~------~--~---
