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 += static_cast< std::uint8_t >(1);
  QVERIFY(!table.banExists(ban2));
  table.addBan(ban2);
  QVERIFY(table.banExists(ban2));