2014-07-12 14:11:52 +02:00
|
|
|
sqlite3: fix build when threads are not used/available
|
|
|
|
|
|
|
|
When threads are not used/available, a function in the sqlite3 extension
|
|
|
|
ends up with a label at the end:
|
|
|
|
|
|
|
|
void _pysqlite_final_callback(sqlite3_context* context)
|
|
|
|
{
|
|
|
|
PyObject* function_result;
|
|
|
|
PyObject** aggregate_instance;
|
|
|
|
int ok;
|
|
|
|
|
|
|
|
#ifdef WITH_THREAD
|
|
|
|
PyGILState_STATE threadstate;
|
|
|
|
|
|
|
|
threadstate = PyGILState_Ensure();
|
|
|
|
#endif
|
|
|
|
|
|
|
|
aggregate_instance = (PyObject**)sqlite3_aggregate_context(context, sizeof(PyObject*));
|
|
|
|
if (!*aggregate_instance) {
|
|
|
|
goto error;
|
|
|
|
}
|
|
|
|
|
|
|
|
[......]
|
|
|
|
|
|
|
|
error:
|
|
|
|
#ifdef WITH_THREAD
|
|
|
|
PyGILState_Release(threadstate);
|
|
|
|
#endif
|
|
|
|
}
|
|
|
|
|
|
|
|
This is not valid, and gcc complains.
|
|
|
|
|
|
|
|
Fix that by adding a dummy statement after the label, so that the label
|
|
|
|
is never the last statement of the function.
|
|
|
|
|
|
|
|
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
|
|
|
|
|
2014-12-28 21:54:54 +01:00
|
|
|
Index: b/Modules/_sqlite/connection.c
|
|
|
|
===================================================================
|
|
|
|
--- a/Modules/_sqlite/connection.c
|
|
|
|
+++ b/Modules/_sqlite/connection.c
|
2014-07-12 14:11:52 +02:00
|
|
|
@@ -786,6 +786,7 @@
|
|
|
|
#ifdef WITH_THREAD
|
|
|
|
PyGILState_Release(threadstate);
|
|
|
|
#endif
|
|
|
|
+ ; /* Make gcc happy: a label can't be at the end of a function */
|
|
|
|
}
|
|
|
|
|
|
|
|
static void _pysqlite_drop_unused_statement_references(pysqlite_Connection* self)
|