[asherkin]

https://github.com/zadroot/Clientpre...cleaner.sp#L44

This could print garbage if it can connect to the database but clientprefs isn't loaded, as decl is used for the error buffer.

Originally Posted by asherkin SQLite isn't designed to have multiple connections open to a file, this could easily corrupt the database (and I'm surprised it doesn't error anyway).

Also going to unapprove because of ^

[Dr. McKay]

Originally Posted by asherkin Also going to unapprove because of ^

The persistent parameter in SQL_Connect is true, meaning that it'll attempt to reuse an already-established database connection. Wouldn't this reuse clientprefs' connection to the SQLite database and therefore not cause errors/corruption?

[asherkin]

Originally Posted by Dr. McKay The persistent parameter in SQL_Connect is true, meaning that it'll attempt to reuse an already-established database connection. Wouldn't this reuse clientprefs' connection to the SQLite database and therefore not cause errors/corruption?

Looks like it!

[r3v]

Can someone add MYSQL support, please?