bpo-44047: Remove unused statement argument to _pysqlite_seterror by erlend-aasland · Pull Request #25915 · python/cpython
-
-
Notifications
You must be signed in to change notification settings - Fork 34.7k
Conversation
This looks good to me.
- Is there a way to exercise this change? (Is there is a need to exercise this change via demo that things will continue to work?)
- Is there a way to exercise this change? (Is there is a need to exercise this change via demo that things will continue to work?)
I'll have a look at the coverage report from the test suite in a couple of hours. I'll see if it is possible to squeeze out a couple more corner cases. (Stuff like memory errors is of course hard to do, but not impossible.) I'll keep you posted :)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters