diff --git a/src/blackgui/components/maininfoareacomponent.cpp b/src/blackgui/components/maininfoareacomponent.cpp
index 1d1f7cf80..de7992fff 100644
--- a/src/blackgui/components/maininfoareacomponent.cpp
+++ b/src/blackgui/components/maininfoareacomponent.cpp
@@ -124,7 +124,6 @@ namespace BlackGui
             case InfoAreaTextMessages:
             case InfoAreaWeather:
                 return QSize(600, 400);
-                break;
             case InfoAreaFlightPlan:
                 return QSize(625, 500);
             default:
diff --git a/src/blackmisc/lockfree.h b/src/blackmisc/lockfree.h
index 1132d4424..59e5a082e 100644
--- a/src/blackmisc/lockfree.h
+++ b/src/blackmisc/lockfree.h
@@ -280,13 +280,13 @@ namespace BlackMisc
         LockFreeUniqueWriter<T> uniqueWrite()
         {
             return { Private::atomic_load(&m_ptr), &m_ptr };
-        };
+        }
 
         //! Return an object which can write a new value, even if there are other writes.
         LockFreeSharedWriter<T> sharedWrite()
         {
             return { Private::atomic_load(&m_ptr), &m_ptr };
-        };
+        }
 
         //! Pass the current value to the functor inspector, and return whatever inspector returns.
         template <typename F>
diff --git a/src/blackmisc/range.h b/src/blackmisc/range.h
index e7831cd32..7c0b3c806 100644
--- a/src/blackmisc/range.h
+++ b/src/blackmisc/range.h
@@ -239,7 +239,7 @@ namespace BlackMisc
         I m_begin;
         I m_end;
 
-        void check(...) {};
+        void check(...) {}
         template <class I2, class F> void check(Iterators::ConditionalIterator<I2, F> *begin, Iterators::ConditionalIterator<I2, F> *end)
         {
             begin->checkEnd(*end);