Build Error Fix (-Werror=array-bounds) 63/108663/2
authorMyungJoo Ham <myungjoo.ham@samsung.com>
Thu, 5 Jan 2017 10:04:15 +0000 (19:04 +0900)
committerMyungJoo Ham <myungjoo.ham@samsung.com>
Thu, 5 Jan 2017 10:09:43 +0000 (02:09 -0800)
This fixes build error in recent build system:

[  227s] /home/abuild/rpmbuild/BUILD/org.tizen.infosquare-0.2/src/action_menu_view.c: In function '_show':
[  227s] /home/abuild/rpmbuild/BUILD/org.tizen.infosquare-0.2/src/action_menu_view.c:340:2: error: array subscript is below array bounds [-Werror=array-bounds]
[  227s]   elm_object_focus_next_object_set(priv->menu_btn[0],
[  227s]   ^
[  227s] /home/abuild/rpmbuild/BUILD/org.tizen.infosquare-0.2/src/action_menu_view.c:342:2: error: array subscript is below array bounds [-Werror=array-bounds]
[  227s]   elm_object_focus_next_object_set(priv->menu_btn[i - 1],

Change-Id: I5eddf0a634347ecea1d04ae2cdd56c5e804c58ea
Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>
src/action_menu_view.c

index 2ce8c06..41e7dab 100644 (file)
@@ -337,10 +337,12 @@ static void _update_menu_area(struct _priv *priv)
                i++;
        }
 
-       elm_object_focus_next_object_set(priv->menu_btn[0],
-                       priv->menu_btn[i - 1], ELM_FOCUS_LEFT);
-       elm_object_focus_next_object_set(priv->menu_btn[i - 1],
-                       priv->menu_btn[0], ELM_FOCUS_RIGHT);
+       if (i > 0) {
+               elm_object_focus_next_object_set(priv->menu_btn[0],
+                               priv->menu_btn[i - 1], ELM_FOCUS_LEFT);
+               elm_object_focus_next_object_set(priv->menu_btn[i - 1],
+                               priv->menu_btn[0], ELM_FOCUS_RIGHT);
+       }
 
        while (col + 1 != ITEMS_IN_ROW) {
                btn = util_add_button(priv->base, NULL, NULL,