skin: fix the incorrect decision logic for VM color
authorGiWoong Kim <giwoong.kim@samsung.com>
Tue, 3 Nov 2015 06:31:34 +0000 (15:31 +0900)
committerSeokYeon Hwang <syeon.hwang@samsung.com>
Thu, 5 Nov 2015 03:22:35 +0000 (12:22 +0900)
Decision logic for VM color(which is used by Pair Tag on skin)
is depended on base port number. The port number is
increased by 10 from 26100.

Change-Id: I8744e837eb77773e9dedf2155abca346498ae211
Signed-off-by: GiWoong Kim <giwoong.kim@samsung.com>
tizen/src/ui/qt5_supplement.cpp
tizen/src/ui/uiinformation.cpp

index 3bc303a37693186bde600d952c1be76d22b46d3e..31c34ab4f38b81be4d66dc99d97adebd44b5cf44 100644 (file)
@@ -113,6 +113,7 @@ void qt5_gui_init(void)
     uiInfo->setResolution(QSize(
         get_display_resolution_width(), get_display_resolution_height()));
     uiInfo->setBasePort(get_emul_vm_base_port());
+    qDebug() << "VM base port:" <<  uiInfo->getBasePort();
 
     uiInfo->setVmDataPath(QDir(
         QString::fromLocal8Bit(get_vm_data_path())).canonicalPath());
index 603577a90181fa3a8491fbcac4e0acc52f0fbd58..1fa31d01e8ab6e4e817640872a223f83380621a4 100644 (file)
@@ -78,7 +78,12 @@ UIState *UiInformation::getUiState()
 
 QColor UiInformation::getVMColor()
 {
-    switch(basePort % 10) {
+    if (basePort <= 0) {
+        qWarning() << "invalid base port:" << basePort;
+        return QColor(255, 0, 0);
+    }
+
+    switch((basePort / 10) % 10) {
     case 0:
         break;
     case 1: