From: GiWoong Kim <giwoong.kim@samsung.com>
Date: Fri, 23 Oct 2015 12:22:05 +0000 (+0900)
Subject: skin: remove memory leak
X-Git-Tag: Tizen_Studio_1.3_Release_p2.3.2~203
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=dc90db55a2fc259e5841db15f4782815c57318ef;p=sdk%2Femulator%2Fqemu.git

skin: remove memory leak

1. remove resource leak in SkinPainter object
2. delete unnecessary null checking

Change-Id: I141116cfd68dcd242bd2581c6deff01c82bf266c
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>
---

diff --git a/tizen/src/ui/mainwindow.cpp b/tizen/src/ui/mainwindow.cpp
index 729c026c08..e449175513 100644
--- a/tizen/src/ui/mainwindow.cpp
+++ b/tizen/src/ui/mainwindow.cpp
@@ -144,10 +144,8 @@ DisplayBase *MainWindow::createDisplay(DisplayType *displayForm)
         /* display swapper */
         swapperThread = new QThread(this);
 
-        if (context) {
-            context->doneCurrent();
-            context->moveToThread(swapperThread);
-        }
+        context->doneCurrent();
+        context->moveToThread(swapperThread);
 
         swapper = new DisplaySwapper(context);
         swapper->moveToThread(swapperThread);
diff --git a/tizen/src/ui/skinpainter.cpp b/tizen/src/ui/skinpainter.cpp
index 65baf3c04d..57ce10a7b8 100644
--- a/tizen/src/ui/skinpainter.cpp
+++ b/tizen/src/ui/skinpainter.cpp
@@ -105,3 +105,12 @@ QRect SkinPainter::getCenteralRect()
 {
     return centeralRect;
 }
+
+SkinPainter::~SkinPainter()
+{
+    qDebug("destroy skin painter");
+
+    if (skin != NULL) {
+        delete skin;
+    }
+}
diff --git a/tizen/src/ui/skinpainter.h b/tizen/src/ui/skinpainter.h
index e2541e33bb..653e1204c9 100644
--- a/tizen/src/ui/skinpainter.h
+++ b/tizen/src/ui/skinpainter.h
@@ -39,6 +39,7 @@ class SkinPainter
 public:
     SkinPainter(QString patchPath, QSize centerPatch, int degree,
         QPoint tagLeftTop, QColor tagColor);
+    ~SkinPainter();
 
     QImage getSkinImage();
     QRect getCenteralRect();