aboutsummaryrefslogtreecommitdiffstats
path: root/src/update.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2008-10-08 11:06:26 +0200
committerMax Kellermann <max@duempel.org>2008-10-08 11:06:26 +0200
commit02e8c000d1eaa2ff3c81bf59b8293b8912d5be3c (patch)
treef1fd1baa2590c5bf6e38c6599bcedc6f2d8e8b96 /src/update.c
parent1f9b614850f6e350d1c94021fe3723e918f1d259 (diff)
downloadmpd-02e8c000d1eaa2ff3c81bf59b8293b8912d5be3c.tar.gz
mpd-02e8c000d1eaa2ff3c81bf59b8293b8912d5be3c.tar.xz
mpd-02e8c000d1eaa2ff3c81bf59b8293b8912d5be3c.zip
song: song_file_update() returns bool
Instead of returning 0 or -1, return true on success and false on failure. This seems more natural, and when the C library was designed, there was no "bool" data type.
Diffstat (limited to 'src/update.c')
-rw-r--r--src/update.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/src/update.c b/src/update.c
index cc2f69843..f4ac7096c 100644
--- a/src/update.c
+++ b/src/update.c
@@ -232,7 +232,7 @@ updateInDirectory(struct directory *directory, const char *name)
return UPDATE_RETURN_UPDATED;
} else if (st.st_mtime != song->mtime) {
LOG("updating %s\n", name);
- if (song_file_update(song) < 0)
+ if (!song_file_update(song))
delete_song(directory, song);
return UPDATE_RETURN_UPDATED;
}
@@ -421,7 +421,7 @@ static enum update_return updatePath(const char *utf8path)
free(path);
if (song->mtime == mtime)
return UPDATE_RETURN_NOUPDATE;
- else if (song_file_update(song) == 0)
+ else if (song_file_update(song))
return UPDATE_RETURN_UPDATED;
else {
delete_song(parentDirectory, song);