Fix some more warnings

src/database/Database.cpp

@@ -45,7 +45,7 @@ namespace db {
 		bool operator()(const std::unique_ptr< Table > &ptr) { return ptr && ptr->getName() == name; }
 	};
 
-	Database::Database(Backend backend) : m_backend(backend){};
+	Database::Database(Backend backend) : m_backend(backend){}
 
 	void Database::init(const ConnectionParameter &parameter) {
 		assert(parameter.applicability() == m_backend);

src/database/Table.h

@@ -22,7 +22,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace db {

src/murmur/database/ACLTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/BanTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/ChannelLinkTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/ChannelListenerTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/ChannelPropertyTable.h

@@ -19,7 +19,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/ChannelTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/ConfigTable.h

@@ -14,7 +14,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/DBBan.cpp

@@ -100,7 +100,7 @@ namespace server {
 			// byte address. Thus, the subnet mask that has to be applied to these last 4 bytes also needs to ignore the
 			// leading 12 bytes = 96 bits.
 			assert(mask <= 32);
-			return mask + 96;
+			return static_cast< std::uint8_t >(mask + 96);
 		}
 
 		bool operator==(const DBBan &lhs, const DBBan &rhs) {

src/murmur/database/GroupMemberTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/GroupTable.h

@@ -18,7 +18,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/LogTable.h

@@ -15,7 +15,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/ServerTable.h

@@ -13,7 +13,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/UserPropertyTable.h

@@ -20,7 +20,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 namespace mumble {
 namespace server {

src/murmur/database/UserTable.h

@@ -23,7 +23,7 @@
 
 namespace soci {
 class session;
-};
+}
 
 class QByteArray;
 

src/tests/TestDatabase/DatabaseTest.cpp

@@ -117,7 +117,7 @@ template<> char *toString(const std::unordered_set< std::string > &set) {
 	return buffer;
 }
 
-}; // namespace QTest
+} // namespace QTest
 
 
 class TestDatabase : public Database {

src/tests/TestDatabase/server/ServerDatabaseTest.cpp

@@ -108,7 +108,7 @@ template<> char *toString(const ::msdb::DBUserData &data) {
 	return buffer;
 }
 
-}; // namespace QTest
+} // namespace QTest
 
 namespace std {
 template< std::size_t size > char *toString(const std::array< std::uint8_t, size > array) {
@@ -1481,7 +1481,7 @@ void ServerDatabaseTest::banTable_general() {
 	// Since we re-used the server ID, base address and prefix length, adding this should error
 	QVERIFY_EXCEPTION_THROWN(table.addBan(ban2), ::mdb::AccessException);
 
-	ban2.prefixLength += 1;
+	ban2.prefixLength++;
 	QVERIFY(!table.banExists(ban2));
 	table.addBan(ban2);
 	QVERIFY(table.banExists(ban2));