From 4cca75b2e17f3145dfe156285f30b86210662b79 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Fri, 25 Apr 2014 18:36:07 +0200 Subject: DatabasePrint: refactor variable/function names --- src/command/DatabaseCommands.cxx | 4 ++-- src/db/DatabasePrint.cxx | 30 +++++++++++++++--------------- src/db/DatabasePrint.hxx | 12 ++++++------ 3 files changed, 23 insertions(+), 23 deletions(-) diff --git a/src/command/DatabaseCommands.cxx b/src/command/DatabaseCommands.cxx index e1f129d0a..6e9d3caa5 100644 --- a/src/command/DatabaseCommands.cxx +++ b/src/command/DatabaseCommands.cxx @@ -159,7 +159,7 @@ handle_count(Client &client, int argc, char *argv[]) } Error error; - return searchStatsForSongsIn(client, "", &filter, error) + return PrintSongCount(client, "", &filter, error) ? CommandResult::OK : print_error(client, error); } @@ -243,7 +243,7 @@ handle_list(Client &client, int argc, char *argv[]) Error error; CommandResult ret = - listAllUniqueTags(client, tagType, group_mask, filter, error) + PrintUniqueTags(client, tagType, group_mask, filter, error) ? CommandResult::OK : print_error(client, error); diff --git a/src/db/DatabasePrint.cxx b/src/db/DatabasePrint.cxx index 4693b8e08..c65de373f 100644 --- a/src/db/DatabasePrint.cxx +++ b/src/db/DatabasePrint.cxx @@ -169,32 +169,32 @@ db_selection_print(Client &client, const DatabaseSelection &selection, } struct SearchStats { - unsigned numberOfSongs; - unsigned long playTime; + unsigned n_songs; + unsigned long total_time_s; }; static void -printSearchStats(Client &client, const SearchStats &stats) +PrintSearchStats(Client &client, const SearchStats &stats) { client_printf(client, "songs: %u\n" "playtime: %lu\n", - stats.numberOfSongs, stats.playTime); + stats.n_songs, stats.total_time_s); } static bool stats_visitor_song(SearchStats &stats, const LightSong &song) { - stats.numberOfSongs++; - stats.playTime += song.GetDuration(); + stats.n_songs++; + stats.total_time_s += song.GetDuration(); return true; } bool -searchStatsForSongsIn(Client &client, const char *name, - const SongFilter *filter, - Error &error) +PrintSongCount(Client &client, const char *name, + const SongFilter *filter, + Error &error) { const Database *db = client.GetDatabase(error); if (db == nullptr) @@ -203,8 +203,8 @@ searchStatsForSongsIn(Client &client, const char *name, const DatabaseSelection selection(name, true, filter); SearchStats stats; - stats.numberOfSongs = 0; - stats.playTime = 0; + stats.n_songs = 0; + stats.total_time_s = 0; using namespace std::placeholders; const auto f = std::bind(stats_visitor_song, std::ref(stats), @@ -212,7 +212,7 @@ searchStatsForSongsIn(Client &client, const char *name, if (!db->Visit(selection, f, error)) return false; - printSearchStats(client, stats); + PrintSearchStats(client, stats); return true; } @@ -243,9 +243,9 @@ PrintUniqueTag(Client &client, TagType tag_type, } bool -listAllUniqueTags(Client &client, unsigned type, uint32_t group_mask, - const SongFilter *filter, - Error &error) +PrintUniqueTags(Client &client, unsigned type, uint32_t group_mask, + const SongFilter *filter, + Error &error) { const Database *db = client.GetDatabase(error); if (db == nullptr) diff --git a/src/db/DatabasePrint.hxx b/src/db/DatabasePrint.hxx index fd151684e..4e71a7552 100644 --- a/src/db/DatabasePrint.hxx +++ b/src/db/DatabasePrint.hxx @@ -39,13 +39,13 @@ db_selection_print(Client &client, const DatabaseSelection &selection, gcc_nonnull(2) bool -searchStatsForSongsIn(Client &client, const char *name, - const SongFilter *filter, - Error &error); +PrintSongCount(Client &client, const char *name, + const SongFilter *filter, + Error &error); bool -listAllUniqueTags(Client &client, unsigned type, uint32_t group_mask, - const SongFilter *filter, - Error &error); +PrintUniqueTags(Client &client, unsigned type, uint32_t group_mask, + const SongFilter *filter, + Error &error); #endif -- cgit v1.2.3