From 0d6b58a772d515824afd83e8d9cfa9d2e56a1f91 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Blot?= Date: Mon, 1 Jan 2018 18:48:52 +0100 Subject: Add unittests on ActiveObject and BanManager class (#6866) * Add unittests on ActiveObject and BanManager class This also permit to fix a bug in ban manager setting bans modified when no modification occurs --- src/ban.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/ban.cpp') diff --git a/src/ban.cpp b/src/ban.cpp index 053e799be..3decc9666 100644 --- a/src/ban.cpp +++ b/src/ban.cpp @@ -1,6 +1,7 @@ /* Minetest Copyright (C) 2013 celeron55, Perttu Ahola +Copyright (C) 2018 nerzhul, Loic BLOT This program is free software; you can redistribute it and/or modify it under the terms of the GNU Lesser General Public License as published by @@ -125,11 +126,11 @@ void BanManager::remove(const std::string &ip_or_name) for (StringMap::iterator it = m_ips.begin(); it != m_ips.end();) { if ((it->first == ip_or_name) || (it->second == ip_or_name)) { m_ips.erase(it++); + m_modified = true; } else { ++it; } } - m_modified = true; } -- cgit v1.2.3