diff options
| author | Max Kellermann <max@duempel.org> | 2008-10-09 19:17:44 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2008-10-09 19:17:44 +0200 |
| commit | f1022bcc12dbdf5b7974fb1510197b12214fec32 (patch) | |
| tree | cabe38f9e1f728423b03bab187e6c55157ffad8d /src/update.c | |
| parent | 79a28e5c72ae01164b88ae6d8a510f47a92d231f (diff) | |
| download | mpd-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 'src/update.c')
| -rw-r--r-- | src/update.c | 4 |
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; |