oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

sticker/Database: move code to BindFind()

Browse Source
This commit is contained in:
Max Kellermann 2014-12-12 22:04:43 +01:00
parent 204a1de3fd
commit 71ece56470
2 changed files with 29 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/lib/sqlite/Util.hxx
View File

@ -80,6 +80,19 @@ BindAll(Error &error, sqlite3_stmt *stmt, Args&&... args)
return BindAll2(error, stmt, 1, std::forward<Args>(args)...);
}
/**
* Wrapper for BindAll() that returns the specified sqlite3_stmt* on
* success and nullptr on error.
*/
template<typename... Args>
static sqlite3_stmt *
BindAllOrNull(Error &error, sqlite3_stmt *stmt, Args&&... args)
{
return BindAll(error, stmt, std::forward<Args>(args)...)
? stmt
: nullptr;
}
/**
* Call sqlite3_stmt() repepatedly until something other than
* SQLITE_BUSY is returned.

24
src/sticker/StickerDatabase.cxx
View File

@ -368,6 +368,20 @@ sticker_load(const char *type, const char *uri, Error &error)
return new sticker(std::move(s));
}
static sqlite3_stmt *
BindFind(const char *type, const char *base_uri, const char *name,
Error &error)
{
assert(type != nullptr);
assert(name != nullptr);
if (base_uri == nullptr)
base_uri = "";
return BindAllOrNull(error, sticker_stmt[STICKER_SQL_FIND],
type, base_uri, name);
}
bool
sticker_find(const char *type, const char *base_uri, const char *name,
void (*func)(const char *uri, const char *value,
@ -375,17 +389,11 @@ sticker_find(const char *type, const char *base_uri, const char *name,
void *user_data,
Error &error)
{
sqlite3_stmt *const stmt = sticker_stmt[STICKER_SQL_FIND];
assert(type != nullptr);
assert(name != nullptr);
assert(func != nullptr);
assert(sticker_enabled());
if (base_uri == nullptr)
base_uri = "";
if (!BindAll(error, stmt, type, base_uri, name))
sqlite3_stmt *const stmt = BindFind(type, base_uri, name, error);
if (stmt == nullptr)
return false;
const bool success = ExecuteForEach(stmt, error,