Revert "mobile-lua: add additional LABELLED_BY relation"

This reverts commit 9d9260eadb124a7418855e7d06afccae66cc0667.

Change-Id: Ie9e30cacaae0c2ee793073c4de1b936701c9e4ae

diff --git a/res/scripts/mobile.lua b/res/scripts/mobile.lua
index 39d5396..c53c2e0 100644 (file)
--- a/res/scripts/mobile.lua
+++ b/res/scripts/mobile.lua
@@ -114,44 +114,16 @@ function trait(obj)
        end
 end
 
-function stringArrayAppend(strings, string)
-       if string ~= "" then
-               table.insert(strings, string)
-       end
-end
-
-function generateStringArray(objects, createFunc)
-       local ret = {}
-       for k, target in ipairs(objects) do
-               stringArrayAppend(ret, createFunc(target))
-       end
-       return ret
-end
-
 function describeObject(obj)
-       local labels = generateStringArray(obj:inRelation(RELATION_LABELLED_BY), function(x) return x:name() end)
-       local descriptions = generateStringArray(obj:inRelation(RELATION_DESCRIBED_BY), function(x) return x:description() end)
-       local traits = generateStringArray(obj:inRelation(RELATION_DESCRIBED_BY), trait)
-
-       -- use original object name if related objects do not provide other
-       if table.getn(labels) == 0 then
-               stringArrayAppend(labels, obj:name())
-       end
-
-       -- use original object description if related objects do not provide other
-       if table.getn(descriptions) == 0 then
-               stringArrayAppend(descriptions, obj:description())
+       local related_trait = {}
+       for k, target in ipairs(obj:inRelation(RELATION_DESCRIBED_BY)) do
+               table.insert(related_trait, trait(target))
        end
-
-       -- use original object trait if related objects do not provide other
-       if table.getn(traits) == 0 then
-               stringArrayAppend(traits, trait(obj))
+       local ret = {obj:name(), trait(obj), obj:description(), table.concat(related_trait, ", ")}
+       for i=#ret,1,-1 do
+               if ret[i] == nil or ret[i] == "" then
+                       table.remove(ret, i)
+               end
        end
-
-       local ret = {}
-       stringArrayAppend(ret, table.concat(labels, ", "))
-       stringArrayAppend(ret, table.concat(traits, ", "))
-       stringArrayAppend(ret, table.concat(descriptions, ", "))
-
        return table.concat(ret, ", ")
 end

diff --git a/src/lua_engine.c b/src/lua_engine.c
index 76cd94f..b2d8207 100644 (file)
--- a/src/lua_engine.c
+++ b/src/lua_engine.c
@@ -166,7 +166,7 @@ static int _accessible_relations(lua_State *L) {
        AtspiAccessible *obj = _pop_class_obj(L, 1, 1, ACCESSIBLE_CLASS_NAME);
        AtspiRelationType type = lua_tonumber(L, -1);
        GError *err = NULL;
-       int i, j, idx;
+       int i, j;
 
        lua_newtable(L);
        if (!obj) return 1;
@@ -174,7 +174,7 @@ static int _accessible_relations(lua_State *L) {
        GERROR_CHECK(err);
        if (!rels) return 1;
 
-       for (i = 0, idx = 1; i < rels->len; i++)
+       for (i = 0; i < rels->len; i++)
        {
                AtspiRelation *rel = g_array_index(rels, AtspiRelation*, i);
                if (atspi_relation_get_relation_type(rel) == type)
@@ -183,7 +183,7 @@ static int _accessible_relations(lua_State *L) {
                        {
                                AtspiAccessible *target = atspi_relation_get_target(rel, j);
                                if (!target) continue;
-                               lua_pushinteger(L, idx++);
+                               lua_pushinteger(L, j);
                                _push_class_obj(L, target, ACCESSIBLE_CLASS_NAME);
                                lua_settable(L, -3);
                        }