summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDaniel Kolesa <d.kolesa@osg.samsung.com>2015-07-07 10:56:53 +0100
committerDaniel Kolesa <d.kolesa@osg.samsung.com>2015-07-07 10:57:24 +0100
commitcd7a78f2a5ea44f458b51e88bcdb9c7d6b0d67b8 (patch)
tree076c07abe5b487905ca5d87f9f5f7b27855a23a5
parent786ab74d6daa18bc1c04e2c97f5ca5212d707b78 (diff)
downloadefl-cd7a78f2a5ea44f458b51e88bcdb9c7d6b0d67b8.tar.gz
eolian: add API: eolian_declaration_get_by_name
Sorry for last minute new API, but this one is necessary to implement proper verification of references in docs, which I would like to have in 1.15. It allows you to retrieve any kind of declaration by full namespace, so that you can check for an existence of an Eolian decl, rather than checking every type individually. @feature
-rw-r--r--src/bindings/luajit/eolian.lua9
-rw-r--r--src/lib/eolian/Eolian.h10
-rw-r--r--src/lib/eolian/eolian_database.c10
-rw-r--r--src/tests/eolian/eolian_parsing.c16
4 files changed, 40 insertions, 5 deletions
diff --git a/src/bindings/luajit/eolian.lua b/src/bindings/luajit/eolian.lua
index fa78cc2dcf..6cf150002d 100644
--- a/src/bindings/luajit/eolian.lua
+++ b/src/bindings/luajit/eolian.lua
@@ -313,6 +313,7 @@ ffi.cdef [[
const char *eolian_variable_full_name_get(const Eolian_Variable *var);
Eina_Iterator *eolian_variable_namespaces_get(const Eolian_Variable *var);
Eina_Bool eolian_variable_is_extern(const Eolian_Variable *var);
+ const Eolian_Declaration *eolian_declaration_get_by_name(const char *name);
Eina_Iterator *eolian_declarations_get_by_file(const char *fname);
Eolian_Declaration_Type eolian_declaration_type_get(const Eolian_Declaration *decl);
const char *eolian_declaration_name_get(const Eolian_Declaration *decl);
@@ -1266,6 +1267,14 @@ M.Variable = ffi.metatype("Eolian_Variable", {
}
})
+M.declaration_get_by_name = function(name)
+ local v = eolian.eolian_declaration_get_by_name(name)
+ if v == nil then
+ return nil
+ end
+ return v
+end
+
M.declarations_get_by_file = function(fname)
return Ptr_ITerator("const Eolian_Declaration*",
eolian.eolian_declarations_get_by_file(fname))
diff --git a/src/lib/eolian/Eolian.h b/src/lib/eolian/Eolian.h
index 5ebd90d1ca..e950e21545 100644
--- a/src/lib/eolian/Eolian.h
+++ b/src/lib/eolian/Eolian.h
@@ -1950,6 +1950,16 @@ EAPI Eina_Iterator *eolian_variable_namespaces_get(const Eolian_Variable *var);
EAPI Eina_Bool eolian_variable_is_extern(const Eolian_Variable *var);
/*
+ * @brief Get a declaration by name.
+ *
+ * @param[in] name the declaration name.
+ * @return the declaration.
+ *
+ * @ingroup Eolian
+ */
+EAPI const Eolian_Declaration *eolian_declaration_get_by_name(const char *name);
+
+/*
* @brief Get a list of declarations in a file.
*
* This function gets an iterator to a list of declarations in a particular
diff --git a/src/lib/eolian/eolian_database.c b/src/lib/eolian/eolian_database.c
index 4ebc9785a7..a897f9d5a6 100644
--- a/src/lib/eolian/eolian_database.c
+++ b/src/lib/eolian/eolian_database.c
@@ -109,6 +109,16 @@ database_decl_add(Eina_Stringshare *name, Eolian_Declaration_Type type,
((Eina_List*)eina_hash_find(_declsf, file), decl));
}
+EAPI const Eolian_Declaration *
+eolian_declaration_get_by_name(const char *name)
+{
+ if (!_decls) return NULL;
+ Eina_Stringshare *shr = eina_stringshare_add(name);
+ const Eolian_Declaration *decl = eina_hash_find(_decls, shr);
+ eina_stringshare_del(shr);
+ return decl;
+}
+
EAPI Eina_Iterator *
eolian_declarations_get_by_file(const char *fname)
{
diff --git a/src/tests/eolian/eolian_parsing.c b/src/tests/eolian/eolian_parsing.c
index 76438cd28b..3be48acd87 100644
--- a/src/tests/eolian/eolian_parsing.c
+++ b/src/tests/eolian/eolian_parsing.c
@@ -1064,39 +1064,45 @@ START_TEST(eolian_decl)
fail_if(!(itr = eolian_declarations_get_by_file("decl.eo")));
fail_if(!eina_iterator_next(itr, (void**)&decl));
- fail_if(eolian_declaration_type_get(decl) == EOLIAN_DECL_UNKNOWN);
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_STRUCT);
fail_if(strcmp(eolian_declaration_name_get(decl), "A"));
fail_if(!(type = eolian_declaration_data_type_get(decl)));
fail_if(eolian_type_type_get(type) != EOLIAN_TYPE_STRUCT);
fail_if(strcmp(eolian_type_name_get(type), "A"));
fail_if(!eina_iterator_next(itr, (void**)&decl));
- fail_if(eolian_declaration_type_get(decl) == EOLIAN_DECL_UNKNOWN);
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_ENUM);
fail_if(strcmp(eolian_declaration_name_get(decl), "B"));
fail_if(!(type = eolian_declaration_data_type_get(decl)));
fail_if(eolian_type_type_get(type) != EOLIAN_TYPE_ENUM);
fail_if(strcmp(eolian_type_name_get(type), "B"));
fail_if(!eina_iterator_next(itr, (void**)&decl));
- fail_if(eolian_declaration_type_get(decl) == EOLIAN_DECL_UNKNOWN);
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_ALIAS);
fail_if(strcmp(eolian_declaration_name_get(decl), "C"));
fail_if(!(type = eolian_declaration_data_type_get(decl)));
fail_if(eolian_type_type_get(type) != EOLIAN_TYPE_ALIAS);
fail_if(strcmp(eolian_type_name_get(type), "C"));
fail_if(!eina_iterator_next(itr, (void**)&decl));
- fail_if(eolian_declaration_type_get(decl) == EOLIAN_DECL_UNKNOWN);
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_VAR);
fail_if(strcmp(eolian_declaration_name_get(decl), "pants"));
fail_if(!(var = eolian_declaration_variable_get(decl)));
fail_if(strcmp(eolian_variable_name_get(var), "pants"));
fail_if(!eina_iterator_next(itr, (void**)&decl));
- fail_if(eolian_declaration_type_get(decl) == EOLIAN_DECL_UNKNOWN);
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_CLASS);
fail_if(strcmp(eolian_declaration_name_get(decl), "Decl"));
fail_if(eolian_declaration_class_get(decl) != class);
fail_if(eina_iterator_next(itr, (void**)&decl));
+ fail_if(!(decl = eolian_declaration_get_by_name("pants")));
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_VAR);
+
+ fail_if(!(decl = eolian_declaration_get_by_name("A")));
+ fail_if(eolian_declaration_type_get(decl) != EOLIAN_DECL_STRUCT);
+
eolian_shutdown();
}
END_TEST