From 91fb2a29deaafcc0ae9c710aec3c5cf54ce60d91 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sun, 18 Jan 2009 15:40:28 +0100 Subject: 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(). --- src/stats.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) (limited to 'src/stats.c') 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), -- cgit v1.2.3