summaryrefslogtreecommitdiff
path: root/src/ban.cpp
diff options
context:
space:
mode:
authorLoïc Blot <nerzhul@users.noreply.github.com>2018-01-01 18:48:52 +0100
committerGitHub <noreply@github.com>2018-01-01 18:48:52 +0100
commit0d6b58a772d515824afd83e8d9cfa9d2e56a1f91 (patch)
tree4e9920e88be4bb77ebae7a3dab237af231a4314e /src/ban.cpp
parent9f6d84aee33e1eb14b542fa36f82e0cd7b29e25c (diff)
downloadminetest-0d6b58a772d515824afd83e8d9cfa9d2e56a1f91.tar.gz
minetest-0d6b58a772d515824afd83e8d9cfa9d2e56a1f91.tar.bz2
minetest-0d6b58a772d515824afd83e8d9cfa9d2e56a1f91.zip
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
Diffstat (limited to 'src/ban.cpp')
-rw-r--r--src/ban.cpp3
1 files changed, 2 insertions, 1 deletions
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 <celeron55@gmail.com>
+Copyright (C) 2018 nerzhul, Loic BLOT <loic.blot@unix-experience.fr>
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;
}