Fix a few memory leaks

src/core/player.cpp

@@ -96,6 +96,10 @@ Player::Player(PlaylistManager* playlists, LastFMService* lastfm,
 #endif
 }
 
+Player::~Player() {
+  delete engine_;
+}
+
 EngineBase* Player::createEngine(Engine::Type engine) {
 
   switch(engine) {

src/core/player.h

@@ -54,6 +54,7 @@ class Player : public QObject {
  public:
   Player(PlaylistManager* playlists, LastFMService* lastfm, Engine::Type engine,
          QObject* parent = 0);
+  ~Player();
 
   EngineBase* createEngine(Engine::Type engine);
   void Init();

src/engines/gstengine.cpp

@@ -609,20 +609,21 @@ GstEngine::PluginDetailsList
   PluginDetailsList ret;
 
   GstRegistry* registry = gst_registry_get_default();
-  GList* features =
+  GList* const features =
       gst_registry_get_feature_list(registry, GST_TYPE_ELEMENT_FACTORY);
 
-  while (features) {
-    GstElementFactory* factory = GST_ELEMENT_FACTORY(features->data);
+  GList* p = features;
+  while (p) {
+    GstElementFactory* factory = GST_ELEMENT_FACTORY(p->data);
     if (QString(factory->details.klass).contains(classname)) {
       PluginDetails details;
-      details.name = QString::fromUtf8(GST_PLUGIN_FEATURE_NAME(features->data));
+      details.name = QString::fromUtf8(GST_PLUGIN_FEATURE_NAME(p->data));
       details.long_name = QString::fromUtf8(factory->details.longname);
       details.description = QString::fromUtf8(factory->details.description);
       details.author = QString::fromUtf8(factory->details.author);
       ret << details;
     }
-    features = g_list_next ( features );
+    p = g_list_next(p);
   }
 
   gst_plugin_feature_list_free(features);

src/library/library.cpp

@@ -23,7 +23,8 @@ const char* Library::kDirsTable = "directories";
 const char* Library::kSubdirsTable = "subdirectories";
 
 Library::Library(Database *db, QObject *parent)
-  : backend_(new LibraryBackend(db, kSongsTable, kDirsTable, kSubdirsTable, this)),
+  : QObject(parent),
+    backend_(new LibraryBackend(db, kSongsTable, kDirsTable, kSubdirsTable, this)),
     model_(new LibraryModel(backend_, parent)),
     watcher_factory_(new BackgroundThreadFactoryImplementation<LibraryWatcher, LibraryWatcher>),
     watcher_(NULL)

src/ui/systemtrayicon.cpp

@@ -36,8 +36,6 @@ SystemTrayIcon::SystemTrayIcon(QObject* parent)
 #endif
 }
 
-SystemTrayIcon::~SystemTrayIcon() {}
-
 bool SystemTrayIcon::event(QEvent* event) {
   if (event->type() == QEvent::Wheel) {
     emit WheelEvent(static_cast<QWheelEvent*>(event)->delta());

src/ui/systemtrayicon.h

@@ -24,7 +24,6 @@ class SystemTrayIcon : public QSystemTrayIcon {
 
  public:
   SystemTrayIcon(QObject* parent = 0);
-  virtual ~SystemTrayIcon();
 
   virtual bool event(QEvent* event);