From 4cca75b2e17f3145dfe156285f30b86210662b79 Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@duempel.org>
Date: Fri, 25 Apr 2014 18:36:07 +0200
Subject: [PATCH] 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