There were some changes to the functions which included passing the db handle as a parameter, rather than using the global db variable that is set by the open_database function when the server is started. This resulted in an error due to sqlite reporting that the database was locked. I've added a commit which temporarily removes the db handle as a function parameter, which has cured the problem. However, outside of C/C++, global variables are regarded as ebul. Do we feel the same and, if so, should be avoid a global db handle.
There were some changes to the functions which included passing the db handle as a parameter, rather than using the global db variable that is set by the open_database function when the server is started. This resulted in an error due to sqlite reporting that the database was locked. I've added a commit which temporarily removes the db handle as a function parameter, which has cured the problem. However, outside of C/C++, global variables are regarded as ebul. Do we feel the same and, if so, should be avoid a global db handle.