From b1d3f609e5ea34fb8a00344f3107bc16fd30687e Mon Sep 17 00:00:00 2001
From: "Ignacio R. Morelle" <shadowm@wesnoth.org>
Date: Mon, 6 Jul 2015 23:31:05 -0300
Subject: [PATCH] Use empty() method instead of size() for emptiness checks

Found by cppcheck.
---
 src/halo.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/halo.cpp b/src/halo.cpp
index fa5ccd22c837..ef6ed9dbfdaa 100644
--- a/src/halo.cpp
+++ b/src/halo.cpp
@@ -332,10 +332,10 @@ void remove(int handle)
 
 void unrender(std::set<map_location> invalidated_locations)
 {
-	if(preferences::show_haloes() == false || haloes.size() == 0) {
+	if(preferences::show_haloes() == false || haloes.empty()) {
 		return;
 	}
-	//assert(invalidated_haloes.size() == 0);
+	//assert(invalidated_haloes.empty());
 
 	// Remove expired haloes
 	std::map<int, effect>::iterator itor = haloes.begin();
@@ -409,8 +409,8 @@ void unrender(std::set<map_location> invalidated_locations)
 
 void render()
 {
-	if(preferences::show_haloes() == false || haloes.size() == 0 ||
-			(new_haloes.size() == 0 && invalidated_haloes.size() == 0)) {
+	if(preferences::show_haloes() == false || haloes.empty() ||
+			(new_haloes.empty() && invalidated_haloes.empty())) {
 		return;
 	}