diff options
| author | Max Kellermann <max@duempel.org> | 2008-10-08 11:06:26 +0200 |
|---|---|---|
| committer | Max Kellermann <max@duempel.org> | 2008-10-08 11:06:26 +0200 |
| commit | 02e8c000d1eaa2ff3c81bf59b8293b8912d5be3c (patch) | |
| tree | f1fd1baa2590c5bf6e38c6599bcedc6f2d8e8b96 /src/song.c | |
| parent | 1f9b614850f6e350d1c94021fe3723e918f1d259 (diff) | |
| download | mpd-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 '')
| -rw-r--r-- | src/song.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/song.c b/src/song.c index 44aba9736..24fd2d733 100644 --- a/src/song.c +++ b/src/song.c @@ -97,7 +97,7 @@ song_free(struct song *song) free(song); } -int +bool song_file_update(struct song *song) { struct decoder_plugin *plugin; @@ -120,9 +120,9 @@ song_file_update(struct song *song) song->tag = plugin->tag_dup(abs_path); if (song->tag == NULL || song->tag->time < 0) - return -1; + return false; - return 0; + return true; } char * |