aboutsummaryrefslogtreecommitdiffstats
path: root/src/update.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-10-09 19:17:44 +0200
committerMax Kellermann <max@duempel.org>2008-10-09 19:17:44 +0200
commitf1022bcc12dbdf5b7974fb1510197b12214fec32 (patch)
treecabe38f9e1f728423b03bab187e6c55157ffad8d /src/update.c
parent79a28e5c72ae01164b88ae6d8a510f47a92d231f (diff)
downloadmpd-f1022bcc12dbdf5b7974fb1510197b12214fec32.tar.gz
mpd-f1022bcc12dbdf5b7974fb1510197b12214fec32.tar.xz
mpd-f1022bcc12dbdf5b7974fb1510197b12214fec32.zip
update: job ID must be positive
The documentation for directory_update_init() was incorrect: a job ID must be positive, not non-negative. If the update queue is full and no job was created, it makes more sense to return 0 instead of -1, because it is more consistent with the return value of isUpdatingDB().
Diffstat (limited to '')
-rw-r--r--src/update.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/update.c b/src/update.c
index 9cb3739be..a962cca98 100644
--- a/src/update.c
+++ b/src/update.c
@@ -436,9 +436,9 @@ int directory_update_init(char *path)
int next_task_id;
if (!path)
- return -1;
+ return 0;
if (update_paths_nr == ARRAY_SIZE(update_paths))
- return -1;
+ return 0;
assert(update_paths_nr < ARRAY_SIZE(update_paths));
update_paths[update_paths_nr++] = path;
next_task_id = update_task_id + update_paths_nr;