Use reference type for iteration and initialize memeber with proper value

Change-Id: Idde632226e16e67d5aaed5dfcab0371339510d77
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>

diff --git a/src/mmimgr/iu/VoiceTouchEngine.cpp b/src/mmimgr/iu/VoiceTouchEngine.cpp
index a14eb6b..6ad4630 100644 (file)
--- a/src/mmimgr/iu/VoiceTouchEngine.cpp
+++ b/src/mmimgr/iu/VoiceTouchEngine.cpp
@@ -44,6 +44,12 @@ static const int __DEFAULT_GRID_NUM_Y2 = 3;
 VoiceTouchEngine::VoiceTouchEngine()
 {
        _I("[VoiceTouchEngine] Constructor");
+       __touchModality.timestamp = -1;
+       __touchModality.mode = MMI_VOICE_TOUCH_MODE_TOOLTIP;
+       __touchModality.object_id = nullptr;
+       __touchModality.coord_x = 0;
+       __touchModality.coord_y = 0;
+
        __currentMode = VOICE_TOUCH_MODE_TOOLTIP_TEXT;
 
        __itemList.clear();
@@ -332,7 +338,7 @@ void VoiceTouchEngine::makeClickableItemInfo(int timestamp, JsonProvider &provid
        provider.setOutputEvent(MMI_KEY_UI_CLICKABLE_OBJECT);
        provider.setUiClickableObject(resultType, __appId.c_str(), tooltipType, __itemList.size(), timestamp);
 
-       for (auto item : *itemList) {
+       for (auto &item : *itemList) {
                provider.addInfoClickableObject(item.index, item.coordX, item.coordY, item.width, item.height, item.label.c_str());
        }
 }
@@ -448,7 +454,7 @@ void VoiceTouchEngine::iterateObjectCallback(gpointer data, gpointer userData)
 
        ClickableItem item;
        item.coordX = info->coord_x;
-       item.coordX = info->coord_y;
+       item.coordY = info->coord_y;
        item.width = info->width;
        item.height = info->height;
        item.objectId = string(info->object_id);