Cleanup: Use default virtual destructors wherever possible

doomsday/apps/client/include/misc/ihplane.h

@@ -44,7 +44,7 @@ class IHPlane
     public:
         IIntercept(ddouble distance) : _distance(distance) {}
 
-        virtual ~IIntercept() {}
+        virtual ~IIntercept() = default;
 
         /**
          * Determines the distance between "this" and the @a other intercept
@@ -75,7 +75,7 @@ class IHPlane
     };
 
 public:
-    virtual ~IHPlane() {}
+    virtual ~IHPlane() = default;
 
     /**
      * Reconfigure the half-plane according to the given Partition line.

doomsday/apps/client/include/render/ilightsource.h

@@ -46,7 +46,7 @@ class ILightSource
     typedef de::Vec3f Colorf;
 
 public:
-    virtual ~ILightSource() {}
+    virtual ~ILightSource() = default;
 
     virtual LightId lightSourceId() const = 0;
 
@@ -78,7 +78,7 @@ class IPointLightSource : public ILightSource
     typedef de::Vec3d Origin;
 
 public:
-    virtual ~IPointLightSource() {}
+    virtual ~IPointLightSource() = default;
 
     /**
      * Returns the position of the light source, in map units.

doomsday/apps/client/include/render/trianglestripbuilder.h

@@ -35,7 +35,7 @@ class IEdge
     class IEvent
     {
     public:
-        virtual ~IEvent() {}
+        virtual ~IEvent() = default;
 
         virtual bool operator < (IEvent const &other) const {
             return distance() < other.distance();
@@ -45,7 +45,7 @@ class IEdge
     };
 
 public:
-    virtual ~IEdge() {}
+    virtual ~IEdge() = default;
 
     virtual bool isValid() const = 0;
 

doomsday/apps/client/include/ui/clientwindow.h

@@ -32,15 +32,15 @@
 
 #undef main
 
-#define DE_GAMEVIEW_X         ClientWindow::main().game().rule().left().valuei()
-#define DE_GAMEVIEW_Y         ClientWindow::main().game().rule().top().valuei()
-#define DE_GAMEVIEW_WIDTH     ClientWindow::main().game().rule().width().valuei()
-#define DE_GAMEVIEW_HEIGHT    ClientWindow::main().game().rule().height().valuei()
+#define DE_GAMEVIEW_X           ClientWindow::main().game().rule().left().valuei()
+#define DE_GAMEVIEW_Y           ClientWindow::main().game().rule().top().valuei()
+#define DE_GAMEVIEW_WIDTH       ClientWindow::main().game().rule().width().valuei()
+#define DE_GAMEVIEW_HEIGHT      ClientWindow::main().game().rule().height().valuei()
 
 /**
  * A helpful macro that changes the origin of the window space coordinate system.
  */
-#define FLIP(y)             (ClientWindow::main().height() - ((y)+1))
+#define FLIP(y)                 (ClientWindow::main().height() - ((y) + 1))
 
 class ConsoleWidget;
 class TaskBarWidget;

doomsday/apps/client/include/ui/editors/variablegroupeditor.h

@@ -48,7 +48,7 @@ class VariableGroupEditor : public de::FoldPanelWidget
         virtual de::ScrollAreaWidget &containerWidget() = 0;
         virtual void resetToDefaults(de::String const &name) = 0;
 
-        virtual ~IOwner() {}
+        virtual ~IOwner() = default;
     };
 
 public:

doomsday/apps/client/include/ui/widgets/icvarwidget.h

@@ -25,7 +25,7 @@
 class ICVarWidget
 {
 public:
-    virtual ~ICVarWidget() {}
+    virtual ~ICVarWidget() = default;
 
     virtual char const *cvarPath() const = 0;
     virtual void updateFromCVar() = 0;

doomsday/apps/plugins/common/src/game/g_eventsequence.cpp

@@ -31,7 +31,7 @@
 class ISequenceCompleteHandler
 {
 public:
-    virtual ~ISequenceCompleteHandler() {}
+    virtual ~ISequenceCompleteHandler() = default;
     virtual void invoke(int player, EventSequenceArg* args, int numArgs) = 0;
 };
 

doomsday/apps/plugins/importidtech1/src/mapimporter.cpp

@@ -44,7 +44,7 @@ class Id1MapElement
 public:
     Id1MapElement(MapImporter &map) : _map(&map) {}
     Id1MapElement(Id1MapElement const &other) : _map(other._map) {}
-    virtual ~Id1MapElement() {}
+    virtual ~Id1MapElement() = default;
 
     MapImporter &map() const {
         DE_ASSERT(_map != 0);

doomsday/libs/appfw/include/de/framework/childwidgetorganizer.h

@@ -46,7 +46,7 @@ class LIBAPPFW_PUBLIC ChildWidgetOrganizer
     class IWidgetFactory
     {
     public:
-        virtual ~IWidgetFactory() {}
+        virtual ~IWidgetFactory() = default;
 
         /**
          * Called when the organizer needs a widget for a context item. This allows

doomsday/libs/appfw/include/de/framework/guiwidget.h

@@ -155,7 +155,7 @@ class LIBAPPFW_PUBLIC GuiWidget : public Widget
     class IEventHandler
     {
     public:
-        virtual ~IEventHandler() {}
+        virtual ~IEventHandler() = default;
 
         /**
          * Handle an event.

doomsday/libs/appfw/include/de/framework/ipersistent.h

@@ -36,7 +36,7 @@ class PersistentState;
 class LIBAPPFW_PUBLIC IPersistent
 {
 public:
-    virtual ~IPersistent() {}
+    virtual ~IPersistent() = default;
 
     virtual void operator >> (PersistentState &toState) const = 0;
     virtual void operator << (PersistentState const &fromState) = 0;

doomsday/libs/core/include/de/concurrency/task.h

@@ -29,7 +29,7 @@ class DE_PUBLIC IRunnable
 {
 public:
     virtual void run() = 0;
-    virtual ~IRunnable() {}
+    virtual ~IRunnable() = default;
 };
 
 /**

doomsday/libs/core/include/de/concurrency/taskpool.h

@@ -56,7 +56,7 @@ class DE_PUBLIC TaskPool
     {
     public:
         virtual void taskFinishedRunning(Task &) = 0;
-        virtual ~IPool() {}
+        virtual ~IPool() = default;
     };
 
     typedef std::function<void ()> TaskFunction;

doomsday/libs/core/include/de/core/logbuffer.h

@@ -51,7 +51,7 @@ class DE_PUBLIC LogBuffer : public Lockable
     class IFilter
     {
     public:
-        virtual ~IFilter() {}
+        virtual ~IFilter() = default;
 
         /**
          * Determines if a log entry should be allowed into the log buffer

doomsday/libs/core/include/de/core/logsink.h

@@ -53,7 +53,7 @@ class DE_PUBLIC LogSink
         typedef StringList Lines;
 
         virtual Lines logEntryToTextLines(const LogEntry &entry) = 0;
-        virtual ~IFormatter() {}
+        virtual ~IFormatter() = default;
     };
 
 public:

doomsday/libs/core/include/de/data/bank.h

@@ -144,7 +144,7 @@ class DE_PUBLIC Bank
     class ISource
     {
     public:
-        virtual ~ISource() {}
+        virtual ~ISource() = default;
 
         /**
          * Returns the timestamp of the source data, which determines when the
@@ -166,7 +166,7 @@ class DE_PUBLIC Bank
     class IData
     {
     public:
-        virtual ~IData() {}
+        virtual ~IData() = default;
 
         /// Serialization should be done?
         virtual bool shouldBeSerialized() const { return true; }

doomsday/libs/core/include/de/data/iblock.h

@@ -14,7 +14,7 @@
  * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser
  * General Public License for more details. You should have received a copy of
  * the GNU Lesser General Public License along with this program; if not, see:
- * http://www.gnu.org/licenses</small> 
+ * http://www.gnu.org/licenses</small>
  */
 
 #ifndef LIBCORE_IBLOCK_H
@@ -33,7 +33,7 @@ namespace de {
 class IBlock
 {
 public:
-    virtual ~IBlock() {}
+    virtual ~IBlock() = default;
 
     /// Empties the data of the block.
     virtual void clear() = 0;

doomsday/libs/core/include/de/data/ibytearray.h

@@ -48,7 +48,7 @@ class DE_PUBLIC IByteArray
     typedef dbyte Byte;
 
 public:
-    virtual ~IByteArray() {}
+    virtual ~IByteArray() = default;
 
     /// Returns the length of the array.
     virtual Size size() const = 0;

doomsday/libs/core/include/de/data/iistream.h

@@ -43,7 +43,7 @@ class IIStream
     DE_ERROR(InputError);
 
 public:
-    virtual ~IIStream() {}
+    virtual ~IIStream() = default;
 
     /**
      * Reads all the available bytes into the array @a bytes. If there is

doomsday/libs/core/include/de/data/info.h

@@ -265,7 +265,7 @@ class DE_PUBLIC Info
     class DE_PUBLIC IIncludeFinder
     {
     public:
-        virtual ~IIncludeFinder() {}
+        virtual ~IIncludeFinder() = default;
 
         /**
          * Finds an Info document.

doomsday/libs/core/include/de/data/iostream.h

@@ -36,7 +36,7 @@ class IOStream
     DE_ERROR(OutputError);
 
 public:
-    virtual ~IOStream() {}
+    virtual ~IOStream() = default;
 
     /**
      * Writes an array of bytes to the stream.

doomsday/libs/core/include/de/data/ireadable.h

@@ -14,9 +14,9 @@
  * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser
  * General Public License for more details. You should have received a copy of
  * the GNU Lesser General Public License along with this program; if not, see:
- * http://www.gnu.org/licenses</small> 
+ * http://www.gnu.org/licenses</small>
  */
- 
+
 #ifndef LIBCORE_IREADABLE_H
 #define LIBCORE_IREADABLE_H
 
@@ -37,7 +37,7 @@ class Reader;
 class DE_PUBLIC IReadable
 {
 public:
-    virtual ~IReadable() {}
+    virtual ~IReadable() = default;
 
     /**
      * Restore the object from the provided Reader.

doomsday/libs/core/include/de/data/iserializable.h

@@ -14,9 +14,9 @@
  * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser
  * General Public License for more details. You should have received a copy of
  * the GNU Lesser General Public License along with this program; if not, see:
- * http://www.gnu.org/licenses</small> 
+ * http://www.gnu.org/licenses</small>
  */
- 
+
 #ifndef LIBCORE_ISERIALIZABLE_H
 #define LIBCORE_ISERIALIZABLE_H
 
@@ -45,7 +45,7 @@ class DE_PUBLIC ISerializable : public IWritable, public IReadable
     DE_ERROR(DeserializationError);
 
 public:
-    virtual ~ISerializable() {}
+    virtual ~ISerializable() = default;
 };
 
 } // namespace de

doomsday/libs/core/include/de/data/iwritable.h

@@ -14,9 +14,9 @@
  * of MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU Lesser
  * General Public License for more details. You should have received a copy of
  * the GNU Lesser General Public License along with this program; if not, see:
- * http://www.gnu.org/licenses</small> 
+ * http://www.gnu.org/licenses</small>
  */
- 
+
 #ifndef LIBCORE_IWRITABLE_H
 #define LIBCORE_IWRITABLE_H
 
@@ -38,7 +38,7 @@ class Writer;
 class DE_PUBLIC IWritable
 {
 public:
-    virtual ~IWritable() {}
+    virtual ~IWritable() = default;
 
     /**
      * Serialize the object using the provided Writer.

doomsday/libs/core/include/de/data/pointerset.h

@@ -48,7 +48,7 @@ class DE_PUBLIC PointerSet
     class DE_PUBLIC IIterationObserver
     {
     public:
-        virtual ~IIterationObserver() {}
+        virtual ~IIterationObserver() = default;
         virtual void pointerSetIteratorsWereInvalidated(Pointer const *oldBase, Pointer const *newBase) = 0;
     };
 

doomsday/libs/core/include/de/data/string.h

@@ -163,7 +163,7 @@ class DE_PUBLIC String : public IByteArray
         DE_ERROR(TypeError);
 
     public:
-        virtual ~IPatternArg() {}
+        virtual ~IPatternArg() = default;
 
         /// Returns the value of the argument as a text string.
         virtual String asText() const = 0;

doomsday/libs/core/include/de/filesys/fileindex.h

@@ -46,7 +46,7 @@ class DE_PUBLIC FileIndex
     class DE_PUBLIC IPredicate
     {
     public:
-        virtual ~IPredicate() {}
+        virtual ~IPredicate() = default;
 
         /**
          * Determines if a file should be included in the index.

doomsday/libs/core/include/de/filesys/idownloadable.h

@@ -30,7 +30,7 @@ namespace de {
 class DE_PUBLIC IDownloadable
 {
 public:
-    virtual ~IDownloadable() {}
+    virtual ~IDownloadable() = default;
 
     virtual Asset &asset() = 0;
 

doomsday/libs/core/include/de/filesys/iinterpreter.h

@@ -36,7 +36,7 @@ namespace filesys {
 class IInterpreter
 {
 public:
-    virtual ~IInterpreter() {}
+    virtual ~IInterpreter() = default;
 
     /**
      * Attempts to interpret a file.

doomsday/libs/core/include/de/scriptsys/iobject.h

@@ -39,7 +39,7 @@ class Variable;
 class DE_PUBLIC IObject
 {
 public:
-    virtual ~IObject() {}
+    virtual ~IObject() = default;
 
     /**
      * Returns the Record that contains the instance namespace of the object.

doomsday/libs/core/include/de/scriptsys/iparser.h

@@ -39,7 +39,7 @@ class Script;
 class IParser
 {
 public:
-    virtual ~IParser() {}
+    virtual ~IParser() = default;
 
     /**
      * Reads an input script in text format, and according to the

doomsday/libs/core/include/de/widgets/isizerule.h

@@ -30,7 +30,7 @@ namespace de {
 class DE_PUBLIC ISizeRule
 {
 public:
-    virtual ~ISizeRule() {}
+    virtual ~ISizeRule() = default;
 
     virtual Rule const &width() const = 0;
     virtual Rule const &height() const = 0;

doomsday/libs/doomsday/include/doomsday/busymode.h

@@ -59,7 +59,7 @@ class LIBDOOMSDAY_PUBLIC BusyMode
     class LIBDOOMSDAY_PUBLIC ITaskRunner
     {
     public:
-        virtual ~ITaskRunner() {}
+        virtual ~ITaskRunner() = default;
 
         struct LIBDOOMSDAY_PUBLIC Result {
             bool wasRun;

doomsday/libs/doomsday/include/doomsday/gamestatefolder.h

@@ -112,7 +112,7 @@ class LIBDOOMSDAY_PUBLIC GameStateFolder : public de::ArchiveFolder
     class IMapStateReaderFactory
     {
     public:
-        virtual ~IMapStateReaderFactory() {}
+        virtual ~IMapStateReaderFactory() = default;
 
         /**
          * Called while loading a saved session to acquire a MapStateReader for the