It'd be cool to prevent spreading the checking around indeed. I don't know how
we could do that cleanly, though. In a certain way, Connection/Result are already
a wrapper to the real Connection/Cursor objects. Introducing another wrapper
might only postpone the same kind of checks to another layer. In any case, if
you have suggestions on how to do it cleanly, please let me know.
[2]
Oops. I think I've fixed them all now.
[3]
Yeah. :-(
[5]
I've now added a "@raise DisconnectionError: ..." to all affected methods in database.py.
[1]
It'd be cool to prevent spreading the checking around indeed. I don't know how
we could do that cleanly, though. In a certain way, Connection/Result are already
a wrapper to the real Connection/Cursor objects. Introducing another wrapper
might only postpone the same kind of checks to another layer. In any case, if
you have suggestions on how to do it cleanly, please let me know.
[2]
Oops. I think I've fixed them all now.
[3]
Yeah. :-(
[5]
I've now added a "@raise DisconnectionError: ..." to all affected methods in database.py.
Does that sound ok?
Thanks for the review!