diff options
author | Max Kellermann <max@duempel.org> | 2009-01-18 15:40:28 +0100 |
---|---|---|
committer | Max Kellermann <max@duempel.org> | 2009-01-18 15:40:28 +0100 |
commit | 91fb2a29deaafcc0ae9c710aec3c5cf54ce60d91 (patch) | |
tree | 03c1630db5f1f991dbe7d1cca7dd57793d61f0b3 /src/stats.c | |
parent | 0d449d8df745c5c527f637f42e33156b92ac174d (diff) | |
download | mpd-91fb2a29deaafcc0ae9c710aec3c5cf54ce60d91.tar.gz mpd-91fb2a29deaafcc0ae9c710aec3c5cf54ce60d91.tar.xz mpd-91fb2a29deaafcc0ae9c710aec3c5cf54ce60d91.zip |
stats: added num_artists, num_albums
Don't recalculate the number of artists and albums each time a client
requests statistics. Calculate that once in stats_update().
Diffstat (limited to 'src/stats.c')
-rw-r--r-- | src/stats.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/src/stats.c b/src/stats.c index ce7ecb2d7..119b12456 100644 --- a/src/stats.c +++ b/src/stats.c @@ -33,12 +33,6 @@ void stats_global_init(void) stats.start_time = time(NULL); } -void stats_update(void) -{ - stats.song_count = countSongsIn(NULL); - stats.song_duration = sumSongTimesIn(NULL); -} - struct visit_data { enum tag_type type; struct strset *set; @@ -78,6 +72,14 @@ getNumberOfTagItems(enum tag_type type) return ret; } +void stats_update(void) +{ + stats.song_count = countSongsIn(NULL); + stats.song_duration = sumSongTimesIn(NULL); + stats.artist_count = getNumberOfTagItems(TAG_ITEM_ARTIST); + stats.album_count = getNumberOfTagItems(TAG_ITEM_ALBUM); +} + int stats_print(struct client *client) { client_printf(client, @@ -88,8 +90,8 @@ int stats_print(struct client *client) "playtime: %li\n" "db_playtime: %li\n" "db_update: %li\n", - getNumberOfTagItems(TAG_ITEM_ARTIST), - getNumberOfTagItems(TAG_ITEM_ALBUM), + stats.artist_count, + stats.album_count, stats.song_count, time(NULL) - stats.start_time, (long)(getPlayerTotalPlayTime() + 0.5), |