➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2014-06-24 09:49 by vstinner, last changed 2022-04-11 14:58 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| assert_exc.patch | vstinner, 2014-06-24 09:49 | review | ||
| Messages (3) | |||
|---|---|---|---|
| msg221444 - (view) | Author: STINNER Victor (vstinner) *  |
Date: 2014-06-24 09:49 | |
Attached patch detects (when Python is compiled in debug mode) if functions that may clear the current exception are called with an exception set. The check avoids loosing an exception. The problem is that the test_sqlite fails with the patch applied. I will open a new patch for that. I already added similar checks in functions of Python/ceval.c. |
|||
| msg221446 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2014-06-24 09:51 | |
> The problem is that the test_sqlite fails with the patch applied. I will open a new patch for that. I opened the issue #21858 for that. |
|||
| msg222046 - (view) | Author: STINNER Victor (vstinner) * |
Date: 2014-07-01 15:17 | |
I don't think anymore that it's a good idea. Almost all C functions can replace the current Python exception. There is no reason to add assertions only in a few functions. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:58:05 | admin | set | github: 66056 |
| 2014-07-01 15:17:36 | vstinner | set | status: open -> closed resolution: not a bug messages: + msg222046 |
| 2014-06-24 09:51:24 | vstinner | set | messages: + msg221446 |
| 2014-06-24 09:49:31 | vstinner | create | |