[EGIT] [core/efl] master 02/04: eina_strbuf: add test cases for eina strbuf prepend functions

2016-02-17 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=d8166c75079d16211de75ea5007353ee57d5ab33

commit d8166c75079d16211de75ea5007353ee57d5ab33
Author: Vivek Ellur 
Date:   Wed Feb 17 14:24:20 2016 -0800

eina_strbuf: add test cases for eina strbuf prepend functions

Summary:
Added test cases for eina_strbuf_prepend_printf and 
eina_strbuf_prepend_vprintf
functions

Signed-off-by: Vivek Ellur 

Reviewers: raster, jpeg, cedric

Subscribers: seoz, jpeg, raster, cedric

Differential Revision: https://phab.enlightenment.org/D3163
---
 src/tests/eina/eina_test_strbuf.c | 60 +++
 1 file changed, 60 insertions(+)

diff --git a/src/tests/eina/eina_test_strbuf.c 
b/src/tests/eina/eina_test_strbuf.c
index b8f27f0..3b2fe33 100644
--- a/src/tests/eina/eina_test_strbuf.c
+++ b/src/tests/eina/eina_test_strbuf.c
@@ -534,6 +534,7 @@ START_TEST(strbuf_substr_get)
const char *str;
 
eina_init();
+
buf = eina_strbuf_new();
fail_unless(buf != NULL);
 
@@ -567,6 +568,64 @@ START_TEST(strbuf_substr_get)
fail_if(substr);
 
eina_strbuf_free(buf);
+
+   eina_shutdown();
+}
+END_TEST
+
+static const char*
+test_prepend_vprintf(Eina_Strbuf *buf, char *fmt, va_list ap)
+{
+   const char *str;
+
+   eina_strbuf_prepend_vprintf(buf, fmt, ap);
+   str = eina_strbuf_string_get(buf);
+   return str;
+}
+
+static const char*
+test_prepend(Eina_Strbuf *buf, ...)
+{
+   const char *str;
+
+   va_list ap;
+
+   va_start(ap, buf);
+   str = test_prepend_vprintf(buf, "%s %s %s %s", ap);
+   return str;
+}
+
+START_TEST(strbuf_prepend_print)
+{
+   Eina_Bool ret;
+   Eina_Strbuf* buf;
+   const char *str;
+
+   eina_init();
+   buf = eina_strbuf_new();
+   fail_unless(buf != NULL);
+
+   ret = eina_strbuf_prepend_printf(buf, "%s", "string");
+   fail_if(ret != EINA_TRUE);
+   str = eina_strbuf_string_get(buf);
+   fail_if(str == NULL || strcmp(str, "string") != 0);
+
+   ret = eina_strbuf_prepend_printf(buf, "This is %d ", 1);
+   fail_if(ret != EINA_TRUE);
+   str = eina_strbuf_string_get(buf);
+   fail_if(str == NULL || strcmp(str, "This is 1 string") != 0);
+
+   ret = eina_strbuf_prepend_printf(buf, "%s   ", "hello");
+   fail_if(ret != EINA_TRUE);
+   str = eina_strbuf_string_get(buf);
+   fail_if(str == NULL || strcmp(str, "hello   This is 1 string") != 0);
+   eina_strbuf_reset(buf);
+
+   str = test_prepend(buf, "This", "is", "test", "string");
+   fail_if(str == NULL || strcmp(str, "This is test string") != 0);
+
+   eina_strbuf_free(buf);
+
eina_shutdown();
 }
 END_TEST
@@ -586,4 +645,5 @@ eina_test_strbuf(TCase *tc)
tcase_add_test(tc, strbuf_trim);
tcase_add_test(tc, strbuf_tolower);
tcase_add_test(tc, strbuf_substr_get);
+   tcase_add_test(tc, strbuf_prepend_print);
 }

-- 




[EGIT] [core/efl] master 01/01: edje_embryo: Fix null dereference issue

2015-12-23 Thread Vivek Ellur
raster pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=abe131d1d7c6fc8963a27620e315fe511a6b3576

commit abe131d1d7c6fc8963a27620e315fe511a6b3576
Author: Vivek Ellur 
Date:   Wed Dec 23 19:12:06 2015 +0900

edje_embryo: Fix null dereference issue

Summary:
Fix Coverity issue ID 1299026

Signed-off-by: Vivek Ellur 

Reviewers: cedric, raster

Reviewed By: raster

Subscribers: raster, cedric

Differential Revision: https://phab.enlightenment.org/D3218
---
 src/lib/edje/edje_embryo.c | 4 
 1 file changed, 4 insertions(+)

diff --git a/src/lib/edje/edje_embryo.c b/src/lib/edje/edje_embryo.c
index 2595603..c4cba4f 100644
--- a/src/lib/edje/edje_embryo.c
+++ b/src/lib/edje/edje_embryo.c
@@ -1293,6 +1293,7 @@ _edje_embryo_fn_set_tween_state_anim(Embryo_Program *ep, 
Embryo_Cell *params)
 if (HASNPARAMS(8))
   {
  GETSTR(tmp, params[8]);
+ if (!tmp) return 0;
  if (!strcmp(tmp, "CURRENT"))
anim_type |= EDJE_TWEEN_MODE_OPT_FROM_CURRENT;
   }
@@ -1303,6 +1304,7 @@ _edje_embryo_fn_set_tween_state_anim(Embryo_Program *ep, 
Embryo_Cell *params)
 if (HASNPARAMS(9))
   {
  GETSTR(tmp, params[9]);
+ if (!tmp) return 0;
  if (!strcmp(tmp, "CURRENT"))
anim_type |= EDJE_TWEEN_MODE_OPT_FROM_CURRENT;
   }
@@ -1319,6 +1321,7 @@ _edje_embryo_fn_set_tween_state_anim(Embryo_Program *ep, 
Embryo_Cell *params)
 if (HASNPARAMS(10))
   {
  GETSTR(tmp, params[10]);
+ if (!tmp) return 0;
  if (!strcmp(tmp, "CURRENT"))
anim_type |= EDJE_TWEEN_MODE_OPT_FROM_CURRENT;
   }
@@ -1335,6 +1338,7 @@ _edje_embryo_fn_set_tween_state_anim(Embryo_Program *ep, 
Embryo_Cell *params)
 if (HASNPARAMS(12))
   {
  GETSTR(tmp, params[10]);
+ if (!tmp) return 0;
  if (!strcmp(tmp, "CURRENT"))
anim_type |= EDJE_TWEEN_MODE_OPT_FROM_CURRENT;
   }

-- 




[EGIT] [core/efl] master 01/01: Eio: Fixes T2831, eio xattr tests failing

2015-12-03 Thread Vivek Ellur
stefan pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=b41e66c1ef9928b506ed1f6e0ef9e033e25edf02

commit b41e66c1ef9928b506ed1f6e0ef9e033e25edf02
Author: Vivek Ellur 
Date:   Thu Dec 3 09:03:59 2015 +0100

Eio: Fixes T2831, eio xattr tests failing

Summary:
@Fix
The eio_file_xattr test was failing in jenkins, so added a new callback
and moved the check condition to callback. The condition is checked
only when it is successfully executed

Signed-off-by: Vivek Ellur 

Reviewers: cedric, stefan_schmidt

Subscribers: jpeg, cedric

Maniphest Tasks: T2831

Differential Revision: https://phab.enlightenment.org/D3400
---
 src/tests/eio/eio_test_xattr.c | 13 ++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/src/tests/eio/eio_test_xattr.c b/src/tests/eio/eio_test_xattr.c
index 6600026..6f15fac 100644
--- a/src/tests/eio/eio_test_xattr.c
+++ b/src/tests/eio/eio_test_xattr.c
@@ -83,6 +83,15 @@ _done_once_cb(void *data EINA_UNUSED, Eio_File *handler 
EINA_UNUSED)
 }
 
 static void
+_done_file_cb(void *data EINA_UNUSED, Eio_File *handler EINA_UNUSED)
+{
+   int *num_of_attr = (int *)data;
+
+   fail_if( (*num_of_attr) != (sizeof (attribute) / sizeof (attribute[0])));
+   ecore_main_loop_quit();
+}
+
+static void
 _done_get_cb(void *data, Eio_File *handler EINA_UNUSED, const char *name, 
unsigned int len EINA_UNUSED)
 
 {
@@ -170,15 +179,13 @@ START_TEST(eio_test_xattr_set)
 
num_of_attr = 0;
fp = eio_file_xattr(test_file_path,
-   _filter_cb, _main_cb, _done_once_cb, _error_cb,
+   _filter_cb, _main_cb, _done_file_cb, _error_cb,
&num_of_attr);
fail_if(num_of_attr != 0);
fail_if(!fp);
 
ecore_main_loop_begin();
 
-   fail_if(num_of_attr != sizeof (attribute)  / sizeof (attribute[0]));
-
close(fd);
unlink(test_file_path);
eio_shutdown();

-- 




[EGIT] [core/efl] master 01/04: eet: Fix warning in eet file

2015-11-23 Thread Vivek Ellur
jpeg pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=0ddd0129ea06cee05d8733968ac3b00318951025

commit 0ddd0129ea06cee05d8733968ac3b00318951025
Author: Vivek Ellur 
Date:   Mon Nov 23 17:35:07 2015 +0900

eet: Fix warning in eet file

Summary:
variable 'total' is of unsigned long long but %lli was used to print..so 
fixed
warning

Signed-off-by: Vivek Ellur 

Reviewers: cedric, jpeg

Reviewed By: jpeg

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3325
---
 src/bin/eet/eet_main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bin/eet/eet_main.c b/src/bin/eet/eet_main.c
index 09ab935..a3652e8 100644
--- a/src/bin/eet/eet_main.c
+++ b/src/bin/eet/eet_main.c
@@ -86,7 +86,7 @@ do_eet_list(const char *file, Eina_Bool verbose)
if (verbose)
  {
 printf("*** ***\n");
-printf("Total payload size : %lli.\n", total);
+printf("Total payload size : %llu.\n", total);
  }
 
eet_close(ef);

-- 




[EGIT] [core/efl] master 04/10: eet: remove useless assignment in eet cipher and silence warning.

2015-11-22 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=f0c826c415ab419ecfbf7d7a563f48eae25b58e1

commit f0c826c415ab419ecfbf7d7a563f48eae25b58e1
Author: Vivek Ellur 
Date:   Fri Nov 20 03:01:54 2015 +0100

eet: remove useless assignment in eet cipher and silence warning.

Summary:

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3310

Signed-off-by: Cedric BAIL 
---
 src/lib/eet/eet_cipher.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/lib/eet/eet_cipher.c b/src/lib/eet/eet_cipher.c
index 0f367ec..aa57e5c 100644
--- a/src/lib/eet/eet_cipher.c
+++ b/src/lib/eet/eet_cipher.c
@@ -260,8 +260,8 @@ eet_identity_close(Eet_Key *key)
EVP_PKEY_free(key->private_key);
 # endif /* ifdef HAVE_GNUTLS */
free(key);
-#else
-   key = NULL;
+# else
+   (void)key;
 #endif /* ifdef HAVE_SIGNATURE */
 }
 

-- 




[EGIT] [core/efl] master 05/10: evil: fix uninitialize warning

2015-11-22 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=01f13181175bfe1d0aefde8877efccd25f628f42

commit 01f13181175bfe1d0aefde8877efccd25f628f42
Author: Vivek Ellur 
Date:   Fri Nov 20 05:52:56 2015 +0100

evil: fix uninitialize warning

Summary:
@Fix

Signed-off-by: Vivek Ellur 

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3324

Signed-off-by: Cedric BAIL 
---
 src/bin/evil/evil_test_pipe.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/bin/evil/evil_test_pipe.c b/src/bin/evil/evil_test_pipe.c
index 54b68b2..e5cfdec 100644
--- a/src/bin/evil/evil_test_pipe.c
+++ b/src/bin/evil/evil_test_pipe.c
@@ -45,7 +45,7 @@ test_pipe_test(void)
intsockets[2];
struct timeval t;
fd_set rfds;
-   intret;
+   intret = 0;
data  *d;
DWORD  thread_id;
HANDLE h;

-- 




[EGIT] [core/efl] master 05/10: eina: add test case for binbuf append function

2015-11-10 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=46777274fed3d20d823c70e3eaf1eba5a815530d

commit 46777274fed3d20d823c70e3eaf1eba5a815530d
Author: Vivek Ellur 
Date:   Tue Nov 10 14:44:14 2015 -0800

eina: add test case for binbuf append function

Summary:
Added test case for eina_binbuf_append_buffer function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3304

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_binbuf.c | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/tests/eina/eina_test_binbuf.c 
b/src/tests/eina/eina_test_binbuf.c
index 1b0de28..ce77803 100644
--- a/src/tests/eina/eina_test_binbuf.c
+++ b/src/tests/eina/eina_test_binbuf.c
@@ -26,7 +26,7 @@
 
 START_TEST(binbuf_simple)
 {
-   Eina_Binbuf *buf;
+   Eina_Binbuf *buf, *test_buf;
unsigned char *txt;
const unsigned char cbuf[] = "Null in the middle \0 and more text 
afterwards and \0 anotehr null just there and another one \0 here.";
size_t size = sizeof(cbuf) - 1; /* We don't care about the real NULL */
@@ -41,6 +41,12 @@ START_TEST(binbuf_simple)
fail_if(memcmp(eina_binbuf_string_get(buf), cbuf, size));
fail_if(size != eina_binbuf_length_get(buf));
 
+   test_buf = eina_binbuf_new();
+   fail_if(!test_buf);
+   fail_if(!eina_binbuf_append_buffer(test_buf, buf));
+   fail_if(memcmp(eina_binbuf_string_get(test_buf), cbuf, size));
+   fail_if(size != eina_binbuf_length_get(test_buf));
+
eina_binbuf_append_length(buf, cbuf, size);
fail_if(memcmp(eina_binbuf_string_get(buf), cbuf, size));
fail_if(memcmp(eina_binbuf_string_get(buf) + size, cbuf, size));

-- 




[EGIT] [core/enlightenment] master 03/03: Fix null dereference issue

2015-11-10 Thread Vivek Ellur
discomfitor pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=dbfa9682bbc1f0993860d951da85c4b305ee9ae9

commit dbfa9682bbc1f0993860d951da85c4b305ee9ae9
Author: Vivek Ellur 
Date:   Tue Nov 10 12:26:48 2015 -0500

Fix null dereference issue

Summary:
@Fix
possible null dereference issue while referencing cs2->id. So added a 
condition
to check it

Signed-off-by: Vivek Ellur 

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3309
---
 src/modules/conf_randr/e_int_config_randr2.c | 7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/modules/conf_randr/e_int_config_randr2.c 
b/src/modules/conf_randr/e_int_config_randr2.c
index 97aa9eb..ac9ea88 100644
--- a/src/modules/conf_randr/e_int_config_randr2.c
+++ b/src/modules/conf_randr/e_int_config_randr2.c
@@ -390,8 +390,11 @@ _cb_rel_to_set(void *data, Evas_Object *obj, void *event)
 if (it == event)
   {
  E_Config_Randr2_Screen *cs2 = _config_screen_n_find(cfdata, i);
- printf("find cs = %p\n", cs2);
- printf("cs id = %s\n", cs2->id);
+ if (cs2)
+   {
+  printf("find cs = %p\n", cs2);
+  printf("cs id = %s\n", cs2->id);
+   }
  if (cs2 == cs) return;
  if (cs2)
{

-- 




[EGIT] [core/enlightenment] master 02/03: Fix memory leak issue

2015-11-10 Thread Vivek Ellur
discomfitor pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=54a613eda97da6060c73212f107d6fa9f32eebd1

commit 54a613eda97da6060c73212f107d6fa9f32eebd1
Author: Vivek Ellur 
Date:   Tue Nov 10 12:26:40 2015 -0500

Fix memory leak issue

Summary:
@Fix

Signed-off-by: Vivek Ellur 

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3307
---
 src/modules/wl_weekeyboard/e_mod_main.c | 11 ++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/src/modules/wl_weekeyboard/e_mod_main.c 
b/src/modules/wl_weekeyboard/e_mod_main.c
index bb5cbe7..c90f0fd 100644
--- a/src/modules/wl_weekeyboard/e_mod_main.c
+++ b/src/modules/wl_weekeyboard/e_mod_main.c
@@ -41,8 +41,17 @@ _wkb_insert_text(const char *text, uint32_t offset, const 
char *insert)
char *new_text = malloc(strlen(text) + strlen(insert) + 1);
uint32_t text_len = 0;
 
+   if (!new_text)
+ {
+ERR("out of memory");
+return NULL;
+ }
+
if ((!text) || (!insert))
- return NULL;
+ {
+free(new_text);
+return NULL;
+ }
 
text_len = strlen(text);
if (offset > text_len)

-- 




[EGIT] [core/enlightenment] master 01/03: e: Fix realloc issue in e font module

2015-11-10 Thread Vivek Ellur
discomfitor pushed a commit to branch master.

http://git.enlightenment.org/core/enlightenment.git/commit/?id=f026fdf9b345837d1145f657b2c8ef04765acc60

commit f026fdf9b345837d1145f657b2c8ef04765acc60
Author: Vivek Ellur 
Date:   Tue Nov 10 12:26:14 2015 -0500

e: Fix realloc issue in e font module

Summary:
@Fix

Signed-off-by: Vivek Ellur 

Reviewers: zmike

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3306
---
 src/bin/e_font.c | 8 +++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/src/bin/e_font.c b/src/bin/e_font.c
index e1f4e74..81423ac 100644
--- a/src/bin/e_font.c
+++ b/src/bin/e_font.c
@@ -159,7 +159,7 @@ _e_font_fontconfig_name_parse(Eina_Hash **font_hash, 
E_Font_Properties *efp, con
s1 = strchr(font, ':');
if (s1)
  {
-char *s2, *name, *style;
+char *s2, *name, *style, *temp;
 int len;
 
 len = s1 - font;
@@ -171,7 +171,13 @@ _e_font_fontconfig_name_parse(Eina_Hash **font_hash, 
E_Font_Properties *efp, con
 if (s2)
   {
  len = s2 - name;
+ temp = name;
  name = realloc(name, sizeof(char) * len + 1);
+ if (!name)
+   {
+  free(temp);
+  return NULL;
+   }
  memset(name, 0, sizeof(char) * len + 1);
  strncpy(name, font, len);
   }

-- 




[EGIT] [core/efl] master 22/24: eina: add test case for eina list search function.

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=7fa841236b38e5816b7f1fc7737a27e09f19a663

commit 7fa841236b38e5816b7f1fc7737a27e09f19a663
Author: Vivek Ellur 
Date:   Mon Nov 9 16:22:21 2015 -0800

eina: add test case for eina list search function.

Summary:
Added test case for eina_list_search_sorted function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3290

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_list.c | 7 +++
 1 file changed, 7 insertions(+)

diff --git a/src/tests/eina/eina_test_list.c b/src/tests/eina/eina_test_list.c
index 29d1d51..b23fa87 100644
--- a/src/tests/eina/eina_test_list.c
+++ b/src/tests/eina/eina_test_list.c
@@ -310,6 +310,7 @@ START_TEST(eina_test_sorted_insert)
int i, count;
Eina_List *l1, *l2, *itr;
void *d;
+   int *res, val = 2009;
 
eina_init();
 
@@ -322,6 +323,12 @@ START_TEST(eina_test_sorted_insert)
fail_if(l1 == NULL);
fail_if(!eina_list_sorted_check(l1));
 
+   res = eina_list_search_sorted(l1, eina_int_cmp, &data[7]);
+   fail_if(*res != 1664);
+
+   res = eina_list_search_sorted(l1, eina_int_cmp, &val);
+   fail_if(res != NULL);
+
l2 = NULL;
EINA_LIST_FOREACH(l1, itr, d)
l2 = eina_list_sorted_insert(l2, eina_int_cmp, d);

-- 




[EGIT] [core/efl] master 23/24: eina: add test case for file statat function.

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=873562608eb99c689e3216a017b46c51a93742a3

commit 873562608eb99c689e3216a017b46c51a93742a3
Author: Vivek Ellur 
Date:   Mon Nov 9 16:26:21 2015 -0800

eina: add test case for file statat function.

Summary:
Added test case for eina_file_statat function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3293

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_file.c | 50 +
 1 file changed, 50 insertions(+)

diff --git a/src/tests/eina/eina_test_file.c b/src/tests/eina/eina_test_file.c
index dc73f12..617ce88 100644
--- a/src/tests/eina/eina_test_file.c
+++ b/src/tests/eina/eina_test_file.c
@@ -681,6 +681,55 @@ START_TEST(eina_test_file_copy)
 }
 END_TEST
 
+START_TEST(eina_test_file_statat)
+{
+   Eina_Tmpstr *test_file1_path, *test_file2_path;
+   Eina_Iterator *it;
+   Eina_Stat st;
+   Eina_File_Direct_Info *info;
+   const char *template = "abcdefghijklmnopqrstuvwxyz";
+   int template_size = strlen(template);
+   int fd, ret;
+
+   eina_init();
+
+   Eina_Tmpstr *test_dirname = get_eina_test_file_tmp_dir();
+   fail_if(test_dirname == NULL);
+
+   test_file1_path = get_full_path(test_dirname, "example1.txt");
+   test_file2_path = get_full_path(test_dirname, "example2.txt");
+
+   fd = open(test_file1_path, O_WRONLY | O_CREAT | O_TRUNC, S_IWUSR | S_IRUSR);
+   fail_if(fd == 0);
+   fail_if(write(fd, template, template_size) != template_size);
+   close(fd);
+
+   fd = open(test_file2_path, O_WRONLY | O_CREAT | O_TRUNC, S_IWUSR | S_IRUSR);
+   fail_if(fd == 0);
+   fail_if(write(fd, template, template_size) != template_size);
+   close(fd);
+
+   it = eina_file_stat_ls(test_dirname);
+   fprintf(stderr, "file=%s\n", test_dirname);
+   EINA_ITERATOR_FOREACH(it, info)
+ {
+ret = eina_file_statat(eina_iterator_container_get(it), info, &st);
+fprintf(stderr, "ret=%d\n", ret);
+fail_if(ret != 0);
+fail_if(st.size != template_size);
+ }
+
+   unlink(test_file1_path);
+   unlink(test_file2_path);
+   fail_if(rmdir(test_dirname) != 0);
+   eina_tmpstr_del(test_file1_path);
+   eina_tmpstr_del(test_file2_path);
+   eina_tmpstr_del(test_dirname);
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_file(TCase *tc)
 {
@@ -695,4 +744,5 @@ eina_test_file(TCase *tc)
tcase_add_test(tc, eina_test_file_xattr);
 #endif
tcase_add_test(tc, eina_test_file_copy);
+   tcase_add_test(tc, eina_test_file_statat);
 }

-- 




[EGIT] [core/efl] master 17/24: eina: add test case for list demote function

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=fec6bd3fef426f75581ddffa0587db10da4052c6

commit fec6bd3fef426f75581ddffa0587db10da4052c6
Author: Vivek Ellur 
Date:   Mon Nov 9 16:18:03 2015 -0800

eina: add test case for list demote function

Summary:
Added test case for eina_list_demote_list function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3283

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_list.c | 5 +
 1 file changed, 5 insertions(+)

diff --git a/src/tests/eina/eina_test_list.c b/src/tests/eina/eina_test_list.c
index 4523311..1fea84b 100644
--- a/src/tests/eina/eina_test_list.c
+++ b/src/tests/eina/eina_test_list.c
@@ -77,6 +77,11 @@ START_TEST(eina_test_simple)
list = eina_list_append(list, &data[2]);
 fail_if(list == NULL);
 
+   list = eina_list_demote_list(list, eina_list_nth_list(list, 1));
+   test1 = eina_list_nth(list, 2);
+   fail_if(test1 == NULL);
+   fail_if(*test1 != 6);
+
list = eina_list_remove(list, &data[0]);
 fail_if(list == NULL);
 

-- 




[EGIT] [core/efl] master 19/24: eina: add test case for list data idx function.

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=4f6e158c13abf4182832df3e26c6af3a47e4b155

commit 4f6e158c13abf4182832df3e26c6af3a47e4b155
Author: Vivek Ellur 
Date:   Mon Nov 9 16:19:43 2015 -0800

eina: add test case for list data idx function.

Summary:
Added test case for eina_list_data_idx function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3287
---
 src/tests/eina/eina_test_list.c | 9 +
 1 file changed, 9 insertions(+)

diff --git a/src/tests/eina/eina_test_list.c b/src/tests/eina/eina_test_list.c
index 1ae20cb..29d1d51 100644
--- a/src/tests/eina/eina_test_list.c
+++ b/src/tests/eina/eina_test_list.c
@@ -77,6 +77,15 @@ START_TEST(eina_test_simple)
list = eina_list_append(list, &data[2]);
 fail_if(list == NULL);
 
+   i = eina_list_data_idx(list, &data[1]);
+   fail_if(i != 0);
+
+   i = eina_list_data_idx(list, &data[2]);
+   fail_if(i != 2);
+
+   i = eina_list_data_idx(list, &data[3]);
+   fail_if(i != -1);
+
list = eina_list_demote_list(list, eina_list_nth_list(list, 1));
test1 = eina_list_nth(list, 2);
fail_if(test1 == NULL);

-- 




[EGIT] [core/efl] master 16/24: eina: add test case for eina list reverse iterator

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=f06eedf3c339af6647fc216ca06f29de53831c70

commit f06eedf3c339af6647fc216ca06f29de53831c70
Author: Vivek Ellur 
Date:   Mon Nov 9 16:17:24 2015 -0800

eina: add test case for eina list reverse iterator

Summary:
Added test case for eina_list_iterator_reversed_new function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3281

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_iterator.c | 66 +
 1 file changed, 66 insertions(+)

diff --git a/src/tests/eina/eina_test_iterator.c 
b/src/tests/eina/eina_test_iterator.c
index 4c14dec..5c4b88d 100644
--- a/src/tests/eina/eina_test_iterator.c
+++ b/src/tests/eina/eina_test_iterator.c
@@ -287,6 +287,71 @@ START_TEST(eina_iterator_list_simple)
 }
 END_TEST
 
+static Eina_Bool
+eina_reverse_iterator_list_data_check(EINA_UNUSED const Eina_List *list,
+  int *data,
+  int *fdata)
+{
+   switch (*fdata)
+ {
+  case 0: fail_if(*data != 1337); break;
+
+  case 1: fail_if(*data != 1); break;
+
+  case 2: fail_if(*data != 42); break;
+
+  case 3: fail_if(*data != 6); break;
+
+  case 4: fail_if(*data != 9); break;
+
+  case 5: fail_if(*data != 7); break;
+
+  case 6: fail_if(*data != 81); break;
+ }
+
+   (*fdata)++;
+
+   return EINA_TRUE;
+}
+
+START_TEST(eina_reverse_iterator_list_simple)
+{
+   Eina_List *list = NULL;
+   Eina_Iterator *it;
+   int data[] = { 6, 9, 42, 1, 7, 1337, 81, 1664 };
+   int i = 0;
+
+   eina_init();
+
+   list = eina_list_append(list, &data[0]);
+   fail_if(list == NULL);
+
+   list = eina_list_prepend(list, &data[1]);
+   fail_if(list == NULL);
+
+   list = eina_list_append(list, &data[2]);
+   fail_if(list == NULL);
+
+   list = eina_list_append(list, &data[3]);
+   fail_if(list == NULL);
+
+   list = eina_list_prepend(list, &data[4]);
+   fail_if(list == NULL);
+
+   list = eina_list_append(list, &data[5]);
+   fail_if(list == NULL);
+
+   list = eina_list_prepend(list, &data[6]);
+   fail_if(list == NULL);
+
+   it = eina_list_iterator_reversed_new(list);
+   fail_if(!it);
+
+   eina_iterator_foreach(it, 
EINA_EACH_CB(eina_reverse_iterator_list_data_check), &i);
+   eina_iterator_free(it);
+}
+END_TEST
+
 typedef struct _Eina_Rbtree_Int Eina_Rbtree_Int;
 struct _Eina_Rbtree_Int
 {
@@ -461,5 +526,6 @@ eina_test_iterator(TCase *tc)
tcase_add_test(tc, eina_iterator_hash_simple);
tcase_add_test(tc, eina_iterator_inlist_simple);
tcase_add_test(tc, eina_iterator_list_simple);
+   tcase_add_test(tc, eina_reverse_iterator_list_simple);
tcase_add_test(tc, eina_iterator_rbtree_simple);
 }

-- 




[EGIT] [core/efl] master 18/24: eina: add test cases for eina list move functions

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=6bf43179cd128b3fe2d491c82b34e5cd729c1a30

commit 6bf43179cd128b3fe2d491c82b34e5cd729c1a30
Author: Vivek Ellur 
Date:   Mon Nov 9 16:18:58 2015 -0800

eina: add test cases for eina list move functions

Summary:
Added test cases for eina_list_move and eina_list_move_list functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3285

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_list.c | 41 +
 1 file changed, 41 insertions(+)

diff --git a/src/tests/eina/eina_test_list.c b/src/tests/eina/eina_test_list.c
index 1fea84b..1ae20cb 100644
--- a/src/tests/eina/eina_test_list.c
+++ b/src/tests/eina/eina_test_list.c
@@ -489,6 +489,46 @@ START_TEST(eina_test_clone)
 }
 END_TEST
 
+START_TEST(eina_test_move)
+{
+   Eina_List *list1 = NULL, *list2 = NULL;
+   Eina_Bool ret;
+   int data1[] = {1, 2, 3, 4, 5};
+   int data2[] = {6, 7, 8, 9, 10};
+   int i, *list_data;
+
+   eina_init();
+
+   for (i = 0; i < 5; i++)
+   {
+  list1 = eina_list_append(list1, &data1[i]);
+  list2 = eina_list_append(list2, &data2[i]);
+   }
+   fail_if(eina_list_count(list1) != 5);
+   fail_if(eina_list_count(list2) != 5);
+
+   ret = eina_list_move(&list1, &list2, &data2[4]);
+   fail_if(ret != EINA_TRUE);
+   fail_if(eina_list_count(list1) != 6);
+   fail_if(eina_list_count(list2) != 4);
+   list_data = eina_list_nth(list1, 5);
+   fail_if(*list_data != 10);
+
+   ret = eina_list_move_list(&list1, &list2,
+ eina_list_nth_list(list2, 1));
+   fail_if(ret != EINA_TRUE);
+   fail_if(eina_list_count(list1) != 7);
+   fail_if(eina_list_count(list2) != 3);
+   list_data = eina_list_nth(list1, 6);
+   fail_if(*list_data != 7);
+
+   eina_list_free(list1);
+   eina_list_free(list2);
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_list(TCase *tc)
 {
@@ -498,4 +538,5 @@ eina_test_list(TCase *tc)
tcase_add_test(tc, eina_test_list_split);
tcase_add_test(tc, eina_test_shuffle);
tcase_add_test(tc, eina_test_clone);
+   tcase_add_test(tc, eina_test_move);
 }

-- 




[EGIT] [core/efl] master 15/24: eina: added test case for matrix4 transpose function

2015-11-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=b5e672aad883143184e61efe4a9a0df1110e3be1

commit b5e672aad883143184e61efe4a9a0df1110e3be1
Author: Vivek Ellur 
Date:   Mon Nov 9 16:16:05 2015 -0800

eina: added test case for matrix4 transpose function

Summary:
Added test case for eina_matrix4_transpose function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3279

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_matrix.c | 12 
 1 file changed, 12 insertions(+)

diff --git a/src/tests/eina/eina_test_matrix.c 
b/src/tests/eina/eina_test_matrix.c
index 11cc2c6..50323fb 100644
--- a/src/tests/eina/eina_test_matrix.c
+++ b/src/tests/eina/eina_test_matrix.c
@@ -167,6 +167,18 @@ START_TEST(eina_matrix4)
wy != wz ||
wz != 0);
 
+   eina_matrix4_values_set(&m,
+   1, 2, 3, 4,
+   5, 6, 7, 8,
+   9, 10, 11, 12,
+   13, 14, 15, 16);
+
+   eina_matrix4_transpose(&n, &m);
+   fail_if(n.xx != 1 || n.xy != 5 || n.xz != 9 || n.xw != 13 ||
+   n.yx != 2 || n.yy != 6 || n.yz != 10 || n.yw != 14 ||
+   n.zx != 3 || n.zy != 7 || n.zz != 11 || n.zw != 15 ||
+   n.wx != 4 || n.wy != 8 || n.wz != 12 || n.ww != 16);
+
eina_shutdown();
 }
 END_TEST

-- 




[EGIT] [core/efl] master 02/12: eina_quaternion: add test cases for quaternion rotation functions

2015-10-30 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=6090073dee6b8a283a2d404efde8ec9205e485d9

commit 6090073dee6b8a283a2d404efde8ec9205e485d9
Author: Vivek Ellur 
Date:   Fri Oct 30 03:59:21 2015 +0100

eina_quaternion: add test cases for quaternion rotation functions

Summary:
Added test cases for eina_quaternion_f16p16_rotate and
eina_quaternion_f16p16_rotation_matrix3_get functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3247

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_quaternion.c | 40 +++
 1 file changed, 40 insertions(+)

diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index 058ca3c..e1b876b 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -57,6 +57,22 @@ eina_matrix3_cmp(const Eina_Matrix3 *a, const Eina_Matrix3 
*b)
 }
 
 static inline Eina_Bool
+eina_matrix3_f16p16_cmp(const Eina_Matrix3_F16p16 *a, const 
Eina_Matrix3_F16p16 *b)
+{
+   if ((a->xx == b->xx) &&
+   (a->xy == b->xy) &&
+   (a->xz == b->xz) &&
+   (a->yx == b->yx) &&
+   (a->yy == b->yy) &&
+   (a->yz == b->yz) &&
+   (a->zx == b->zx) &&
+   (a->zy == b->zy) &&
+   (a->zz == b->zz))
+ return EINA_TRUE;
+   return EINA_FALSE;
+}
+
+static inline Eina_Bool
 eina_point_3d_cmp(const Eina_Point_3D *a, const Eina_Point_3D *b)
 {
if (FLOAT_CMP(a->x, b->x) &&
@@ -414,6 +430,29 @@ START_TEST(eina_test_quaternion_lerp)
 }
 END_TEST
 
+START_TEST(eina_test_quaternion_f16p16_rotate_matrix)
+{
+   Eina_Quaternion_F16p16 q = {65536, 65536, 65536, 0};
+   Eina_Point_3D_F16p16 r = { 65536, 65536, 65536 };
+   Eina_Point_3D_F16p16 c = { 0, 0, 0 }, res = {65536, 65536, 65536};
+   Eina_Matrix3_F16p16 m, mres = {-262144, 131072, 131072,
+  131072, -262144, 131072,
+  131072, 131072, -262144 };
+
+   eina_init();
+
+   eina_quaternion_f16p16_rotate(&r, &c, &q);
+   fail_if(r.x != res.x ||
+   r.y != res.y ||
+   r.z != res.z);
+
+   eina_quaternion_f16p16_rotation_matrix3_get(&m, &q);
+   fail_if(!eina_matrix3_f16p16_cmp(&m, &mres));
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_quaternion(TCase *tc)
 {
@@ -431,4 +470,5 @@ eina_test_quaternion(TCase *tc)
tcase_add_test(tc, eina_test_matrix_recompose);
tcase_add_test(tc, eina_test_quaternion_f16p16_lerp);
tcase_add_test(tc, eina_test_quaternion_lerp);
+   tcase_add_test(tc, eina_test_quaternion_f16p16_rotate_matrix);
 }

-- 




[EGIT] [core/efl] master 04/12: eina_hash: add test cases for eina_hash_add/del by hash functions

2015-10-30 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=85f8daac6d3554f980d8737ee7b904d6a5be203d

commit 85f8daac6d3554f980d8737ee7b904d6a5be203d
Author: Vivek Ellur 
Date:   Fri Oct 30 04:06:25 2015 +0100

eina_hash: add test cases for eina_hash_add/del by hash functions

Summary:
Added test cases for eina_hash_add_by_hash/del_by_hash functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3245

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_hash.c | 35 +++
 1 file changed, 35 insertions(+)

diff --git a/src/tests/eina/eina_test_hash.c b/src/tests/eina/eina_test_hash.c
index b2f9704..cc44bff 100644
--- a/src/tests/eina/eina_test_hash.c
+++ b/src/tests/eina/eina_test_hash.c
@@ -418,6 +418,40 @@ START_TEST(eina_hash_seed)
 }
 END_TEST
 
+START_TEST(eina_hash_add_del_by_hash)
+{
+   Eina_Hash *hash = NULL;
+   int array[] = { 1, 42, 4, 5, 6 };
+   int key_len, key_hash;
+
+   fail_if(eina_init() != 2);
+
+   hash = eina_hash_new(EINA_KEY_LENGTH(_eina_string_key_length),
+EINA_KEY_CMP(_eina_string_key_cmp),
+EINA_KEY_HASH(eina_hash_crc),
+NULL,
+EINA_HASH_BUCKET_SIZE);
+   fail_if(hash == NULL);
+   fail_if(eina_hash_add(hash, "1", &array[0]) != EINA_TRUE);
+   fail_if(eina_hash_add(hash, "42", &array[1]) != EINA_TRUE);
+   fail_if(eina_hash_add(hash, "5", &array[3]) != EINA_TRUE);
+   fail_if(eina_hash_add(hash, "6", &array[4]) != EINA_TRUE);
+
+   key_len = _eina_string_key_length("4");
+   key_hash = eina_hash_crc("4", key_len);
+
+   fail_if(eina_hash_add_by_hash(hash, "4", key_len, key_hash, &array[2]) != 
EINA_TRUE);
+   fail_if(eina_hash_del_by_hash(hash, "4", key_len, key_hash, &array[2]) != 
EINA_TRUE);
+   fail_if(eina_hash_del_by_hash(hash, "4", key_len, key_hash, &array[2]) != 
EINA_FALSE);
+
+   key_len = _eina_string_key_length("42");
+   key_hash =  eina_hash_crc("42", key_len);
+   fail_if(eina_hash_del_by_hash(hash, "42", key_len, key_hash, &array[1]) != 
EINA_TRUE);
+
+   fail_if(eina_hash_population(hash) != 3);
+}
+END_TEST
+
 void eina_test_hash(TCase *tc)
 {
tcase_add_test(tc, eina_hash_simple);
@@ -428,4 +462,5 @@ void eina_test_hash(TCase *tc)
tcase_add_test(tc, eina_hash_seed);
tcase_add_test(tc, eina_hash_int32_fuzze);
tcase_add_test(tc, eina_hash_string_fuzze);
+   tcase_add_test(tc, eina_hash_add_del_by_hash);
 }

-- 




[EGIT] [core/efl] master 01/12: eina_quaternion: correct the API name in header file

2015-10-30 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=0080ee0d607faa82a4376f9366ffb9e638a32f25

commit 0080ee0d607faa82a4376f9366ffb9e638a32f25
Author: Vivek Ellur 
Date:   Fri Oct 30 03:57:06 2015 +0100

eina_quaternion: correct the API name in header file

Summary:
This is not an ABI break as we never did provide the broken name symbol, 
just a typo
in the header that wasn't detected until now.

@fix

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3248

Signed-off-by: Cedric BAIL 
---
 src/lib/eina/eina_quaternion.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/eina/eina_quaternion.h b/src/lib/eina/eina_quaternion.h
index c63aeab..df66589 100644
--- a/src/lib/eina/eina_quaternion.h
+++ b/src/lib/eina/eina_quaternion.h
@@ -87,7 +87,7 @@ EAPI void eina_quaternion_f16p16_nlerp(Eina_Quaternion_F16p16 
*out,
 EAPI void eina_quaternion_f16p16_rotate(Eina_Point_3D_F16p16 *p,
 const Eina_Point_3D_F16p16 *center,
 const Eina_Quaternion_F16p16 *q); /**< 
@since 1.15 */
-EAPI void eina_quaternion_f16p16_rotation_matri3_get(Eina_Matrix3_F16p16 *m,
+EAPI void eina_quaternion_f16p16_rotation_matrix3_get(Eina_Matrix3_F16p16 *m,
  const 
Eina_Quaternion_F16p16 *q); /**< @since 1.15 */
 
 EAPI void eina_quaternion_set(Eina_Quaternion *q, double x,

-- 




[EGIT] [core/efl] master 05/12: ector: fix null pointer dereference issue

2015-10-30 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=96f936e6d371421809b2f8804b029f966e2b14d7

commit 96f936e6d371421809b2f8804b029f966e2b14d7
Author: Vivek Ellur 
Date:   Fri Oct 30 06:39:15 2015 +0100

ector: fix null pointer dereference issue

Summary:
Fix Coverity CID1293003

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3220

Signed-off-by: Cedric BAIL 
---
 src/lib/ector/software/ector_software_rasterizer.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/src/lib/ector/software/ector_software_rasterizer.c 
b/src/lib/ector/software/ector_software_rasterizer.c
index b436eed..0cb74a0 100644
--- a/src/lib/ector/software/ector_software_rasterizer.c
+++ b/src/lib/ector/software/ector_software_rasterizer.c
@@ -50,6 +50,9 @@ _blend_gradient(int count, const SW_FT_Span *spans, void 
*user_data)
if (data->type == LinearGradient) fetchfunc = &fetch_linear_gradient;
if (data->type == RadialGradient) fetchfunc = &fetch_radial_gradient;
 
+   if (!fetchfunc)
+ return;
+
comp_func = ector_comp_func_span_get(data->op, data->mul_col, 
data->gradient->alpha);
 
// move to the offset location

-- 




[EGIT] [core/efl] master 12/12: ecore: add test case for idler enterer function

2015-10-30 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=081e9894d4c5636ed3bd59344cebca1b0a7fe7a3

commit 081e9894d4c5636ed3bd59344cebca1b0a7fe7a3
Author: Vivek Ellur 
Date:   Sat Oct 31 02:37:17 2015 +0100

ecore: add test case for idler enterer function

Summary:
Added test case for ecore_idle_enterer_before_add function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3253

Signed-off-by: Cedric BAIL 
---
 src/tests/ecore/ecore_test_ecore.c | 21 +
 1 file changed, 21 insertions(+)

diff --git a/src/tests/ecore/ecore_test_ecore.c 
b/src/tests/ecore/ecore_test_ecore.c
index 08a00e2..a5331b4 100644
--- a/src/tests/ecore/ecore_test_ecore.c
+++ b/src/tests/ecore/ecore_test_ecore.c
@@ -106,6 +106,26 @@ START_TEST(ecore_test_ecore_main_loop_idle_enterer)
 }
 END_TEST
 
+START_TEST(ecore_test_ecore_main_loop_idle_before_enterer)
+{
+   Eina_Bool did = EINA_FALSE;
+   Ecore_Idle_Enterer *idle_enterer;
+   int ret;
+
+   ret = ecore_init();
+   fail_if(ret < 1);
+
+   idle_enterer = ecore_idle_enterer_before_add(_quit_cb, &did);
+   fail_if(idle_enterer == NULL);
+
+   ecore_main_loop_begin();
+
+   fail_if(did == EINA_FALSE);
+
+   ret = ecore_shutdown();
+}
+END_TEST
+
 START_TEST(ecore_test_ecore_main_loop_idle_exiter)
 {
Eina_Bool did = EINA_FALSE;
@@ -681,6 +701,7 @@ void ecore_test_ecore(TCase *tc)
tcase_add_test(tc, ecore_test_ecore_main_loop);
tcase_add_test(tc, ecore_test_ecore_main_loop_idler);
tcase_add_test(tc, ecore_test_ecore_main_loop_idle_enterer);
+   tcase_add_test(tc, ecore_test_ecore_main_loop_idle_before_enterer);
tcase_add_test(tc, ecore_test_ecore_main_loop_idle_exiter);
tcase_add_test(tc, ecore_test_ecore_main_loop_timer);
tcase_add_test(tc, ecore_test_ecore_main_loop_fd_handler);

-- 




[EGIT] [core/efl] master 03/12: eina_quaternion: add test case for rotation function

2015-10-30 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=dc391da42430624e1ee91bfba8fc326a9c77ed83

commit dc391da42430624e1ee91bfba8fc326a9c77ed83
Author: Vivek Ellur 
Date:   Fri Oct 30 04:00:33 2015 +0100

eina_quaternion: add test case for rotation function

Summary:
Added test case for eina_quaternion_rotate function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3246

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_quaternion.c | 21 +
 1 file changed, 21 insertions(+)

diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index e1b876b..2401dcb 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -453,6 +453,26 @@ START_TEST(eina_test_quaternion_f16p16_rotate_matrix)
 }
 END_TEST
 
+START_TEST(eina_test_quaternion_rotate)
+{
+   Eina_Point_3D r = { 3, 3, 3 };
+   Eina_Point_3D c = { 0, 0, 0}, res = {3.0, 3.0, 3.0};
+   Eina_Point_3D res1 = {3.0, 3.0, -9.0};
+   Eina_Quaternion q = {1, 1, 1, 0};
+
+   eina_init();
+
+   eina_quaternion_rotate(&r, &c, &q);
+   fail_if(!eina_point_3d_cmp(&r, &res));
+
+   eina_quaternion_set(&q, 1, 1, 0, 0);
+   eina_quaternion_rotate(&r, &c, &q);
+   fail_if(!eina_point_3d_cmp(&r, &res1));
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_quaternion(TCase *tc)
 {
@@ -471,4 +491,5 @@ eina_test_quaternion(TCase *tc)
tcase_add_test(tc, eina_test_quaternion_f16p16_lerp);
tcase_add_test(tc, eina_test_quaternion_lerp);
tcase_add_test(tc, eina_test_quaternion_f16p16_rotate_matrix);
+   tcase_add_test(tc, eina_test_quaternion_rotate);
 }

-- 




[EGIT] [core/elementary] master 01/02: dayselector: fix warning related to argument type

2015-10-22 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=a10e9664c8a51e9d8d8997ab53ec7e76b9a26220

commit a10e9664c8a51e9d8d8997ab53ec7e76b9a26220
Author: Vivek Ellur 
Date:   Thu Oct 22 12:37:44 2015 -0700

dayselector: fix warning related to argument type

Summary:
@fix

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Differential Revision: https://phab.enlightenment.org/D3215

Signed-off-by: Cedric BAIL 
---
 src/lib/elm_dayselector.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/lib/elm_dayselector.c b/src/lib/elm_dayselector.c
index 6f60147..0f76ab9 100644
--- a/src/lib/elm_dayselector.c
+++ b/src/lib/elm_dayselector.c
@@ -401,7 +401,7 @@ _items_create(Evas_Object *obj)
 strftime(buf, sizeof(buf), "%a", &time_daysel);
 elm_object_text_set(chk, buf);
 
-snprintf(buf, sizeof(buf), "day%d", idx);
+snprintf(buf, sizeof(buf), "day%u", idx);
 elm_layout_content_set(obj, buf, chk);
 
 // XXX: ACCESS

-- 




[EGIT] [core/elementary] master 02/02: elm_index: Fix memory leak issue in realloc failure

2015-10-22 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=4492153c6ad360a97ab990f340ebf5be55218789

commit 4492153c6ad360a97ab990f340ebf5be55218789
Author: Vivek Ellur 
Date:   Thu Oct 22 12:38:28 2015 -0700

elm_index: Fix memory leak issue in realloc failure

Summary:
@Fix

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Differential Revision: https://phab.enlightenment.org/D3214
---
 src/lib/elm_index.c | 9 ++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/src/lib/elm_index.c b/src/lib/elm_index.c
index 83194a7..7f3cb39 100644
--- a/src/lib/elm_index.c
+++ b/src/lib/elm_index.c
@@ -770,14 +770,17 @@ _sel_eval(Evas_Object *obj,
 {
if (label && last)
  {
-label = realloc(label, strlen(label) +
+char *temp;
+
+temp = realloc(label, strlen(label) +
 strlen(last) + 1);
-if (!label)
+if (!temp)
   {
+ free(label);
  free(last);
  return;
   }
-strcat(label, last);
+label = strcat(temp, last);
  }
 }
   free(last);

-- 




[EGIT] [core/efl] master 02/05: eina_queue: add test cases in eina thread queue module

2015-10-12 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=4f604e4d1ee0d32d48d13676f2aef514b47c4885

commit 4f604e4d1ee0d32d48d13676f2aef514b47c4885
Author: Vivek Ellur 
Date:   Mon Oct 12 12:09:04 2015 -0700

eina_queue: add test cases in eina thread queue module

Summary:
Added test cases for eina_thread_queue_parent_get and 
eina_thread_queue_fd_get
functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3165

Signed-off-by: Cedric BAIL 
---
 .../ecore/ecore_test_ecore_thread_eina_thread_queue.c  | 14 +-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/tests/ecore/ecore_test_ecore_thread_eina_thread_queue.c 
b/src/tests/ecore/ecore_test_ecore_thread_eina_thread_queue.c
index 616cb72..d5bdc27 100644
--- a/src/tests/ecore/ecore_test_ecore_thread_eina_thread_queue.c
+++ b/src/tests/ecore/ecore_test_ecore_thread_eina_thread_queue.c
@@ -196,6 +196,8 @@ th22_do(void *data EINA_UNUSED, Ecore_Thread *th 
EINA_UNUSED)
 START_TEST(ecore_test_ecore_thread_eina_thread_queue_t3)
 {
int val1 = 99, val2 = 99, cnt = 0;
+   Eina_Thread_Queue *parent;
+
eina_init();
ecore_init();
 
@@ -204,6 +206,12 @@ START_TEST(ecore_test_ecore_thread_eina_thread_queue_t3)
thqmaster = eina_thread_queue_new();
eina_thread_queue_parent_set(thq1, thqmaster);
eina_thread_queue_parent_set(thq2, thqmaster);
+
+   parent = eina_thread_queue_parent_get(thq1);
+   fail_if(parent != thqmaster);
+   parent = eina_thread_queue_parent_get(thq2);
+   fail_if(parent != thqmaster);
+
ecore_thread_feedback_run(th21_do, NULL, NULL, NULL, NULL, EINA_TRUE);
ecore_thread_feedback_run(th22_do, NULL, NULL, NULL, NULL, EINA_TRUE);
for (;;)
@@ -546,7 +554,7 @@ START_TEST(ecore_test_ecore_thread_eina_thread_queue_t7)
 {
Msg7 *msg;
void *ref;
-   int msgcnt = 0;
+   int msgcnt = 0, ret;
 
eina_init();
ecore_init();
@@ -558,6 +566,10 @@ START_TEST(ecore_test_ecore_thread_eina_thread_queue_t7)
 fail();
  }
eina_thread_queue_fd_set(thq1, p[1]);
+
+   ret = eina_thread_queue_fd_get(thq1);
+   fail_if(ret != p[1]);
+
ecore_thread_feedback_run(thspeed21_do, NULL, NULL, NULL, NULL, EINA_TRUE);
for (;;)
  {

-- 




[EGIT] [core/efl] master 03/05: eina_tiler: add test cases for eina tiler area set/get functions

2015-10-12 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=2136ad5ecb8ec7860eba0e038e69c0f101e15144

commit 2136ad5ecb8ec7860eba0e038e69c0f101e15144
Author: Vivek Ellur 
Date:   Mon Oct 12 12:11:53 2015 -0700

eina_tiler: add test cases for eina tiler area set/get functions

Summary:
Added test cases for eina_tiler_area_size_set and get functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3164

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_tiler.c | 12 +++-
 1 file changed, 11 insertions(+), 1 deletion(-)

diff --git a/src/tests/eina/eina_test_tiler.c b/src/tests/eina/eina_test_tiler.c
index 6766b62..0f3961e 100644
--- a/src/tests/eina/eina_test_tiler.c
+++ b/src/tests/eina/eina_test_tiler.c
@@ -134,10 +134,20 @@ START_TEST(eina_test_tiler_all)
Eina_Rectangle *rp;
Eina_Rectangle r;
int i = 0;
+   int width, height;
 
eina_init();
 
-   tl = eina_tiler_new(640, 480);
+   tl = eina_tiler_new(1, 1);
+
+   eina_tiler_area_size_get(tl, &width, &height);
+   fail_if(width != 1 && height != 1);
+
+   width = 640;
+   height = 480;
+   eina_tiler_area_size_set(tl, width, height);
+   eina_tiler_area_size_get(tl, &width, &height);
+   fail_if(width != 640 && height != 480);
 
eina_tiler_tile_size_set(tl, 32, 32);
 

-- 




[EGIT] [core/efl] master 03/09: eio_monitor: fix memory leak in eio monitor module

2015-10-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=d64f50acfc01400e7465cf0288df09dfce7c3a94

commit d64f50acfc01400e7465cf0288df09dfce7c3a94
Author: Vivek Ellur 
Date:   Thu Oct 8 12:17:29 2015 +0200

eio_monitor: fix memory leak in eio monitor module

Summary:
Free the allocated memory in eio monitor win32 module.

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3160

Signed-off-by: Cedric BAIL 
---
 src/lib/eio/eio_monitor_win32.c | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/src/lib/eio/eio_monitor_win32.c b/src/lib/eio/eio_monitor_win32.c
index 06a5f0c..43c38f3 100644
--- a/src/lib/eio/eio_monitor_win32.c
+++ b/src/lib/eio/eio_monitor_win32.c
@@ -86,7 +86,10 @@ _eio_monitor_win32_cb(void *data, Ecore_Win32_Handler *wh 
EINA_UNUSED)
 return 0;
 
   if (fni->FileName[0] == 0)
-return ECORE_CALLBACK_CANCEL;
+{
+   free(wname);
+   return ECORE_CALLBACK_CANCEL;
+}
 
   memcpy(wname, fni->FileName, fni->FileNameLength);
   wname[fni->FileNameLength / sizeof(wchar_t)] = 0;

-- 




[EGIT] [core/efl] master 02/09: eio_monitor: fix memory leak issue

2015-10-09 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=491fdb94b0c85aff965662a2152863cd3bd19d11

commit 491fdb94b0c85aff965662a2152863cd3bd19d11
Author: Vivek Ellur 
Date:   Thu Oct 8 12:16:23 2015 +0200

eio_monitor: fix memory leak issue

Summary:
@fix

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D3161

Signed-off-by: Cedric BAIL 
---
 src/lib/eio/eio_monitor_cocoa.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/lib/eio/eio_monitor_cocoa.c b/src/lib/eio/eio_monitor_cocoa.c
index 8cadeaf..d491de4 100644
--- a/src/lib/eio/eio_monitor_cocoa.c
+++ b/src/lib/eio/eio_monitor_cocoa.c
@@ -330,6 +330,8 @@ void eio_monitor_backend_add(Eio_Monitor *monitor)
 
if (!_stream)
  {
+free(monitor_path);
+free(backend);
 eio_monitor_fallback_add(monitor);
 return;
  }

-- 




[EGIT] [core/elementary] master 01/06: elm_video: port eo_event callbacks to use eo_event arrays

2015-10-04 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=ba30cf95b58aaeae5d877d79f8ce2d73a7dcb257

commit ba30cf95b58aaeae5d877d79f8ce2d73a7dcb257
Author: Vivek Ellur 
Date:   Sun Oct 4 12:05:51 2015 +0200

elm_video: port eo_event callbacks to use eo_event arrays

Summary:
Changed eo_event_callback_add to use eo_event_callback_array_add
in elm_video.c

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Differential Revision: https://phab.enlightenment.org/D3096

Signed-off-by: Cedric BAIL 
---
 src/lib/elm_video.c | 49 +
 1 file changed, 37 insertions(+), 12 deletions(-)

diff --git a/src/lib/elm_video.c b/src/lib/elm_video.c
index 3502971..aea83a9 100644
--- a/src/lib/elm_video.c
+++ b/src/lib/elm_video.c
@@ -27,6 +27,32 @@ static const Evas_Smart_Cb_Description _smart_callbacks[] = {
{NULL, NULL}
 };
 
+
+static Eina_Bool
+_on_open_done(void *data,
+  Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
+  void *event_info EINA_UNUSED);
+static Eina_Bool
+_on_playback_started(void *data,
+ Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
+ void *event_info EINA_UNUSED);
+static Eina_Bool
+_on_playback_finished(void *data,
+  Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
+  void *event_info EINA_UNUSED);
+static Eina_Bool
+_on_aspect_ratio_updated(void *data,
+ Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
+ void *event_info EINA_UNUSED);
+static Eina_Bool
+_on_title_changed(void *data,
+  Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
+  void *event_info EINA_UNUSED);
+static Eina_Bool
+_on_audio_level_changed(void *data,
+Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
+void *event_info EINA_UNUSED);
+
 static Eina_Bool _key_action_move(Evas_Object *obj, const char *params);
 static Eina_Bool _key_action_play(Evas_Object *obj, const char *params);
 
@@ -36,6 +62,15 @@ static const Elm_Action key_actions[] = {
{NULL, NULL}
 };
 
+EO_CALLBACKS_ARRAY_DEFINE(_video_cb,
+   { EMOTION_OBJECT_EVENT_OPEN_DONE, _on_open_done },
+   { EMOTION_OBJECT_EVENT_PLAYBACK_STARTED, _on_playback_started },
+   { EMOTION_OBJECT_EVENT_PLAYBACK_FINISHED, _on_playback_finished },
+   { EMOTION_OBJECT_EVENT_FRAME_RESIZE, _on_aspect_ratio_updated },
+   { EMOTION_OBJECT_EVENT_TITLE_CHANGE, _on_title_changed },
+   { EMOTION_OBJECT_EVENT_AUDIO_LEVEL_CHANGE, _on_audio_level_changed }
+);
+
 static Eina_Bool
 _key_action_move(Evas_Object *obj, const char *params)
 {
@@ -155,6 +190,7 @@ _on_playback_started(void *data,
return EINA_TRUE;
 
 }
+
 static Eina_Bool
 _on_playback_finished(void *data,
   Eo *obj EINA_UNUSED, const Eo_Event_Description *desc 
EINA_UNUSED,
@@ -251,18 +287,7 @@ _elm_video_evas_object_smart_add(Eo *obj, Elm_Video_Data 
*priv)
 
elm_layout_content_set(obj, "elm.swallow.video", priv->emotion);
 
-   eo_do(priv->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_OPEN_DONE, _on_open_done, obj));
-   eo_do(priv->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_PLAYBACK_STARTED, _on_playback_started, obj));
-   eo_do(priv->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_PLAYBACK_FINISHED, _on_playback_finished, obj));
-   eo_do(priv->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_FRAME_RESIZE, _on_aspect_ratio_updated, obj));
-   eo_do(priv->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_TITLE_CHANGE, _on_title_changed, obj));
-   eo_do(priv->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_AUDIO_LEVEL_CHANGE, _on_audio_level_changed, obj));
+   eo_do(priv->emotion, eo_event_callback_array_add(_video_cb(), obj));
 
evas_object_event_callback_add
  (obj, EVAS_CALLBACK_CHANGED_SIZE_HINTS, _on_size_hints_changed, NULL);

-- 




[EGIT] [core/elementary] master 02/06: elm_spinner: ported eo_event_callbacks to use eo_event arrays

2015-09-23 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=baa946bbf786e3db7922ad212d57c094573350d0

commit baa946bbf786e3db7922ad212d57c094573350d0
Author: Vivek Ellur 
Date:   Wed Sep 23 15:03:48 2015 -0700

elm_spinner: ported eo_event_callbacks to use eo_event arrays

Summary:
Ported eo_event_callback_add to use eo_event_callback_array_add

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Differential Revision: https://phab.enlightenment.org/D3092

Signed-off-by: Cedric BAIL 
---
 src/lib/elm_spinner.c | 67 +++
 1 file changed, 46 insertions(+), 21 deletions(-)

diff --git a/src/lib/elm_spinner.c b/src/lib/elm_spinner.c
index 09cb983..3c5b31b 100644
--- a/src/lib/elm_spinner.c
+++ b/src/lib/elm_spinner.c
@@ -41,12 +41,49 @@ static const Evas_Smart_Cb_Description _smart_callbacks[] = 
{
 static Eina_Bool _key_action_spin(Evas_Object *obj, const char *params);
 static Eina_Bool _key_action_toggle(Evas_Object *obj, const char *params);
 
+static Eina_Bool
+_inc_button_clicked_cb(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_inc_button_pressed_cb(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_inc_button_unpressed_cb(void *data, Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_inc_dec_button_mouse_move_cb(void *data, Eo *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED, 
void *event_info);
+static Eina_Bool
+_dec_button_clicked_cb(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_dec_button_pressed_cb(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_dec_button_unpressed_cb(void *data, Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+
 static const Elm_Action key_actions[] = {
{"spin", _key_action_spin},
{"toggle", _key_action_toggle},
{NULL, NULL}
 };
 
+EO_CALLBACKS_ARRAY_DEFINE(_inc_button_cb,
+   { EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _inc_button_clicked_cb},
+   { EVAS_CLICKABLE_INTERFACE_EVENT_PRESSED, _inc_button_pressed_cb},
+   { EVAS_CLICKABLE_INTERFACE_EVENT_UNPRESSED, _inc_button_unpressed_cb},
+   { EVAS_OBJECT_EVENT_MOUSE_MOVE, _inc_dec_button_mouse_move_cb }
+);
+
+EO_CALLBACKS_ARRAY_DEFINE(_dec_button_cb,
+   { EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _dec_button_clicked_cb},
+   { EVAS_CLICKABLE_INTERFACE_EVENT_PRESSED, _dec_button_pressed_cb},
+   { EVAS_CLICKABLE_INTERFACE_EVENT_UNPRESSED, _dec_button_unpressed_cb},
+   { EVAS_OBJECT_EVENT_MOUSE_MOVE, _inc_dec_button_mouse_move_cb }
+);
+
+
 static void _access_increment_decrement_info_say(Evas_Object *obj,
  Eina_Bool is_incremented);
 
@@ -747,10 +784,9 @@ _dec_button_unpressed_cb(void *data,
return EINA_TRUE;
 }
 
-static void
-_inc_dec_button_mouse_move_cb(void *data,
-  Evas *evas EINA_UNUSED,
-  Evas_Object *obj EINA_UNUSED,
+static Eina_Bool
+_inc_dec_button_mouse_move_cb(void *data, Eo *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED,
   void *event_info)
 {
Evas_Event_Mouse_Move *ev = event_info;
@@ -761,6 +797,8 @@ _inc_dec_button_mouse_move_cb(void *data,
 ecore_timer_del(sd->longpress_timer);
 sd->longpress_timer = NULL;
  }
+
+   return EINA_TRUE;
 }
 
 EOLIAN static void
@@ -1039,15 +1077,8 @@ _elm_spinner_evas_object_smart_add(Eo *obj, 
Elm_Spinner_Data *priv)
 priv->inc_button = elm_button_add(obj);
 elm_object_style_set(priv->inc_button, "spinner/increase/default");
 
-eo_do(priv->inc_button, eo_event_callback_add
-  (EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _inc_button_clicked_cb, 
obj));
-eo_do(priv->inc_button, eo_event_callback_add
-  (EVAS_CLICKABLE_INTERFACE_EVENT_PRESSED, _inc_button_pressed_cb, 
obj));
-eo_do(priv->inc_button, eo_event_callback_add
-  (EVAS_CLICKABLE_INTERFACE_EVENT_UNPRESSED, _inc_button_unpressed_cb, 
obj));
-evas_object_event_callback_add
-  (priv->inc_button, EVAS_CALLBACK_MOUSE_MOVE, 
_inc_dec_button_mouse_move_cb, obj);
-
+eo_do(priv->inc_button,
+  eo_event_callback_array_add(_inc_button_cb(), obj));
 
 elm_layout_content_set(obj, "elm.swallow.inc_b

[EGIT] [core/elementary] master 05/06: elc_popup: ported eo_event callbacks to use eo_event arrays

2015-09-23 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=4b160aaacee3238307916918acac9366fcc94d5b

commit 4b160aaacee3238307916918acac9366fcc94d5b
Author: Vivek Ellur 
Date:   Wed Sep 23 15:23:46 2015 -0700

elc_popup: ported eo_event callbacks to use eo_event arrays

Summary:
Changed eo_event_callback_add to eo_event_callback_array_add function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Differential Revision: https://phab.enlightenment.org/D3099

Signed-off-by: Cedric BAIL 
---
 src/lib/elc_popup.c | 24 +++-
 1 file changed, 15 insertions(+), 9 deletions(-)

diff --git a/src/lib/elc_popup.c b/src/lib/elc_popup.c
index 73a1302..a1d7849 100644
--- a/src/lib/elc_popup.c
+++ b/src/lib/elc_popup.c
@@ -44,12 +44,23 @@ static const Evas_Smart_Cb_Description _smart_callbacks[] = 
{
 
 static Eina_Bool _key_action_move(Evas_Object *obj, const char *params);
 static void _parent_geom_cb(void *data, Evas *e EINA_UNUSED, Evas_Object *obj, 
void *event_info EINA_UNUSED);
+static Eina_Bool
+_block_clicked_cb(void *data, Eo *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_timeout_cb(void *data, Eo *obj EINA_UNUSED,
+const Eo_Event_Description *desc EINA_UNUSED, void *event_info 
EINA_UNUSED);
 
 static const Elm_Action key_actions[] = {
{"move", _key_action_move},
{NULL, NULL}
 };
 
+EO_CALLBACKS_ARRAY_DEFINE(_notify_cb,
+   { ELM_NOTIFY_EVENT_BLOCK_CLICKED, _block_clicked_cb },
+   { ELM_NOTIFY_EVENT_TIMEOUT, _timeout_cb }
+);
+
 static void  _on_content_del(void *data, Evas *e, Evas_Object *obj, void 
*event_info);
 
 EOLIAN static Eina_Bool
@@ -199,10 +210,8 @@ _elm_popup_evas_object_smart_del(Eo *obj, Elm_Popup_Data 
*sd)
evas_object_event_callback_del_full(sd->parent, EVAS_CALLBACK_RESIZE, 
_parent_geom_cb, obj);
evas_object_event_callback_del_full(sd->parent, EVAS_CALLBACK_MOVE, 
_parent_geom_cb, obj);
 
-   eo_do(sd->notify, eo_event_callback_del(
- ELM_NOTIFY_EVENT_BLOCK_CLICKED, _block_clicked_cb, obj));
-   eo_do(sd->notify, eo_event_callback_del(
- ELM_NOTIFY_EVENT_TIMEOUT, _timeout_cb, obj));
+   eo_do(sd->notify,
+ eo_event_callback_array_del(_notify_cb(), obj));
evas_object_event_callback_del
  (sd->content, EVAS_CALLBACK_DEL, _on_content_del);
evas_object_event_callback_del(obj, EVAS_CALLBACK_SHOW, _on_show);
@@ -1501,11 +1510,8 @@ _elm_popup_evas_object_smart_add(Eo *obj, Elm_Popup_Data 
*priv)
  _size_hints_changed_cb, priv->main_layout);
 
priv->content_text_wrap_type = ELM_WRAP_MIXED;
-   eo_do(priv->notify, eo_event_callback_add
- (ELM_NOTIFY_EVENT_BLOCK_CLICKED,_block_clicked_cb, obj));
-
-   eo_do(priv->notify, eo_event_callback_add
- (ELM_NOTIFY_EVENT_TIMEOUT, _timeout_cb, obj));
+   eo_do(priv->notify,
+ eo_event_callback_array_add(_notify_cb(), obj));
 
elm_widget_can_focus_set(obj, EINA_TRUE);
elm_widget_can_focus_set(priv->main_layout, EINA_TRUE);

-- 




[EGIT] [core/elementary] master 03/06: elc_multibuttonentry: ported eo_event callbacks to use eo_event callback array

2015-09-23 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=3c01ffd399ea68390475ec28923b8d0b05313a4b

commit 3c01ffd399ea68390475ec28923b8d0b05313a4b
Author: Vivek Ellur 
Date:   Wed Sep 23 15:21:46 2015 -0700

elc_multibuttonentry: ported eo_event callbacks to use eo_event callback 
array

Summary:
Changed eo_event_callback_add to use eo_event_callback_array_add

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Differential Revision: https://phab.enlightenment.org/D3097

Signed-off-by: Cedric BAIL 
---
 src/lib/elc_multibuttonentry.c | 29 +
 1 file changed, 21 insertions(+), 8 deletions(-)

diff --git a/src/lib/elc_multibuttonentry.c b/src/lib/elc_multibuttonentry.c
index f9f3d6c..c094552 100644
--- a/src/lib/elc_multibuttonentry.c
+++ b/src/lib/elc_multibuttonentry.c
@@ -44,6 +44,26 @@ static const Evas_Smart_Cb_Description _smart_callbacks[] = {
{NULL, NULL}
 };
 
+static Eina_Bool
+_entry_changed_cb(void *data, Eo *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_entry_focus_in_cb(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_entry_focus_out_cb(void *data, Eo *obj EINA_UNUSED,
+const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_entry_clicked_cb(void *data, Eo *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+
+EO_CALLBACKS_ARRAY_DEFINE(_multi_buttonentry_cb,
+   { ELM_MULTIBUTTONENTRY_EVENT_CHANGED, _entry_changed_cb },
+   { ELM_WIDGET_EVENT_FOCUSED, _entry_focus_in_cb },
+   { ELM_WIDGET_EVENT_UNFOCUSED, _entry_focus_out_cb },
+   { EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _entry_clicked_cb }
+);
+
 EOLIAN static Eina_Bool
 _elm_multibuttonentry_elm_widget_translate(Eo *obj EINA_UNUSED, 
Elm_Multibuttonentry_Data *sd)
 {
@@ -1160,14 +1180,7 @@ _callbacks_register(Evas_Object *obj)
  (sd->entry, EVAS_CALLBACK_KEY_DOWN, _entry_key_down_cb, obj);
evas_object_event_callback_add
  (sd->entry, EVAS_CALLBACK_RESIZE, _entry_resize_cb, obj);
-   eo_do(sd->entry, eo_event_callback_add
- (ELM_MULTIBUTTONENTRY_EVENT_CHANGED, _entry_changed_cb, obj));
-   eo_do(sd->entry, eo_event_callback_add
- (ELM_WIDGET_EVENT_FOCUSED, _entry_focus_in_cb, obj));
-   eo_do(sd->entry, eo_event_callback_add
- (ELM_WIDGET_EVENT_UNFOCUSED, _entry_focus_out_cb, obj));
-   eo_do(sd->entry, eo_event_callback_add
- (EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _entry_clicked_cb, obj));
+   eo_do(sd->entry, eo_event_callback_array_add(_multi_buttonentry_cb(), obj));
 }
 
 static void

-- 




[EGIT] [core/elementary] master 04/06: elc_player: ported eo_event callbacks to use eo_event arrays

2015-09-23 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=cd659af5231477ddd8b98890247261a4f6d5e9cc

commit cd659af5231477ddd8b98890247261a4f6d5e9cc
Author: Vivek Ellur 
Date:   Wed Sep 23 15:23:11 2015 -0700

elc_player: ported eo_event callbacks to use eo_event arrays

Summary:
Changed eo_event_callback_add to use eo_event_callback_array_add

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Differential Revision: https://phab.enlightenment.org/D3098

Signed-off-by: Cedric BAIL 
---
 src/lib/elc_player.c | 59 
 1 file changed, 41 insertions(+), 18 deletions(-)

diff --git a/src/lib/elc_player.c b/src/lib/elc_player.c
index 1eeb35d..ac4a23f 100644
--- a/src/lib/elc_player.c
+++ b/src/lib/elc_player.c
@@ -57,6 +57,28 @@ static const Evas_Smart_Cb_Description _smart_callbacks[] = {
{ NULL, NULL }
 };
 
+static Eina_Bool
+_update_frame(void *data, Eo *obj,
+  const Eo_Event_Description *desc, void *event_info);
+static Eina_Bool
+_update_slider(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void *event_info 
EINA_UNUSED);
+static Eina_Bool
+_play_started(void *data, Eo *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED, void *event_info 
EINA_UNUSED);
+static Eina_Bool
+_play_finished(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void *event_info 
EINA_UNUSED);
+static Eina_Bool
+_update_position(void *data, Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info EINA_UNUSED);
+static Eina_Bool
+_drag_start(void *data, Eo *obj EINA_UNUSED,
+const Eo_Event_Description *desc EINA_UNUSED, void *event_info 
EINA_UNUSED);
+static Eina_Bool
+_drag_stop(void *data, Eo *obj EINA_UNUSED,
+   const Eo_Event_Description *desc EINA_UNUSED, void *event_info 
EINA_UNUSED);
+
 static Eina_Bool _key_action_move(Evas_Object *obj, const char *params);
 static Eina_Bool _key_action_play(Evas_Object *obj, const char *params);
 
@@ -66,6 +88,21 @@ static const Elm_Action key_actions[] = {
{NULL, NULL}
 };
 
+EO_CALLBACKS_ARRAY_DEFINE(_emotion_cb,
+   { EMOTION_OBJECT_EVENT_FRAME_DECODE, _update_frame },
+   { EMOTION_OBJECT_EVENT_FRAME_RESIZE, _update_slider },
+   { EMOTION_OBJECT_EVENT_LENGTH_CHANGE, _update_slider },
+   { EMOTION_OBJECT_EVENT_POSITION_UPDATE, _update_frame },
+   { EMOTION_OBJECT_EVENT_PLAYBACK_STARTED, _play_started },
+   { EMOTION_OBJECT_EVENT_PLAYBACK_FINISHED, _play_finished }
+);
+
+EO_CALLBACKS_ARRAY_DEFINE(_slider_cb,
+   { ELM_SLIDER_EVENT_CHANGED, _update_position },
+   { ELM_SLIDER_EVENT_SLIDER_DRAG_START, _drag_start },
+   { ELM_SLIDER_EVENT_SLIDER_DRAG_STOP, _drag_stop }
+);
+
 static Eina_Bool
 _key_action_move(Evas_Object *obj, const char *params)
 {
@@ -625,18 +662,8 @@ _elm_player_elm_container_content_set(Eo *obj, 
Elm_Player_Data *sd, const char *
  elm_layout_signal_emit(obj, "elm,player,play", "elm");
else elm_layout_signal_emit(obj, "elm,player,pause", "elm");
 
-   eo_do(sd->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_FRAME_DECODE, _update_frame, obj));
-   eo_do(sd->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_FRAME_RESIZE, _update_slider, obj));
-   eo_do(sd->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_LENGTH_CHANGE, _update_slider, obj));
-   eo_do(sd->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_POSITION_UPDATE, _update_frame, obj));
-   eo_do(sd->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_PLAYBACK_STARTED, _play_started, obj));
-   eo_do(sd->emotion, eo_event_callback_add
- (EMOTION_OBJECT_EVENT_PLAYBACK_FINISHED, _play_finished, obj));
+   eo_do(sd->emotion,
+ eo_event_callback_array_add(_emotion_cb(), obj));
 
/* FIXME: track info from video */
 end:
@@ -683,12 +710,8 @@ _elm_player_evas_object_smart_add(Eo *obj, Elm_Player_Data 
*priv)
  (priv->slider, EVAS_HINT_EXPAND, EVAS_HINT_EXPAND);
elm_layout_content_set(obj, "elm.swallow.media_player.positionslider",
   priv->slider);
-   eo_do(priv->slider, eo_event_callback_add
- (ELM_SLIDER_EVENT_CHANGED, _update_position, obj));
-   eo_do(priv->slider, eo_event_callback_add
- (ELM_SLIDER_EVENT_SLIDER_DRAG_START, _drag_start, obj));
-   eo_do(priv->slider, eo_event_callback_add
- (ELM_SLIDER_EVENT_SLIDER_DRAG_STOP, _drag_stop, obj));
+   eo_do(priv->slider,
+ eo_event_callback_array_add(_slider_cb(), obj));
 
priv->vslider = elm_slider_add(obj);
elm_slider_indicator_show_set(priv->vslider, EINA_FALSE);

-- 




[EGIT] [core/elementary] master 01/01: elementary: Port evas object smart callbacks to use eo event callbacks

2015-08-28 Thread Vivek Ellur
hermet pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=d040f90d15a69f394e17ab19119e541d95bf9c4c

commit d040f90d15a69f394e17ab19119e541d95bf9c4c
Author: Vivek Ellur 
Date:   Fri Aug 28 16:30:27 2015 +0900

elementary: Port evas object smart callbacks to use eo event callbacks

Summary:
Changed evas_object_smart_callback_add APIs to use eo_event_callback_add API

Signed-off-by: Vivek Ellur 

Reviewers: cedric, Hermet

Reviewed By: Hermet

Differential Revision: https://phab.enlightenment.org/D2995
---
 src/modules/prefs/Makefile.am|  1 +
 src/modules/prefs/elm_button.c   |  8 +---
 src/modules/prefs/elm_check.c|  9 +
 src/modules/prefs/elm_datetime.c |  8 +---
 src/modules/prefs/elm_entry.c| 13 -
 src/modules/prefs/elm_slider.c   |  9 +
 src/modules/prefs/elm_spinner.c  |  8 +---
 7 files changed, 34 insertions(+), 22 deletions(-)

diff --git a/src/modules/prefs/Makefile.am b/src/modules/prefs/Makefile.am
index c7b4759..d7280cb 100644
--- a/src/modules/prefs/Makefile.am
+++ b/src/modules/prefs/Makefile.am
@@ -14,6 +14,7 @@ elm_prefs_swallow.edj: Makefile elm_prefs_swallow.edc
$(top_builddir)/src/modules/prefs/elm_prefs_swallow.edj
 
 AM_CPPFLAGS = \
+-DELM_INTERNAL_API_ARGESFSDFEFC=1 \
 -I. \
 -I$(top_builddir) \
 -I$(top_srcdir) \
diff --git a/src/modules/prefs/elm_button.c b/src/modules/prefs/elm_button.c
index 53c03f3..cff42e3 100644
--- a/src/modules/prefs/elm_button.c
+++ b/src/modules/prefs/elm_button.c
@@ -8,14 +8,15 @@ static Elm_Prefs_Item_Type supported_types[] =
ELM_PREFS_TYPE_UNKNOWN
 };
 
-static void
+static Eina_Bool
 _item_changed_cb(void *data,
- Evas_Object *obj,
+ Eo *obj, const Eo_Event_Description *desc EINA_UNUSED,
  void *event_info EINA_UNUSED)
 {
Elm_Prefs_Item_Changed_Cb prefs_it_changed_cb = data;
 
prefs_it_changed_cb(obj);
+   return EINA_TRUE;
 }
 
 static Evas_Object *
@@ -27,7 +28,8 @@ elm_prefs_button_add(const Elm_Prefs_Item_Iface *iface 
EINA_UNUSED,
 {
Evas_Object *obj = elm_button_add(prefs);
 
-   evas_object_smart_callback_add(obj, "clicked", _item_changed_cb, cb);
+   eo_do(obj, eo_event_callback_add
+ (EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _item_changed_cb, cb));
 
return obj;
 }
diff --git a/src/modules/prefs/elm_check.c b/src/modules/prefs/elm_check.c
index 8940c3b..b41243e 100644
--- a/src/modules/prefs/elm_check.c
+++ b/src/modules/prefs/elm_check.c
@@ -6,14 +6,15 @@ static Elm_Prefs_Item_Type supported_types[] =
ELM_PREFS_TYPE_UNKNOWN
 };
 
-static void
+static Eina_Bool
 _item_changed_cb(void *data,
- Evas_Object *obj,
+ Eo *obj, const Eo_Event_Description *desc EINA_UNUSED,
  void *event_info EINA_UNUSED)
 {
 Elm_Prefs_Item_Changed_Cb prefs_it_changed_cb = data;
 
 prefs_it_changed_cb(obj);
+return EINA_TRUE;
 }
 
 static Evas_Object *
@@ -25,8 +26,8 @@ elm_prefs_check_add(const Elm_Prefs_Item_Iface *iface 
EINA_UNUSED,
 {
Evas_Object *obj = elm_check_add(prefs);
 
-   evas_object_smart_callback_add(obj, "changed", _item_changed_cb, cb);
-
+   eo_do(obj, eo_event_callback_add
+ (ELM_CHECK_EVENT_CHANGED, _item_changed_cb, cb));
elm_check_state_set(obj, spec.b.def);
 
return obj;
diff --git a/src/modules/prefs/elm_datetime.c b/src/modules/prefs/elm_datetime.c
index 72c9f79..3637e91 100644
--- a/src/modules/prefs/elm_datetime.c
+++ b/src/modules/prefs/elm_datetime.c
@@ -6,14 +6,15 @@ static Elm_Prefs_Item_Type supported_types[] =
ELM_PREFS_TYPE_UNKNOWN
 };
 
-static void
+static Eina_Bool
 _item_changed_cb(void *data,
- Evas_Object *obj,
+ Eo *obj, const Eo_Event_Description *desc EINA_UNUSED,
  void *event_info EINA_UNUSED)
 {
Elm_Prefs_Item_Changed_Cb prefs_it_changed_cb = data;
 
prefs_it_changed_cb(obj);
+   return EINA_TRUE;
 }
 
 static Evas_Object *
@@ -32,7 +33,8 @@ elm_prefs_datetime_add(const Elm_Prefs_Item_Iface *iface 
EINA_UNUSED,
elm_datetime_field_visible_set(obj, ELM_DATETIME_MINUTE, EINA_FALSE);
elm_datetime_field_visible_set(obj, ELM_DATETIME_AMPM, EINA_FALSE);
 
-   evas_object_smart_callback_add(obj, "changed", _item_changed_cb, cb);
+   eo_do(obj, eo_event_callback_add
+ (ELM_DATETIME_EVENT_CHANGED, _item_changed_cb, cb));
 
t.tm_year = spec.d.min.y - 1900;
t.tm_mon = spec.d.min.m - 1;
diff --git a/src/modules/prefs/elm_entry.c b/src/modules/prefs/elm_entry.c
index e6cf68f..301842d 100644
--- a/src/modules/prefs/elm_entry.c
+++ b/src/modules/prefs/elm_entry.c
@@ -1,4 +1,5 @@
 #include "private.h"
+#include "elm_widget.h"
 #include 
 #include 
 
@@ -93,14 +94,15 @@ mismatch:
return EINA_FALSE;
 }
 
-static void
+static Eina_Bool
 _item_changed_cb(void *data,
- Evas_Object *obj,
+

[EGIT] [core/elementary] master 01/01: elm_colorclass: Changed evas object smart callbacks to use eo event callbacks

2015-08-28 Thread Vivek Ellur
hermet pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=97a3144827ce65296c8a19fef0209b767db497c6

commit 97a3144827ce65296c8a19fef0209b767db497c6
Author: Vivek Ellur 
Date:   Fri Aug 28 16:16:37 2015 +0900

elm_colorclass: Changed evas object smart callbacks to use eo event 
callbacks

Summary:
Replaced evas_object_smart_callback_add with eo_event_callback_add api

Signed-off-by: Vivek Ellur 

Reviewers: cedric, Hermet

Reviewed By: Hermet

Differential Revision: https://phab.enlightenment.org/D2998
---
 src/lib/elm_color_class.c | 34 +++---
 1 file changed, 23 insertions(+), 11 deletions(-)

diff --git a/src/lib/elm_color_class.c b/src/lib/elm_color_class.c
index 9c1b384..aa014d5 100644
--- a/src/lib/elm_color_class.c
+++ b/src/lib/elm_color_class.c
@@ -127,8 +127,10 @@ _colorclass_select(void *data, Evas_Object *obj 
EINA_UNUSED, const char *sig, co
cc->current->color[cc->num].b, 
cc->current->color[cc->num].a);
 }
 
-static void
-_colorclass_changed(void *data, Evas_Object *obj EINA_UNUSED, void *event_info 
EINA_UNUSED)
+static Eina_Bool
+_colorclass_changed(void *data, Evas_Object *obj EINA_UNUSED,
+const Eo_Event_Description *desc EINA_UNUSED,
+void *event_info EINA_UNUSED)
 {
Colorclass_UI *cc = data;
 
@@ -148,15 +150,18 @@ _colorclass_changed(void *data, Evas_Object *obj 
EINA_UNUSED, void *event_info E
_colorclass_cc_update(cc, cc->num);
cc->change_reset = 0;
cc->changed = 1;
+   return EINA_TRUE;
 }
 
-static void
-_colorclass_reset(void *data, Evas_Object *obj EINA_UNUSED, void *event_info 
EINA_UNUSED)
+static Eina_Bool
+_colorclass_reset(void *data, Evas_Object *obj EINA_UNUSED,
+  const Eo_Event_Description *desc EINA_UNUSED,
+  void *event_info EINA_UNUSED)
 {
Colorclass_UI *cc = data;
Colorclass color;
 
-   if (!cc->current) return;
+   if (!cc->current) return EINA_FALSE;
if (cc->winid && remote_iface)
  {
 Eldbus_Message *msg;
@@ -166,7 +171,7 @@ _colorclass_reset(void *data, Evas_Object *obj EINA_UNUSED, 
void *event_info EIN
 eldbus_message_arguments_append(msg, "s", cc->current->name);
 eldbus_service_signal_send(remote_iface, msg);
 cc->change_reset = 1;
-return;
+return EINA_FALSE;
  }
edje_color_class_del(cc->current->name);
edje_color_class_get(cc->current->name,
@@ -185,10 +190,13 @@ _colorclass_reset(void *data, Evas_Object *obj 
EINA_UNUSED, void *event_info EIN
_colorclass_cc_update(cc, 0);
_colorclass_cc_update(cc, 1);
_colorclass_cc_update(cc, 2);
+   return EINA_TRUE;
 }
 
-static void
-_colorclass_activate(void *data, Evas_Object *obj EINA_UNUSED, void 
*event_info)
+static Eina_Bool
+_colorclass_activate(void *data, Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED,
+ void *event_info)
 {
Colorclass_UI *cc = data;
Elm_Object_Item *it = event_info;
@@ -224,6 +232,7 @@ _colorclass_activate(void *data, Evas_Object *obj 
EINA_UNUSED, void *event_info)
elm_colorselector_color_set(cc->cs, cc->current->color[0].r, 
cc->current->color[0].g,
cc->current->color[0].b, 
cc->current->color[0].a);
elm_layout_signal_emit(cc->ly, "elm,colors,show", "elm");
+   return EINA_TRUE;
 }
 
 static void
@@ -730,19 +739,22 @@ elm_color_class_editor_add(Evas_Object *obj, uint64_t 
winid)
elm_scroller_policy_set(gl, ELM_SCROLLER_POLICY_OFF, 
ELM_SCROLLER_POLICY_AUTO);
elm_genlist_mode_set(gl, ELM_LIST_COMPRESS);
elm_object_part_content_set(ly, "elm.swallow.list", gl);
-   evas_object_smart_callback_add(gl, "selected", _colorclass_activate, cc);
+   eo_do(gl, eo_event_callback_add
+ (EVAS_SELECTABLE_INTERFACE_EVENT_SELECTED, _colorclass_activate, cc));
 
cc->reset = bt = elm_button_add(ly);
elm_object_style_set(bt, "colorclass");
/* FIXME: translate */
elm_object_text_set(bt, "Reset");
elm_object_part_content_set(ly, "elm.swallow.reset", bt);
-   evas_object_smart_callback_add(bt, "clicked", _colorclass_reset, cc);
+   eo_do(bt, eo_event_callback_add
+ (EVAS_CLICKABLE_INTERFACE_EVENT_CLICKED, _colorclass_reset, cc));
 
cc->cs = cs = elm_colorselector_add(ly);
elm_colorselector_mode_set(cs, ELM_COLORSELECTOR_COMPONENTS);
elm_object_part_content_set(ly, "elm.swallow.colors", cs);
-   evas_object_smart_callback_add(cs, "changed,user", _colorclass_changed, cc);
+   eo_do(cs, eo_event_callback_add
+ (ELM_COLORSELECTOR_EVENT_CHANGED_USER, _colorclass_changed, cc));
 
EINA_LIST_FREE(ccs, ecc)
  elm_genlist_item_append(gl, &itc, ecc, NULL, 0, NULL, NULL);

-- 




[EGIT] [core/elementary] master 01/01: elm_conform: Port evas smart callbacks to eo

2015-08-28 Thread Vivek Ellur
hermet pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=0b65301d3bbc9e843a45b6c399560ffb85718bc2

commit 0b65301d3bbc9e843a45b6c399560ffb85718bc2
Author: Vivek Ellur 
Date:   Fri Aug 28 16:14:31 2015 +0900

elm_conform: Port evas smart callbacks to eo

Summary:
Changed evas_object_smart_callback_add functions to eo_event_callback_add 
functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric, Hermet

Reviewed By: Hermet

Differential Revision: https://phab.enlightenment.org/D2996
---
 src/lib/elm_conform.c | 20 
 1 file changed, 12 insertions(+), 8 deletions(-)

diff --git a/src/lib/elm_conform.c b/src/lib/elm_conform.c
index 543aa7d..dd32aa4 100644
--- a/src/lib/elm_conform.c
+++ b/src/lib/elm_conform.c
@@ -348,9 +348,10 @@ _land_indicator_connect_cb(void *data)
return ECORE_CALLBACK_RENEW;
 }
 
-static void
+static Eina_Bool
 _land_indicator_disconnected(void *data,
- Evas_Object *obj EINA_UNUSED,
+ Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED,
  void *event_info EINA_UNUSED)
 {
Evas_Object *conform = data;
@@ -359,11 +360,13 @@ _land_indicator_disconnected(void *data,
 
sd->land_indi_timer = ecore_timer_add(ELM_CONFORM_INDICATOR_TIME,
  _land_indicator_connect_cb, conform);
+   return EINA_TRUE;
 }
 
-static void
+static Eina_Bool
 _port_indicator_disconnected(void *data,
- Evas_Object *obj EINA_UNUSED,
+ Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED,
  void *event_info EINA_UNUSED)
 {
Evas_Object *conform = data;
@@ -372,6 +375,7 @@ _port_indicator_disconnected(void *data,
 
sd->port_indi_timer = ecore_timer_add(ELM_CONFORM_INDICATOR_TIME,
  _port_indicator_connect_cb, conform);
+   return EINA_TRUE;
 }
 
 static Evas_Object *
@@ -407,8 +411,8 @@ _create_portrait_indicator(Evas_Object *obj)
  }
 
elm_widget_sub_object_add(obj, port_indicator);
-   evas_object_smart_callback_add(port_indicator, "image.deleted", 
_port_indicator_disconnected, obj);
-
+   eo_do(port_indicator, eo_event_callback_add
+ (ELM_PLUG_EVENT_IMAGE_DELETED, _port_indicator_disconnected, NULL));
evas_object_size_hint_min_set(port_indicator, -1, 0);
evas_object_size_hint_max_set(port_indicator, -1, 0);
 
@@ -448,8 +452,8 @@ _create_landscape_indicator(Evas_Object *obj)
  }
 
elm_widget_sub_object_add(obj, land_indicator);
-   evas_object_smart_callback_add(land_indicator, 
"image.deleted",_land_indicator_disconnected, obj);
-
+   eo_do(land_indicator, eo_event_callback_add
+ (ELM_PLUG_EVENT_IMAGE_DELETED, _land_indicator_disconnected, NULL));
evas_object_size_hint_min_set(land_indicator, -1, 0);
evas_object_size_hint_max_set(land_indicator, -1, 0);
return land_indicator;

-- 




[EGIT] [core/elementary] master 01/01: elementary: Port evas smart callbacks to Eo callbacks

2015-08-28 Thread Vivek Ellur
hermet pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=74981a3dbe01702181331c19fb880274faa2a0a7

commit 74981a3dbe01702181331c19fb880274faa2a0a7
Author: Vivek Ellur 
Date:   Fri Aug 28 15:58:54 2015 +0900

elementary: Port evas smart callbacks to Eo callbacks

Summary:
Changed evas_object_smart_callback_add to eo_event_callback_add functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric, Hermet

Reviewed By: Hermet

Differential Revision: https://phab.enlightenment.org/D2997
---
 .../datetime_input_ctxpopup.c  | 47 ++
 1 file changed, 31 insertions(+), 16 deletions(-)

diff --git a/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c 
b/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c
index 05d59dc..8e2d397 100644
--- a/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c
+++ b/src/modules/datetime_input_ctxpopup/datetime_input_ctxpopup.c
@@ -33,13 +33,16 @@ _diskselector_item_free_cb(void *data, Evas_Object *obj 
EINA_UNUSED, void *event
free(data);
 }
 
-static void
-_ctxpopup_dismissed_cb(void *data, Evas_Object *obj, void *event_info 
EINA_UNUSED)
+static Eina_Bool
+_ctxpopup_dismissed_cb(void *data, Eo *obj, const Eo_Event_Description *desc 
EINA_UNUSED,
+   void *event_info EINA_UNUSED)
 {
Ctxpopup_Module_Data *ctx_mod;
ctx_mod = (Ctxpopup_Module_Data *)data;
evas_object_del(obj);
ctx_mod->ctxpopup = NULL;
+
+   return EINA_TRUE;
 }
 
 static void
@@ -84,15 +87,16 @@ _field_value_get(struct tm *tim, Elm_Datetime_Field_Type  
field_type)
return (*timearr[field_type]);
 }
 
-static void
-_diskselector_cb(void *data EINA_UNUSED, Evas_Object *obj EINA_UNUSED, void 
*event_info)
+static Eina_Bool
+_diskselector_cb(void *data EINA_UNUSED, Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED, void 
*event_info)
 {
DiskItem_Data *disk_data;
struct tm curr_time;
const char *fmt;
 
disk_data = (DiskItem_Data *)elm_object_item_data_get(event_info);
-   if (!disk_data || !(disk_data->ctx_mod)) return;
+   if (!disk_data || !(disk_data->ctx_mod)) return EINA_FALSE;
 
elm_datetime_value_get(disk_data->ctx_mod->mod_data.base, &curr_time);
fmt = 
disk_data->ctx_mod->mod_data.field_format_get(disk_data->ctx_mod->mod_data.base,
 disk_data->sel_field_type);
@@ -102,25 +106,32 @@ _diskselector_cb(void *data EINA_UNUSED, Evas_Object *obj 
EINA_UNUSED, void *eve
_field_value_set(&curr_time, disk_data->sel_field_type, 
disk_data->sel_field_value);
elm_datetime_value_set(disk_data->ctx_mod->mod_data.base, &curr_time);
evas_object_hide(disk_data->ctx_mod->ctxpopup);
+
+   return EINA_TRUE;
 }
 
-static void
-_ampm_clicked_cb(void *data, Evas_Object *obj EINA_UNUSED, void *event_info 
EINA_UNUSED)
+static Eina_Bool
+_ampm_clicked_cb(void *data, Eo *obj EINA_UNUSED,
+ const Eo_Event_Description *desc EINA_UNUSED,
+ void *event_info EINA_UNUSED)
 {
Ctxpopup_Module_Data *ctx_mod;
struct tm curr_time;
 
ctx_mod = (Ctxpopup_Module_Data *)data;
-   if (!ctx_mod) return;
+   if (!ctx_mod) return EINA_FALSE;
 
elm_datetime_value_get(ctx_mod->mod_data.base, &curr_time);
if (curr_time.tm_hour >= 12) curr_time.tm_hour -= 12;
else curr_time.tm_hour += 12;
elm_datetime_value_set(ctx_mod->mod_data.base, &curr_time);
+   return EINA_TRUE;
 }
 
-static void
-_field_clicked_cb(void *data, Evas_Object *obj, void *event_info EINA_UNUSED)
+static Eina_Bool
+_field_clicked_cb(void *data, Eo *obj,
+  const Eo_Event_Description *desc EINA_UNUSED,
+  void *event_info EINA_UNUSED)
 {
Ctxpopup_Module_Data *ctx_mod;
Evas_Object *diskselector;
@@ -136,7 +147,7 @@ _field_clicked_cb(void *data, Evas_Object *obj, void 
*event_info EINA_UNUSED)
Evas_Coord x = 0, y = 0, w = 0, h = 0, width;
 
ctx_mod = (Ctxpopup_Module_Data *)data;
-   if (!ctx_mod) return;
+   if (!ctx_mod) return EINA_FALSE;
 
snprintf(buf, sizeof(buf), "datetime/%s", elm_object_style_get(obj));
 
@@ -146,12 +157,13 @@ _field_clicked_cb(void *data, Evas_Object *obj, void 
*event_info EINA_UNUSED)
evas_object_size_hint_weight_set(ctx_mod->ctxpopup, EVAS_HINT_EXPAND,
 EVAS_HINT_EXPAND);
evas_object_size_hint_align_set(ctx_mod->ctxpopup, EVAS_HINT_FILL, 0.5);
-   evas_object_smart_callback_add(ctx_mod->ctxpopup, "dismissed",
-  _ctxpopup_dismissed_cb, ctx_mod);
+   eo_do(ctx_mod->ctxpopup, eo_event_callback_add
+ (ELM_CTXPOPUP_EVENT_DISMISSED, _ctxpopup_dismissed_cb, ctx_mod));
elm_ctxpopup_hover_parent_set(ctx_mod->ctxpopup, elm_widget_top_get(obj));
 
diskselector = 
elm_diskselector_add(elm_widget_top_g

[EGIT] [core/elementary] master 01/01: elm_entry: Convert eo docs to new format

2015-08-07 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=f218bbb01c1d1671bb3ac92b9d48b5f8c1b89692

commit f218bbb01c1d1671bb3ac92b9d48b5f8c1b89692
Author: Vivek Ellur 
Date:   Fri Aug 7 13:04:27 2015 +0100

elm_entry: Convert eo docs to new format

Summary:
Changed elm_entry.eo docs to the new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2924
---
 src/lib/elm_entry.eo | 1092 +-
 1 file changed, 365 insertions(+), 727 deletions(-)

diff --git a/src/lib/elm_entry.eo b/src/lib/elm_entry.eo
index e15334a..33819e1 100644
--- a/src/lib/elm_entry.eo
+++ b/src/lib/elm_entry.eo
@@ -104,97 +104,74 @@ class Elm.Entry (Elm.Layout, Elm_Interface_Scrollable, 
Evas.Clickable_Interface,
methods {
   @property scrollable {
  set {
-/*@
-Enable or disable scrolling in entry
+[[Enable or disable scrolling in entry
 
-Normally the entry is not scrollable unless you enable it with 
this call.
-
-@ingroup Entry */
+  Normally the entry is not scrollable unless you enable it with 
this call.
+]]
  }
  get {
-/*@
-Get the scrollable state of the entry
-
-Normally the entry is not scrollable. This gets the scrollable 
state
-of the entry. See elm_entry_scrollable_set() for more information.
-
-@return The scrollable state
+[[Get the scrollable state of the entry
 
-@ingroup Entry */
+  Normally the entry is not scrollable. This gets the scrollable 
state
+  of the entry.
+]]
  }
  values {
-scroll: bool; /*@ true if it is to be scrollable, false otherwise 
*/
+scroll: bool; [[$true if it is to be scrollable, $false 
otherwise.]]
  }
   }
   @property input_panel_show_on_demand {
  set {
-/*@
-Set the attribute to show the input panel in case of only an 
user's explicit Mouse Up event.
-It doesn't request to show the input panel even though it has 
focus.
-
-@ingroup Entry */
+[[Set the attribute to show the input panel in case of only an 
user's explicit Mouse Up event.
+  It doesn't request to show the input panel even though it has 
focus.
+]]
  }
  get {
-/*@
-Get the attribute to show the input panel in case of only an 
user's explicit Mouse Up event.
-
-@return $true if the input panel will be shown in case of only 
Mouse up event.
-@since 1.9
+[[Get the attribute to show the input panel in case of only an 
user's explicit Mouse Up event.
 
-@ingroup Entry */
+  @since 1.9
+]]
  }
  values {
-ondemand: bool; /*@ If true, the input panel will be shown in case 
of only Mouse up event.
+ondemand: bool; [[If true, the input panel will be shown in case 
of only Mouse up event.
 (Focus event will be ignored.)
-@since 1.9 */
+
+@since 1.9.]]
  }
   }
   @property context_menu_disabled {
  set {
-/*@
-This disables the entry's contextual (longpress) menu.
-
-@ingroup Entry */
+[[This disables the entry's contextual (longpress) menu.]]
  }
  get {
-/*@
-This returns whether the entry's contextual (longpress) menu is
-disabled.
-
-@return If true, the menu is disabled
-
-@ingroup Entry */
+[[This returns whether the entry's contextual (longpress) menu is
+  disabled.
+]]
  }
  values {
-disabled: bool; /*@ If true, the menu is disabled */
+disabled: bool; [[If $true, the menu is disabled.]]
  }
   }
   @property cnp_mode {
  set {
-/*@
-Control pasting of text and images for the widget.
+[[Control pasting of text and images for the widget.
 
-Normally the entry allows both text and images to be pasted.
-By setting cnp_mode to be #ELM_CNP_MODE_NO_IMAGE, this prevents 
images from being copy or past.
-By setting cnp_mode to be #ELM_CNP_MODE_PLAINTEXT, this remove all 
tags in text .
+  Normally the entry allows both text and images to be pasted.
+  By setting cnp_mode to be #ELM_CNP_MODE_NO_IMAGE, this prevents 
images from being copy or past.
+  By setting cnp_mode to be #ELM_CNP_MODE_PLAINTEXT, this remove 
all tags in text .
 
-   

[EGIT] [core/elementary] master 01/03: elm_fileselector: Converted eo docs of fileselector_entry to new format

2015-07-29 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=a95a7297eabc839a51bce1c027cc63f45b3ab629

commit a95a7297eabc839a51bce1c027cc63f45b3ab629
Author: Vivek Ellur 
Date:   Wed Jul 29 14:44:00 2015 +0100

elm_fileselector: Converted eo docs of fileselector_entry to new format

Summary:
Converted docs of elm_fileselector_entry.eo to the new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2882
---
 src/lib/elm_fileselector_entry.eo | 103 --
 1 file changed, 43 insertions(+), 60 deletions(-)

diff --git a/src/lib/elm_fileselector_entry.eo 
b/src/lib/elm_fileselector_entry.eo
index 065cd84..00c51c2 100644
--- a/src/lib/elm_fileselector_entry.eo
+++ b/src/lib/elm_fileselector_entry.eo
@@ -5,95 +5,78 @@ class Elm.Fileselector_Entry (Elm.Layout, 
Elm_Interface_Fileselector,
methods {
   @property inwin_mode {
  set {
-/*@
-Set whether a given file selector entry widget's internal file
-selector will raise an Elementary "inner window", instead of a
-dedicated Elementary window. By default, it depends on the current
-profile.
-
-@see elm_win_inwin_add() for more information on inner windows
-@see elm_fileselector_entry_inwin_mode_get()
-
-@ingroup File_Selector_Entry */
+[[Set whether a given file selector entry widget's internal file
+  selector will raise an Elementary "inner window", instead of a
+  dedicated Elementary window. By default, it depends on the 
current
+  profile.
+
+  See also \@ref elm_win_inwin_add for more information on inner 
windows,
+  @.inwin_mode.get.
+]]
  }
  get {
-/*@
-Get whether a given file selector entry widget's internal file
-selector will raise an Elementary "inner window", instead of a
-dedicated Elementary window.
-
-@return @c EINA_TRUE if will use an inner window, @c EINA_FALSE
-if it will use a dedicated window
+[[Get whether a given file selector entry widget's internal file
+  selector will raise an Elementary "inner window", instead of a
+  dedicated Elementary window.
 
-@see elm_fileselector_entry_inwin_mode_set() for more details
-
-@ingroup File_Selector_Entry */
+  See also @.inwin_mode.set for more details.
+]]
  }
  values {
-value: bool; /*@ @c EINA_TRUE to make it use an inner window, @c
-EINA_FALSE to make it use a dedicated window */
+value: bool; [[$true to make it use an inner window, $false
+to make it use a dedicated window.]]
  }
   }
   @property window_size {
  set {
-/*@
-Set the size of a given file selector entry widget's window,
-holding the file selector itself.
-
-@note it will only take any effect if the file selector entry
-widget is @b not under "inwin mode". The default size for the
-window (when applicable) is 400x400 pixels.
+[[Set the size of a given file selector entry widget's window,
+  holding the file selector itself.
 
-@see elm_fileselector_entry_window_size_get()
+  Note: it will only take any effect if the file selector entry
+  widget is not under "inwin mode". The default size for the
+  window (when applicable) is 400x400 pixels.
 
-@ingroup File_Selector_Entry */
+  See also @.window_size.get.
+]]
  }
  get {
-/*@
-Get the size of a given file selector entry widget's window,
-holding the file selector itself.
+[[Get the size of a given file selector entry widget's window,
+  holding the file selector itself.
 
-@note Use @c NULL pointers on the size values you're not
-interested in: they'll be ignored by the function.
+  Note: Use $null pointers on the size values you're not
+  interested in: they'll be ignored by the function.
 
-@see elm_fileselector_entry_window_size_set(), for more details
-
-@ingroup File_Selector_Entry */
+  See also @.window_size.set, for more details.
+]]
  }
  values {
-width: Evas.Coord; /*@ The window's width */
-height: Evas.Coord; /*@ The window's height */
+width: Evas.Coord; [[The window'

[EGIT] [core/elementary] master 03/03: elm_frame: Changed eo docs of elm_frame to new format

2015-07-29 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=a043cd62bf4fe0e49adfd01bca78975d56dd70c3

commit a043cd62bf4fe0e49adfd01bca78975d56dd70c3
Author: Vivek Ellur 
Date:   Wed Jul 29 14:44:12 2015 +0100

elm_frame: Changed eo docs of elm_frame to new format

Summary:
Changed docsw of elm_frame.eo according to the new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2885
---
 src/lib/elm_frame.eo | 58 
 1 file changed, 22 insertions(+), 36 deletions(-)

diff --git a/src/lib/elm_frame.eo b/src/lib/elm_frame.eo
index aa95b68..6e24604 100644
--- a/src/lib/elm_frame.eo
+++ b/src/lib/elm_frame.eo
@@ -4,59 +4,45 @@ class Elm.Frame (Elm.Layout, Evas.Clickable_Interface)
methods {
   @property collapse {
  set {
-/*@
-@brief Manually collapse a frame without animations
-Use this to toggle the collapsed state of a frame, bypassing 
animations.
-
-@ingroup Frame */
+[[Manually collapse a frame without animations
+  Use this to toggle the collapsed state of a frame, bypassing 
animations.
+]]
  }
  get {
-/*@
-@brief Determine the collapse state of a frame
-@return true if collapsed, false otherwise
-
-Use this to determine the collapse state of a frame.
-
-@ingroup Frame */
+[[Determine the collapse state of a frame
+  Use this to determine the collapse state of a frame.
+]]
  }
  values {
-collapse: bool; /*@ true to collapse, false to expand */
+collapse: bool; [[$true to collapse, $false to expand.]]
  }
   }
   @property autocollapse {
  set {
-/*@
-@brief Toggle autocollapsing of a frame
-When @p enable is EINA_TRUE, clicking a frame's label will 
collapse the frame
-vertically, shrinking it to the height of the label.
-By default, this is DISABLED.
-
-@ingroup Frame */
+[[Toggle autocollapsing of a frame
+  When $enable is $true, clicking a frame's label will collapse 
the frame
+  vertically, shrinking it to the height of the label.
+  By default, this is DISABLED.
+]]
  }
  get {
-/*@
-@brief Determine autocollapsing of a frame
-@return Whether autocollapse is enabled
+[[Determine autocollapsing of a frame
 
-When this returns EINA_TRUE, clicking a frame's label will 
collapse the frame
-vertically, shrinking it to the height of the label.
-By default, this is DISABLED.
-
-@ingroup Frame */
+  When this returns $true, clicking a frame's label will collapse 
the frame
+  vertically, shrinking it to the height of the label.
+  By default, this is DISABLED.
+]]
  }
  values {
-autocollapse: bool; /*@ Whether to enable autocollapse */
+autocollapse: bool; [[Whether to enable autocollapse.]]
  }
   }
   collapse_go {
- /*@
- @brief Manually collapse a frame with animations
- Use this to toggle the collapsed state of a frame, triggering 
animations.
-
- @ingroup Frame */
-
+ [[Manually collapse a frame with animations
+   Use this to toggle the collapsed state of a frame, triggering 
animations.
+ ]]
  params {
-@in collapse: bool; /*@ true to collapse, false to expand */
+@in collapse: bool; [[$true to collapse, $false to expand.]]
  }
   }
}

-- 




[EGIT] [core/elementary] master 02/03: elm_flipselector: convert docs of elm_flipselector.eo to new format

2015-07-29 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=0882c1ecb61bebb100c567d254de50b2585eee1f

commit 0882c1ecb61bebb100c567d254de50b2585eee1f
Author: Vivek Ellur 
Date:   Wed Jul 29 14:44:07 2015 +0100

elm_flipselector: convert docs of elm_flipselector.eo to new format

Summary:
Changed docs of elm_flipselector.eo and elm_flipselector_item.eo to new 
format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2884
---
 src/lib/elm_flipselector.eo  | 221 ---
 src/lib/elm_flipselector_item.eo |  59 ---
 2 files changed, 113 insertions(+), 167 deletions(-)

diff --git a/src/lib/elm_flipselector.eo b/src/lib/elm_flipselector.eo
index 00598ed..4260e3f 100644
--- a/src/lib/elm_flipselector.eo
+++ b/src/lib/elm_flipselector.eo
@@ -5,190 +5,153 @@ class Elm.Flipselector (Elm.Layout, 
Elm_Interface_Atspi_Widget_Action,
methods {
   @property first_interval {
  set {
-/*@
-Set the interval on time updates for a user mouse button hold
-on a flip selector widget.
+[[Set the interval on time updates for a user mouse button hold
+  on a flip selector widget.
 
-This interval value is @b decreased while the user holds the
-mouse pointer either flipping up or flipping down a given flip
-selector.
+  This interval value is decreased while the user holds the
+  mouse pointer either flipping up or flipping down a given flip
+  selector.
 
-This helps the user to get to a given item distant from the
-current one easier/faster, as it will start to flip quicker and
-quicker on mouse button holds.
+  This helps the user to get to a given item distant from the
+  current one easier/faster, as it will start to flip quicker and
+  quicker on mouse button holds.
 
-The calculation for the next flip interval value, starting from
-the one set with this call, is the previous interval divided by
-1.05, so it decreases a little bit.
+  The calculation for the next flip interval value, starting from
+  the one set with this call, is the previous interval divided by
+  1.05, so it decreases a little bit.
 
-The default starting interval value for automatic flips is
-@b 0.85 seconds.
+  The default starting interval value for automatic flips is
+  0.85 seconds.
 
-@see elm_flipselector_first_interval_get()
-
-@ingroup Flipselector */
+  See also @.first_interval.get.
+]]
  }
  get {
-/*@
-Get the interval on time updates for an user mouse button hold
-on a flip selector widget.
-
-@return The (first) interval value, in seconds, set on it
-
-@see elm_flipselector_first_interval_set() for more details
+[[Get the interval on time updates for an user mouse button hold
+  on a flip selector widget.
 
-@ingroup Flipselector */
+  See also @.first_interval.set for more details.
+]]
  }
  values {
-interval: double; /*@ The (first) interval value in seconds */
+interval: double; [[The (first) interval value in seconds.]]
  }
   }
   @property items {
  get {
-/*@
-Get the internal list of items in a given flip selector widget.
-
-@return The list of items (#Elm_Object_Item as data) or
-@c NULL on errors.
-
-This list is @b not to be modified in any way and must not be
-freed. Use the list members with functions like
-elm_object_item_text_set(),
-elm_object_item_text_get(),
-elm_object_item_del(),
-elm_flipselector_item_selected_get(),
-elm_flipselector_item_selected_set().
-
-@warning This list is only valid until @p obj object's internal
-items list is changed. It should be fetched again with another
-call to this function when changes happen.
-
-@ingroup Flipselector */
+[[Get the internal list of items in a given flip selector widget.
+
+  This list is not to be modified in any way and must not be
+  freed. Use the list members with functions like
+  \@ref elm_object_item_text_set,
+  \@ref elm_object_item_text_get,
+  \@ref elm_object_item_del,
+  \@ref elm_flipselector_item_selected_get,
+  \@ref elm_flipselector_item_selected_set.
+
+  Warning: This list is only 

[EGIT] [core/efl] master 04/09: eina_quaternion: add test cases for various eina quaternion operations

2015-07-27 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=df50704764e0fd75b4be70c36a85611d85446bff

commit df50704764e0fd75b4be70c36a85611d85446bff
Author: Vivek Ellur 
Date:   Tue Jul 28 00:55:56 2015 +0200

eina_quaternion: add test cases for various eina quaternion operations

Summary:
Added test cases for scale, multiply, normalize,dot operations for 
quaternion
Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2790

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_quaternion.c | 79 +++
 1 file changed, 79 insertions(+)

diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index 78d1e2e..2e47580 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -180,6 +180,80 @@ START_TEST(eina_test_quaternion_f16p16)
 }
 END_TEST
 
+START_TEST(eina_test_quaternion_dot)
+{
+   Eina_Quaternion q = {1, 3, 4, 5};
+   Eina_Quaternion r = {7, 9, 5, 1};
+   double res;
+
+   eina_init();
+
+   res = eina_quaternion_dot(&q, &r);
+   fail_if(res != 59);
+
+   eina_shutdown();
+}
+END_TEST
+
+START_TEST(eina_test_quaternion_scale)
+{
+   Eina_Quaternion q = {1, 3, 4, 5};
+   double scale = 2;
+   Eina_Quaternion r, res = {2, 6, 8, 10};
+
+   eina_init();
+
+   eina_quaternion_scale(&r, &q, scale);
+   fail_if(!eina_quaternion_cmp(&r, &res));
+
+   eina_shutdown();
+}
+END_TEST
+
+START_TEST(eina_test_quaternion_set)
+{
+   Eina_Quaternion q, r = {1, 3, 4, 5};
+
+   eina_init();
+
+   eina_quaternion_set(&q, 1, 3, 4, 5);
+
+   fail_if(!eina_quaternion_cmp(&q, &r));
+
+   eina_shutdown();
+}
+END_TEST
+
+START_TEST(eina_test_quaternion_mul)
+{
+   Eina_Quaternion p = {1, 3, 4, 5};
+   Eina_Quaternion q = {7, 9, 5, 1};
+   Eina_Quaternion r, res = {15, 71, 17, -49};
+
+   eina_init();
+
+   eina_quaternion_mul(&r, &p, &q);
+   fprintf(stderr, "%f %f %f %f\n", res.w, res.x, res.y, res.z);
+   fail_if(!eina_quaternion_cmp(&r, &res));
+
+   eina_shutdown();
+}
+END_TEST
+
+START_TEST(eina_test_quaternion_normalized)
+{
+   Eina_Quaternion p = {1, 1, 1, 1};
+   Eina_Quaternion res, q = {0.5, 0.5, 0.5, 0.5};
+
+   eina_init();
+
+   eina_quaternion_normalized(&res, &p);
+   fail_if(!eina_quaternion_cmp(&q, &res));
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_quaternion(TCase *tc)
 {
@@ -188,4 +262,9 @@ eina_test_quaternion(TCase *tc)
tcase_add_test(tc, eina_test_quaternion_matrix);
tcase_add_test(tc, eina_test_quaternion_op);
tcase_add_test(tc, eina_test_quaternion_f16p16);
+   tcase_add_test(tc, eina_test_quaternion_dot);
+   tcase_add_test(tc, eina_test_quaternion_scale);
+   tcase_add_test(tc, eina_test_quaternion_set);
+   tcase_add_test(tc, eina_test_quaternion_mul);
+   tcase_add_test(tc, eina_test_quaternion_normalized);
 }

-- 




[EGIT] [core/efl] master 05/09: eina_quaternion: add forgotten implementation of converting eina_matrix3 to eina_quaternion

2015-07-27 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=f3768834c52d37f1cc13d4293e3322180ff5d6d7

commit f3768834c52d37f1cc13d4293e3322180ff5d6d7
Author: Vivek Ellur 
Date:   Tue Jul 28 01:00:34 2015 +0200

eina_quaternion: add forgotten implementation of converting eina_matrix3 to 
eina_quaternion

Summary:
Implemenation of eina_matrix3_quaternion_get function

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2786

This was a function I forgot to finish implement. Thanks Vivek to take care 
of it.
As it comes with a test case, is self contained and fix a missing bit of 
code I will
push it at this point in time of our release process. Sorry everyone for 
that late push.

Signed-off-by: Cedric BAIL 
---
 src/lib/eina/eina_quaternion.c| 50 +--
 src/tests/eina/eina_test_quaternion.c |  5 
 2 files changed, 53 insertions(+), 2 deletions(-)

diff --git a/src/lib/eina/eina_quaternion.c b/src/lib/eina/eina_quaternion.c
index 8c751ac..0334d57 100644
--- a/src/lib/eina/eina_quaternion.c
+++ b/src/lib/eina/eina_quaternion.c
@@ -624,7 +624,53 @@ eina_quaternion_rotation_matrix3_get(Eina_Matrix3 *m,
 }
 
 EAPI void
-eina_matrix3_quaternion_get(Eina_Quaternion *q EINA_UNUSED,
-const Eina_Matrix3 *m EINA_UNUSED)
+eina_matrix3_quaternion_get(Eina_Quaternion *q,
+const Eina_Matrix3 *m)
 {
+   double tval;
+   double w, x, y, z;
+
+   tval = m->xx + m->yy + m->zz;
+
+   if (tval > 0)
+ {
+
+double s = 0.5 / sqrtf(tval + 1.0);
+
+w = (0.25 / s);
+x = ((m->zy - m->yz) * s);
+y = ((m->xz - m->zx) * s);
+z = ((m->yx - m->xy) * s);
+ }
+   else if ((m->xx > m->yy) && (m->xx > m->zz))
+ {
+double s = 2.0 * sqrtf(1.0 + m->xx - m->yy - m->zz);
+
+w = ((m->zy - m->yz) / s);
+x = (0.25 * s);
+y = ((m->xy + m->yx) / s);
+z = ((m->xz + m->zx) / s);
+ }
+   else if (m->yy > m->zz)
+ {
+double s = 2.0 * sqrtf(1.0 + m->yy - m->xx - m->zz);
+
+w = ((m->xz - m->zx) / s);
+x = ((m->xy + m->yx) / s);
+y = (0.25 * s);
+z = ((m->yz + m->zy) / s);
+ }
+   else
+ {
+double s = 2.0 * sqrtf(1.0 + m->zz - m->xx - m->yy);
+
+w = ((m->yx - m->xy) / s);
+x = ((m->xz + m->zx) / s);
+y = ((m->yz + m->zy) / s);
+z = (0.25 * s);
+ }
+   q->w = w;
+   q->x = x;
+   q->y = y;
+   q->z = z;
 }
diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index 2e47580..1ffbc1a 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -90,6 +90,8 @@ END_TEST
 START_TEST(eina_test_quaternion_matrix)
 {
Eina_Quaternion q = { 7, 9, 5, 1 };
+   Eina_Quaternion q1 = {7, 9, 5, -1 };
+   Eina_Quaternion tq;
Eina_Matrix3 m = {
  -211, 136, 52,
  116, -147, 104,
@@ -102,6 +104,9 @@ START_TEST(eina_test_quaternion_matrix)
eina_quaternion_rotation_matrix3_get(&tm, &q);
fail_if(!eina_matrix3_cmp(&tm, &m));
 
+   eina_matrix3_quaternion_get(&tq, &m);
+   fail_if(!eina_quaternion_cmp(&tq, &q) && !eina_quaternion_cmp(&tq, &q1));
+
eina_shutdown();
 }
 END_TEST

-- 




[EGIT] [core/efl] master 03/09: eina_quaternion: add test cases for eina f16p16 quaternion functions

2015-07-27 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=309962ed8f8bcc9f73d1a28b9a46cb3dca78cc27

commit 309962ed8f8bcc9f73d1a28b9a46cb3dca78cc27
Author: Vivek Ellur 
Date:   Tue Jul 28 00:48:25 2015 +0200

eina_quaternion: add test cases for eina f16p16 quaternion functions

Summary:
Added test cases for various operations on eina f16p16 quaternions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2792

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_quaternion.c | 58 +++
 1 file changed, 58 insertions(+)

diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index dd704d5..78d1e2e 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -123,6 +123,63 @@ START_TEST(eina_test_quaternion_op)
 }
 END_TEST
 
+START_TEST(eina_test_quaternion_f16p16)
+{
+   Eina_F16p16 x = 65536, y = 65536;
+   Eina_F16p16 z = 65536, w = 65536;
+   Eina_F16p16 res;
+   Eina_Quaternion p, q = {65536, 65536, 65536, 65536};
+   Eina_Quaternion_F16p16 t, s, r = {65536, 65536, 65536, 65536};
+
+
+   eina_init();
+
+   eina_quaternion_f16p16_set(&p, x, y, z, w);
+   fail_if(!eina_quaternion_cmp(&p, &q));
+
+   res = eina_quaternion_f16p16_norm(&r);
+   fail_if(res != 131072);
+
+   eina_quaternion_f16p16_negative(&s, &r);
+   fail_if(s.x != s.y ||
+   s.y != s.z ||
+   s.z != s.w ||
+   s.w != -65536);
+
+   eina_quaternion_f16p16_add(&t, &s, &r);
+   fail_if(t.x != t.y ||
+   t.y != t.z ||
+   t.z != t.w ||
+   t.w != 0);
+
+   res = 1;
+   eina_quaternion_f16p16_scale(&t, &r, res);
+   fail_if(t.x != t.y ||
+   t.y != t.z ||
+   t.z != t.w ||
+   t.w != 65536);
+
+   eina_quaternion_f16p16_conjugate(&t, &r);
+   fail_if(t.x != -65536 ||
+   t.y != -65536 ||
+   t.z != -65536 ||
+   t.w != 65536);
+
+   s.x = 65536;
+   s.y = 65536;
+   s.z = 65536;
+   s.w = 65536;
+   res = eina_quaternion_f16p16_dot(&s, &r);
+   fail_if(res != 262144);
+
+   eina_quaternion_f16p16_mul(&t, &s, &r);
+   fail_if(t.x != 131072 ||
+   t.y != 131072 ||
+   t.z != 131072 ||
+   t.w != 0);
+}
+END_TEST
+
 void
 eina_test_quaternion(TCase *tc)
 {
@@ -130,4 +187,5 @@ eina_test_quaternion(TCase *tc)
tcase_add_test(tc, eina_test_quaternion_conjugate);
tcase_add_test(tc, eina_test_quaternion_matrix);
tcase_add_test(tc, eina_test_quaternion_op);
+   tcase_add_test(tc, eina_test_quaternion_f16p16);
 }

-- 




[EGIT] [core/elementary] master 02/02: elm_diskselector: convert eo docs of diskselector to new format

2015-07-27 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=217971caccb381dc7de01fe4f3cca83f1885ff87

commit 217971caccb381dc7de01fe4f3cca83f1885ff87
Author: Vivek Ellur 
Date:   Mon Jul 27 10:18:43 2015 +0100

elm_diskselector: convert eo docs of diskselector to new format

Summary:
Converted docs of elm_diskselector.eo and elm_diskselector_item.eo to the 
new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2864
---
 src/lib/elm_diskselector.eo  | 281 +--
 src/lib/elm_diskselector_item.eo |  74 +--
 2 files changed, 151 insertions(+), 204 deletions(-)

diff --git a/src/lib/elm_diskselector.eo b/src/lib/elm_diskselector.eo
index 9c1a55f..d0d4d51 100644
--- a/src/lib/elm_diskselector.eo
+++ b/src/lib/elm_diskselector.eo
@@ -7,230 +7,189 @@ class Elm.Diskselector (Elm.Widget, 
Elm_Interface_Scrollable,
methods {
   @property side_text_max_length {
  set {
-/*@
-Set the side labels max length.
+[[Set the side labels max length.
 
-Length is the number of characters of items' label that will be
-visible when it's set on side positions. It will just crop
-the string after defined size. E.g.:
+  Length is the number of characters of items' label that will be
+  visible when it's set on side positions. It will just crop
+  the string after defined size. E.g.:
 
-An item with label "January" would be displayed on side position as
-"Jan" if max length is set to 3, or "Janu", if this property
-is set to 4.
+  An item with label "January" would be displayed on side position 
as
+  "Jan" if max length is set to 3, or "Janu", if this property
+  is set to 4.
 
-When it's selected, the entire label will be displayed, except for
-width restrictions. In this case label will be cropped and "..."
-will be concatenated.
+  When it's selected, the entire label will be displayed, except 
for
+  width restrictions. In this case label will be cropped and "..."
+  will be concatenated.
 
-Default side label max length is 3.
+  Default side label max length is 3.
 
-This property will be applied over all items, included before or
-later this function call.
-
-@ingroup Diskselector */
+  This property will be applied over all items, included before or
+  later this function call.
+]]
  }
  get {
-/*@
-Get the side labels max length.
-
-@see elm_diskselector_side_text_max_length_set() for details.
+[[Get the side labels max length.
 
-@return The max length defined for side labels, or 0 if not a valid
-diskselector.
-
-@ingroup Diskselector */
+  See also @.side_text_max_length.set for details.
+]]
  }
  values {
-len: int; /*@ The max length defined for side labels. */
+len: int; [[The max length defined for side labels.]]
  }
   }
   @property round_enabled {
  set {
-/*@
-Enable or disable round mode.
-
-Disabled by default. If round mode is enabled the items list will
-work like a circular list, so when the user reaches the last item,
-the first one will popup.
+[[Enable or disable round mode.
 
-@see elm_diskselector_round_enabled_get()
+  Disabled by default. If round mode is enabled the items list will
+  work like a circular list, so when the user reaches the last 
item,
+  the first one will popup.
 
-@ingroup Diskselector */
+  See also @.round_enabled.get.
+]]
  }
  get {
-/*@
-Get a value whether round mode is enabled or not.
+[[Get a value whether round mode is enabled or not.
 
-@see elm_diskselector_round_enabled_set() for details.
-
-@return @c EINA_TRUE means round mode is enabled. @c EINA_FALSE 
indicates
-it's disabled. If @p obj is @c NULL, @c EINA_FALSE is returned.
-
-@ingroup Diskselector */
+  See also @.round_enabled.set for details.
+]]
  }
  values {
-enabled: bool; /*@ @c EINA_TRUE to enable round mode or @c 
EINA_FALSE to
-disable it. */
+enabled: bool; [[$true to enable round 

[EGIT] [core/elementary] master 01/02: elm_ctxpopup: convert eo docs to new format

2015-07-27 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=110fb9499fb0097a181d55960f8bac642850fe19

commit 110fb9499fb0097a181d55960f8bac642850fe19
Author: Vivek Ellur 
Date:   Mon Jul 27 10:18:37 2015 +0100

elm_ctxpopup: convert eo docs to new format

Summary:
Converted docs of elm_ctxpopup.eo and elm_ctxpopup_item.eo to new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2862
---
 src/lib/elm_ctxpopup.eo  | 240 ++-
 src/lib/elm_ctxpopup_item.eo |  30 +++---
 2 files changed, 112 insertions(+), 158 deletions(-)

diff --git a/src/lib/elm_ctxpopup.eo b/src/lib/elm_ctxpopup.eo
index 916272c..bd89305 100644
--- a/src/lib/elm_ctxpopup.eo
+++ b/src/lib/elm_ctxpopup.eo
@@ -14,241 +14,201 @@ class Elm.Ctxpopup (Elm.Layout, 
Elm_Interface_Atspi_Widget_Action)
methods {
   @property horizontal {
  set {
-/*@
-@brief Change the ctxpopup's orientation to horizontal or vertical.
-
-@ingroup Ctxpopup */
+[[Change the ctxpopup's orientation to horizontal or vertical.]]
  }
  get {
-/*@
-@brief Get the value of current ctxpopup object's orientation.
-
-@return @c EINA_TRUE for horizontal mode, @c EINA_FALSE for 
vertical mode (or errors)
+[[Get the value of current ctxpopup object's orientation.
 
-@see elm_ctxpopup_horizontal_set()
-
-@ingroup Ctxpopup */
+  See also @.horizontal.set.
+]]
  }
  values {
-horizontal: bool; /*@ @c EINA_TRUE for horizontal mode, @c 
EINA_FALSE for vertical */
+horizontal: bool; [[$true for horizontal mode, $false for 
vertical.]]
  }
   }
   @property auto_hide_disabled {
  set {
-/*@
-@brief Set ctxpopup auto hide mode triggered by ctxpopup policy.
-@since 1.9
+[[Set ctxpopup auto hide mode triggered by ctxpopup policy.
 
-Use this function when user wants ctxpopup not to hide 
automatically.
-By default, ctxpopup is dismissed whenever mouse clicked its 
background area, language is changed,
-and its parent geometry is updated(changed).
-Not to hide ctxpopup automatically, disable auto hide function by 
calling this API,
-then ctxpopup won't be dismissed in those scenarios.
+  Use this function when user wants ctxpopup not to hide 
automatically.
+  By default, ctxpopup is dismissed whenever mouse clicked its 
background area, language is changed,
+  and its parent geometry is updated(changed).
+  Not to hide ctxpopup automatically, disable auto hide function 
by calling this API,
+  then ctxpopup won't be dismissed in those scenarios.
 
-Default value of disabled is @c EINA_FALSE.
+  Default value of disabled is $false.
 
-@see elm_ctxpopup_auto_hide_disabled_get()
+  See also @.auto_hide_disabled.get.
 
-@ingroup Ctxpopup */
+  @since 1.9
+]]
  }
  get {
-/*@
-@brief Get ctxpopup auto hide mode triggered by ctxpopup policy.
-@since 1.9
-
-@return auto hide mode's state of a ctxpopup
+[[Get ctxpopup auto hide mode triggered by ctxpopup policy.
 
-@see elm_ctxpopup_auto_hide_disabled_set() for more information.
+  See also @.auto_hide_disabled.set for more information.
 
-@ingroup Ctxpopup */
+  @since 1.9
+]]
  }
  values {
-disabled: bool; /*@ auto hide enable/disable. */
+disabled: bool; [[auto hide enable/disable.]]
  }
   }
   @property hover_parent {
  set {
-/*@
-@brief Set the Ctxpopup's parent
+[[Set the Ctxpopup's parent
 
-Set the parent object.
+  Set the parent object.
 
-@note elm_ctxpopup_add() will automatically call this function
-with its @c parent argument.
+  Note: \@ref elm_ctxpopup_add will automatically call this 
function
+  with its $parent argument.
 
-@see elm_ctxpopup_add()
-@see elm_hover_parent_set()
-
-@ingroup Ctxpopup */
+  See also \@ref elm_ctxpopup_add,
+  \@ref elm_hover_parent_set.
+]]
  }
  get {
-/*@
-@brief Get the Ctxpopup's parent
-
-@see elm_ctxpopup_hover_parent_set() for more information
+[[Get the Ctxpopup's parent
 
-@ingroup Ctxpopup

[EGIT] [core/elementary] master 01/01: elm_dayselector: converted eo docs of dayselector to new format

2015-07-24 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=5c14fa2f3197a5867de1b42ce697c897c5fe13ce

commit 5c14fa2f3197a5867de1b42ce697c897c5fe13ce
Author: Vivek Ellur 
Date:   Fri Jul 24 13:43:36 2015 +0100

elm_dayselector: converted eo docs of dayselector to new format

Summary:
Converted docs of elm_dayselector.eo to the new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2863
---
 src/lib/elm_dayselector.eo | 173 +++--
 1 file changed, 72 insertions(+), 101 deletions(-)

diff --git a/src/lib/elm_dayselector.eo b/src/lib/elm_dayselector.eo
index 5cc6ea9..457cb7a 100644
--- a/src/lib/elm_dayselector.eo
+++ b/src/lib/elm_dayselector.eo
@@ -23,154 +23,125 @@ class Elm.Dayselector (Elm.Layout)
methods {
   @property week_start {
  set {
-/*@
-Set the starting day of Dayselector.
+[[Set the starting day of Dayselector.
 
-@see Elm_Dayselector_Day
-@see elm_dayselector_week_start_get()
-
-@ingroup Dayselector */
+  See also @Elm.Dayselector.Day,
+  @.week_start.get.
+]]
  }
  get {
-/*@
-Get the starting day of Dayselector.
-
-@return Day from where Dayselector displays all the weekdays in 
order.
-
-@see Elm_Dayselector_Day
-@see elm_dayselector_week_start_set()
+[[Get the starting day of Dayselector.
 
-@ingroup Dayselector */
+  See also @Elm.Dayselector.Day,
+  @.week_start.set.
+]]
  }
  values {
-day: Elm.Dayselector.Day; /*@ Dayselector_Day the first day that 
the user wants to display. */
+day: Elm.Dayselector.Day; [[Dayselector_Day the first day that the 
user wants to display.]]
  }
   }
   @property weekend_length {
  set {
-/*@
-Set the weekend length of Dayselector.
+[[Set the weekend length of Dayselector.
 
-@see elm_dayselector_weekend_length_get()
-
-@ingroup Dayselector */
+  See also @.weekend_length.get.
+]]
  }
  get {
-/*@
-Get the weekend length of Dayselector.
-
-@return Number of days marked as a weekend.
-
-@see Elm_Dayselector_Day
-@see elm_dayselector_weekend_length_set()
+[[Get the weekend length of Dayselector.
 
-@ingroup Dayselector */
+  See also @Elm.Dayselector.Day,
+  @.weekend_length.set.
+]]
  }
  values {
-length: uint; /*@ Weekend length, number of days as an integer. */
+length: uint; [[Weekend length, number of days as an integer.]]
  }
   }
   @property weekend_start {
  set {
-/*@
-Set the weekend starting day of Dayselector.
+[[Set the weekend starting day of Dayselector.
 
-@see Elm_Dayselector_Day
-@see elm_dayselector_weekend_start_get()
-
-@ingroup Dayselector */
+  See also @Elm.Dayselector.Day,
+  @.weekend_start.get.
+]]
  }
  get {
-/*@
-Get the weekend starting day of Dayselector.
-
-@return Elm.Dayselector.Day Day from where weekend starts.
-
-@see Elm_Dayselector_Day
-@see elm_dayselector_weekend_start_set()
+[[Get the weekend starting day of Dayselector.
 
-@ingroup Dayselector */
+  See also @Elm.Dayselector.Day,
+  @.weekend_start.set.
+]]
  }
  values {
-day: Elm.Dayselector.Day; /*@ Dayselector_Day the first day from 
where weekend starts. */
+day: Elm.Dayselector.Day; [[Dayselector_Day the first day from 
where weekend starts.]]
  }
   }
   weekdays_names_set {
- /*@
- Set weekdays names to be displayed by the Dayselector.
-
- By default or if @a weekdays is @c NULL, weekdays abbreviations get 
from system are displayed:
- E.g. for an en_US locale: "Sun, Mon, Tue, Wed, Thu, Fri, Sat"
-
- The first string should be related to Sunday, the second to Monday...
-
- The usage should be like this:
- @code
- const char *weekdays[] =
- {
- "Sunday", "Monday", "Tuesday", "Wednesday",
- "Thursday", "Friday", "Saturday"
- };
- elm_dayselector_wekdays_names_set(calendar, weekdays);
- @endcode
-
- @see elm_dayselector_weekdays_name_get()
- @see

[EGIT] [core/elementary] master 01/01: elm_container: Updated eo docs for elm_container

2015-07-22 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=e921412f2ae41b2bfe856aefd4175ab5889a7e17

commit e921412f2ae41b2bfe856aefd4175ab5889a7e17
Author: Vivek Ellur 
Date:   Wed Jul 22 10:21:24 2015 +0100

elm_container: Updated eo docs for elm_container

Summary:
Updated docs of elm_container.eo to the new format

Signed-off-by: Vivek Ellur 

Reviewers: q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2852
---
 src/lib/elm_container.eo | 16 
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/src/lib/elm_container.eo b/src/lib/elm_container.eo
index b9aa60f..82463a0 100644
--- a/src/lib/elm_container.eo
+++ b/src/lib/elm_container.eo
@@ -6,29 +6,29 @@ class Elm.Container (Elm.Widget)
methods {
   @property content_swallow_list {
  get {
-/*@ Get the list of swallow parts in the object. */
+[[Get the list of swallow parts in the object.]]
  }
  values {
-ret: free(own(list *), eina_list_free); /*@ list */
+ret: free(own(list *), eina_list_free); [[list]]
  }
   }
   content_set {
- /*@ Swallow the given object into the given part of the container. */
+ [[Swallow the given object into the given part of the container.]]
  params {
-@in name: const(char)*; /*@ the part in which to swallow the 
object */
-@in content: Evas.Object *; /*@ the object to swallow */
+@in name: const(char)*; [[the part in which to swallow the object]]
+@in content: Evas.Object *; [[the object to swallow.]]
  }
  return: bool;
   }
   content_get {
- /*@ Get the object swallowed in the given part of the container. */
+ [[Get the object swallowed in the given part of the container.]]
  params {
-@in name: const(char)* @nullable; /*@ the part in which the object 
is swallowed */
+@in name: const(char)* @nullable; [[the part in which the object 
is swallowed.]]
  }
  return: Evas.Object *;
   }
   content_unset {
- /*@ Unswallow the object in the given part of the container and 
return it. */
+ [[Unswallow the object in the given part of the container and return 
it.]]
  params {
 @in name: const(char)* @nullable;
  }

-- 




[EGIT] [core/elementary] master 01/01: elm_clock: convert elm_clock eo docs to new format

2015-07-22 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=8edb1d7254ef751566485d98714c7a167f0a4f75

commit 8edb1d7254ef751566485d98714c7a167f0a4f75
Author: Vivek Ellur 
Date:   Wed Jul 22 10:20:35 2015 +0100

elm_clock: convert elm_clock eo docs to new format

Summary:
Converted docs of elm_clock.eo to the new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2849
---
 src/lib/elm_clock.eo | 253 +--
 1 file changed, 105 insertions(+), 148 deletions(-)

diff --git a/src/lib/elm_clock.eo b/src/lib/elm_clock.eo
index 04f8224..7809538 100644
--- a/src/lib/elm_clock.eo
+++ b/src/lib/elm_clock.eo
@@ -24,239 +24,196 @@ class Elm.Clock (Elm.Layout)
methods {
   @property show_am_pm {
  set {
-/*@
-Set if the given clock widget must show hours in military or
-am/pm mode
+[[Set if the given clock widget must show hours in military or
+  am/pm mode
 
-This function sets if the clock must show hours in military or
-am/pm mode. In some countries like Brazil the military mode
-(00-24h-format) is used, in opposition to the USA, where the
-am/pm mode is more commonly used.
+  This function sets if the clock must show hours in military or
+  am/pm mode. In some countries like Brazil the military mode
+  (00-24h-format) is used, in opposition to the USA, where the
+  am/pm mode is more commonly used.
 
-@see elm_clock_show_am_pm_get()
-
-@ingroup Clock */
+  See also @.show_am_pm.get.
+]]
  }
  get {
-/*@
-Get if the given clock widget shows hours in military or am/pm
-mode
-
-@return @c EINA_TRUE, if in am/pm mode, @c EINA_FALSE if in
-military
-
-This function gets if the clock shows hours in military or am/pm
-mode.
+[[Get if the given clock widget shows hours in military or am/pm
+  mode
 
-@see elm_clock_show_am_pm_set() for more details
+  This function gets if the clock shows hours in military or am/pm
+  mode.
 
-@ingroup Clock */
+  See also @.show_am_pm.set for more details.
+]]
  }
  values {
-am_pm: bool; /*@ @c EINA_TRUE to put it in am/pm mode, @c 
EINA_FALSE
-to military mode */
+am_pm: bool; [[$true to put it in am/pm mode,
+   $false to military mode]]
  }
   }
   @property first_interval {
  set {
-/*@
-Set the first interval on time updates for a user mouse button hold
-on clock widgets' time edition.
+[[Set the first interval on time updates for a user mouse button 
hold
+  on clock widgets' time edition.
 
-This interval value is @b decreased while the user holds the
-mouse pointer either incrementing or decrementing a given the
-clock digit's value.
+  This interval value is decreased while the user holds the
+  mouse pointer either incrementing or decrementing a given the
+  clock digit's value.
 
-This helps the user to get to a given time distant from the
-current one easier/faster, as it will start to flip quicker and
-quicker on mouse button holds.
+  This helps the user to get to a given time distant from the
+  current one easier/faster, as it will start to flip quicker and
+  quicker on mouse button holds.
 
-The calculation for the next flip interval value, starting from
-the one set with this call, is the previous interval divided by
-1.05, so it decreases a little bit.
+  The calculation for the next flip interval value, starting from
+  the one set with this call, is the previous interval divided by
+  1.05, so it decreases a little bit.
 
-The default starting interval value for automatic flips is
-@b 0.85 seconds.
+  The default starting interval value for automatic flips is
+  0.85 seconds.
 
-@see elm_clock_first_interval_get()
-
-@ingroup Clock */
+  See also @.first_interval.get.
+]]
  }
  get {
-/*@
-Get the first interval on time updates for a user mouse button hold
-on clock widgets' time edition.
-
-@return The first interval value, in seconds, set on it
+[[Get the first interval on time

[EGIT] [core/elementary] master 01/01: elm_color_item: Convert eo docs to new format

2015-07-21 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=3f3c0f364d104cbf524859ac2d3ce9115209e0c1

commit 3f3c0f364d104cbf524859ac2d3ce9115209e0c1
Author: Vivek Ellur 
Date:   Tue Jul 21 15:01:46 2015 +0100

elm_color_item: Convert eo docs to new format

Summary:
Updated docs for elm_color_item.eo and elm_colorselector.eo files

Signed-off-by: Vivek Ellur 

Reviewers: q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2850
---
 src/lib/elm_color_item.eo|  78 +--
 src/lib/elm_colorselector.eo | 109 +++
 2 files changed, 71 insertions(+), 116 deletions(-)

diff --git a/src/lib/elm_color_item.eo b/src/lib/elm_color_item.eo
index 99320cb..b918c49 100644
--- a/src/lib/elm_color_item.eo
+++ b/src/lib/elm_color_item.eo
@@ -2,54 +2,42 @@ class Elm.Color_Item(Elm.Widget_Item)
 {
eo_prefix: elm_obj_color_item;
methods {
-@property color {
- get {
- /*@
- Get Palette item's color.
+  @property color {
+ get {
+[[Get Palette item's color.]]
+ }
+ set {
+[[Set Palette item's color.]]
+ }
+ values {
+r: int; [[red-value of color]]
+g: int; [[green-value of color]]
+b: int; [[blue-value of color]]
+a: int; [[alpha-value of color]]
+ }
+  }
+  @property selected {
+ get {
+[[Get the selected state of color palette item
 
- @ingroup Colorselector
- */
- }
- set {
- /*@
- Set Palette item's color.
+  @since 1.9
+]]
+ }
+ set {
+[[Set the selected state of color palette item
 
- @ingroup Colorselector
- */
- }
- values {
- r: int; /*@ red-value of color   */
- g: int; /*@ green-value of color */
- b: int; /*@ blue-value of color  */
- a: int; /*@ alpha-value of color */
- }
-}
-@property selected {
- get {
- /*@
- Get the selected state of color palette item
-
- @since 1.9
- @ingroup Colorselector
- */
- }
- set {
- /*@
- Set the selected state of color palette item
-
- @since 1.9
- @ingroup Colorselector
- */
- }
- values {
- selected: bool; /*@ @c EINA_TRUE if selected @c EINA_FALSE 
otherwise */
- }
-}
+  @since 1.9
+]]
+ }
+ values {
+ selected: bool; [[$true if selected $false otherwise]]
+ }
+  }
}
implements {
-Eo.Base.constructor;
-Eo.Base.destructor;
-Elm.Widget_Item.access_register;
-Elm.Widget_Item.signal_emit;
+  Eo.Base.constructor;
+  Eo.Base.destructor;
+  Elm.Widget_Item.access_register;
+  Elm.Widget_Item.signal_emit;
}
 }
diff --git a/src/lib/elm_colorselector.eo b/src/lib/elm_colorselector.eo
index fee1c23..cc036c7 100644
--- a/src/lib/elm_colorselector.eo
+++ b/src/lib/elm_colorselector.eo
@@ -19,121 +19,88 @@ class Elm.Colorselector (Elm.Layout, 
Elm_Interface_Atspi_Widget_Action,
methods {
   @property color {
  set {
-/*@
-Set color to colorselector
+[[Set color to colorselector.]]
 
-@ingroup Colorselector */
  }
  get {
-/*@
-Get current color from colorselector
-
-@ingroup Colorselector */
+[[Get current color from colorselector.]]
  }
  values {
-r: int; /*@ r-value of color */
-g: int; /*@ g-value of color */
-b: int; /*@ b-value of color */
-a: int; /*@ a-value of color */
+r: int; [[r-value of color]]
+g: int; [[g-value of color]]
+b: int; [[b-value of color]]
+a: int; [[a-value of color]]
  }
   }
   @property palette_name {
  set {
-/*@
-Set current palette's name
-
-When colorpalette name is set, colors will be loaded from and 
saved to config
-using the set name. If no name is set then colors will be loaded 
from or
-saved to "default" config.
+[[Set current palette's name
 
-@ingroup Colorselector */
+  When colorpalette name is set, colors will be loaded from and 
saved to config
+  using the set name. If no name is

[EGIT] [core/elementary] master 01/01: elm_calendar: Convert eo docs to new format

2015-07-21 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=a92650ed19cc3e0e6e41fec6a2478e68339dc064

commit a92650ed19cc3e0e6e41fec6a2478e68339dc064
Author: Vivek Ellur 
Date:   Tue Jul 21 11:53:55 2015 +0100

elm_calendar: Convert eo docs to new format

Summary:
Changed the docs of elm_calendar.eo to the new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2846
---
 src/lib/elm_calendar.eo | 464 +---
 1 file changed, 207 insertions(+), 257 deletions(-)

diff --git a/src/lib/elm_calendar.eo b/src/lib/elm_calendar.eo
index d7807cb..d494e6a 100644
--- a/src/lib/elm_calendar.eo
+++ b/src/lib/elm_calendar.eo
@@ -76,386 +76,336 @@ class Elm.Calendar (Elm.Layout, 
Elm_Interface_Atspi_Widget_Action)
methods {
   @property first_day_of_week {
  set {
-/*@
-Set the first day of week to use on calendar widgets'.
-
-@ingroup Calendar */
+[[Set the first day of week to use on calendar widgets'.]]
  }
  get {
-/*@
-Get the first day of week, who are used on calendar widgets'.
-
-@return An int which correspond to the first day of the week 
(Sunday = 0, Monday = 1,
-..., Saturday = 6)
+[[Get the first day of week, who are used on calendar widgets'.
 
-@see elm_calendar_first_day_of_week_set() for more details
+  See also  @.first_day_of_week.set for more details.
 
-@ingroup Calendar */
+]]
  }
  values {
-day: Elm.Calendar.Weekday; /*@ An int which correspond to the 
first day of the week (Sunday = 0, Monday = 1,
-..., Saturday = 6) */
+day: Elm.Calendar.Weekday; [[An int which correspond to the first 
day of the week (Sunday = 0, Monday = 1,
+..., Saturday = 6).]]
  }
   }
   @property selectable {
  set {
-/*@
-Define which fields of a @b tm struct will be taken into account, 
when
-elm_calendar_selected_time_set() is invoked.
+[[Define which fields of a tm struct will be taken into account, 
when
+  Elm.Calendar.selected_time.set is invoked.
+
+  By Default the bitmask is set to use all fields of a tm struct 
(year,
+  month and day of the month).
 
-By Default the bitmask is set to use all fields of a @b tm struct 
(year,
-month and day of the month).
 
-@ingroup Calendar
-@see elm_calendar_selected_time_set
-@since 1.8 */
+  See also @.selected_time_set.
+
+  @since 1.8
+]]
  }
  get {
-/*@
-Get how elm_calendar_selected_time_set manage a date
+[[Get how elm_calendar_selected_time_set manage a date
 
-@return The flag used to manage a date with a 
elm_calendar_selected_time_set
+  See also @.selectable.set,
+  @.selected_time_set.
 
-@ingroup Calendar
-@see elm_calendar_selectable_set
-@see elm_calendar_selected_time_set
-@since 1.8 */
+  @since 1.8
+]]
  }
  values {
-selectable: Elm.Calendar.Selectable; /*@ A bitmask of 
Elm_Calendar_Selectable */
+selectable: Elm.Calendar.Selectable; [[A bitmask of 
Elm_Calendar_Selectable]]
  }
   }
   @property interval {
  set {
-/*@
-Set the interval on time updates for an user mouse button hold
-on calendar widgets' month/year selection.
-
-This interval value is @b decreased while the user holds the
-mouse pointer either selecting next or previous month/year.
+[[Set the interval on time updates for an user mouse button hold
+  on calendar widgets' month/year selection.
 
-This helps the user to get to a given month distant from the
-current one easier/faster, as it will start to change quicker and
-quicker on mouse button holds.
+  This interval value is decreased while the user holds the
+  mouse pointer either selecting next or previous month/year.
 
-The calculation for the next change interval value, starting from
-the one set with this call, is the previous interval divided by
-1.05, so it decreases a little bit.
+  This helps the user to get to a given month distant from the
+  current one easier/faster, as it will start to change quicker and
+  quicker on mouse button holds.
 
-The default starting interval value for automatic c

[EGIT] [core/elementary] master 01/02: elm_box: convert eo docs to new format

2015-07-16 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=dc060651185f1e1e90a4885e2b4dd8004d1c660a

commit dc060651185f1e1e90a4885e2b4dd8004d1c660a
Author: Vivek Ellur 
Date:   Thu Jul 16 15:13:21 2015 +0100

elm_box: convert eo docs to new format

Summary:
Converted elm_box.eo documentation to new format

Signed-off-by: Vivek Ellur 

Reviewers: q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2826
---
 src/lib/elm_box.eo | 338 -
 1 file changed, 155 insertions(+), 183 deletions(-)

diff --git a/src/lib/elm_box.eo b/src/lib/elm_box.eo
index 6610970..ca33d26 100644
--- a/src/lib/elm_box.eo
+++ b/src/lib/elm_box.eo
@@ -4,304 +4,276 @@ class Elm.Box (Elm.Widget)
methods {
   @property homogeneous {
  set {
-/*@
-Set the box to arrange its children homogeneously
+[[Set the box to arrange its children homogeneously
 
-If enabled, homogeneous layout makes all items the same size, 
according
-to the size of the largest of its children.
+  If enabled, homogeneous layout makes all items the same size, 
according
+  to the size of the largest of its children.
 
-@note This flag is ignored if a custom layout function is set.
+  Note: This flag is ignored if a custom layout function is set.
 
-@ingroup Box */
+]]
  }
  get {
-/*@
-Get whether the box is using homogeneous mode or not
-
-@return @c EINA_TRUE if it's homogeneous, @c EINA_FALSE otherwise
-
-@ingroup Box */
+[[Get whether the box is using homogeneous mode or not ($true if
+  it's homogeneous, $false otherwise)]]
  }
  values {
-homogeneous: bool; /*@ The homogeneous flag */
+homogeneous: bool; [[The homogeneous flag]]
  }
   }
   @property align {
  set {
-/*@
-Set the alignment of the whole bounding box of contents.
+[[Set the alignment of the whole bounding box of contents.
 
-Sets how the bounding box containing all the elements of the box, 
after
-their sizes and position has been calculated, will be aligned 
within
-the space given for the whole box widget.
+  Sets how the bounding box containing all the elements of the 
box, after
+  their sizes and position has been calculated, will be aligned 
within
+  the space given for the whole box widget.
 
-@ingroup Box */
+]]
  }
  get {
-/*@
-Get the alignment of the whole bounding box of contents.
+[[Get the alignment of the whole bounding box of contents.
 
-@see elm_box_align_set()
+  See also @.align.set.
 
-@ingroup Box */
+]]
  }
  values {
-horizontal: double; /*@ The horizontal alignment of elements */
-vertical: double; /*@ The vertical alignment of elements */
+horizontal: double; [[The horizontal alignment of elements]]
+vertical: double; [[The vertical alignment of elements]]
  }
   }
   @property horizontal {
  set {
-/*@
-Set the horizontal orientation
+[[Set the horizontal orientation
 
-By default, box object arranges their contents vertically from top 
to
-bottom.
-By calling this function with @p horizontal as @c EINA_TRUE, the 
box will
-become horizontal, arranging contents from left to right.
+  By default, box object arranges their contents vertically from 
top to
+  bottom.
+  By calling this function with $horizontal as $true, the box will
+  become horizontal, arranging contents from left to right.
 
-@note This flag is ignored if a custom layout function is set.
+  Note: This flag is ignored if a custom layout function is set.
 
-@ingroup Box */
+]]
  }
  get {
-/*@
-Get the horizontal orientation
-
-@return @c EINA_TRUE if the box is set to horizontal mode, @c 
EINA_FALSE otherwise
-
-@ingroup Box */
+[[Get the horizontal orientation ($true if the box is set to
+  horizontal mode, $false otherwise)]]
  }
  values {
-horizontal: bool; /*@ The horizontal flag (@c EINA_TRUE = 
horizontal,
-@c EINA_FALSE = vertical) */
+horizontal: bool; [[The horizontal flag]]
  }
   }
   @property padding {
  set {
-/*@
-Set the space (padding) between

[EGIT] [core/elementary] master 02/02: elm_button: Converted docs of elm_button eo to new format

2015-07-16 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=77b436736990c413d21f2d7bae1a67da32b30e6e

commit 77b436736990c413d21f2d7bae1a67da32b30e6e
Author: Vivek Ellur 
Date:   Thu Jul 16 15:13:27 2015 +0100

elm_button: Converted docs of elm_button eo to new format

Summary:
Changed the docs of elm_bubble.eo and elm_button.eo to the new format

Signed-off-by: Vivek Ellur 

Reviewers: q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2827
---
 src/lib/elm_bubble.eo | 24 ++-
 src/lib/elm_button.eo | 81 +--
 2 files changed, 43 insertions(+), 62 deletions(-)

diff --git a/src/lib/elm_bubble.eo b/src/lib/elm_bubble.eo
index 94a1812..a98c07d 100644
--- a/src/lib/elm_bubble.eo
+++ b/src/lib/elm_bubble.eo
@@ -18,28 +18,22 @@ class Elm.Bubble (Elm.Layout, Evas.Clickable_Interface)
methods {
   @property pos {
  set {
-/*@
-Set the corner of the bubble
+[[Set the corner of the bubble
 
-This function sets the corner of the bubble. The corner will be 
used to
-determine where the arrow in the frame points to and where label, 
icon and
-info are shown.
+  This function sets the corner of the bubble. The corner will be 
used to
+  determine where the arrow in the frame points to and where 
label, icon and
+  info are shown.
 
-
-@ingroup Bubble */
+]]
  }
  get {
-/*@
-Get the corner of the bubble
-
-@return The given corner for the bubble.
-
-This function gets the selected corner of the bubble.
+[[Get the corner of the bubble
 
-@ingroup Bubble */
+  This function gets the selected corner of the bubble.
+]]
  }
  values {
-pos: Elm.Bubble.Pos; /*@ The given corner for the bubble. */
+pos: Elm.Bubble.Pos; [[The given corner for the bubble.]]
  }
   }
}
diff --git a/src/lib/elm_button.eo b/src/lib/elm_button.eo
index cd09409..ad58c37 100644
--- a/src/lib/elm_button.eo
+++ b/src/lib/elm_button.eo
@@ -5,90 +5,77 @@ class Elm.Button (Elm.Layout, Evas.Clickable_Interface,
methods {
   @property autorepeat_initial_timeout {
  set {
-/*@
-Set the initial timeout before the autorepeat event is generated
+[[Set the initial timeout before the autorepeat event is generated
 
-Sets the timeout, in seconds, since the button is pressed until the
-first @c repeated signal is emitted. If @p t is 0.0 or less, there
-won't be any delay and the event will be fired the moment the 
button is
-pressed.
+  Sets the timeout, in seconds, since the button is pressed until 
the
+  first $repeated signal is emitted. If $t is 0.0 or less, there
+  won't be any delay and the event will be fired the moment the 
button is
+  pressed.
 
-@see elm_button_autorepeat_set()
-@see elm_button_autorepeat_gap_timeout_set()
+  See also @.autorepeat.set,
+  @.autorepeat_gap_timeout.set.
 
-@ingroup Button */
+]]
  }
  get {
-/*@
-Get the initial timeout before the autorepeat event is generated
+[[Get the initial timeout before the autorepeat event is generated
 
-@return Timeout in seconds
+  See also @.autorepeat_initial_timeout.set.
 
-@see elm_button_autorepeat_initial_timeout_set()
-
-@ingroup Button */
+]]
  }
  values {
-t: double; /*@ Timeout in seconds */
+t: double; [[Timeout in seconds]]
  }
   }
   @property autorepeat_gap_timeout {
  set {
-/*@
-Set the interval between each generated autorepeat event
+[[Set the interval between each generated autorepeat event
 
-After the first @c repeated event is fired, all subsequent ones 
will
-follow after a delay of @p t seconds for each.
+  After the first $repeated event is fired, all subsequent ones 
will
+  follow after a delay of $t seconds for each.
 
-@see elm_button_autorepeat_initial_timeout_set()
+  See also @.autorepeat_initial_timeout.set.
 
-@ingroup Button */
+]]
  }
  get {
-/*@
-Get the interval between each generated autorepeat event
-
-@return Interval in seconds
-
-@ingroup Button */
+[[Get the interval between each generated autorepeat event]]
  }
  values {
-t: double; /*

[EGIT] [core/elementary] master 01/01: elm_bg: convert eo docs to new format

2015-07-16 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=3656fb4e2269c013d55419d5e606fbc2429aa19c

commit 3656fb4e2269c013d55419d5e606fbc2429aa19c
Author: Vivek Ellur 
Date:   Thu Jul 16 11:21:15 2015 +0100

elm_bg: convert eo docs to new format

Summary:
Converted docs of elm_bg.eo to new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Differential Revision: https://phab.enlightenment.org/D2819
---
 src/lib/elm_atspi_bridge.eo |  4 +-
 src/lib/elm_bg.eo   | 97 +
 2 files changed, 47 insertions(+), 54 deletions(-)

diff --git a/src/lib/elm_atspi_bridge.eo b/src/lib/elm_atspi_bridge.eo
index eafd9f4..378060c 100644
--- a/src/lib/elm_atspi_bridge.eo
+++ b/src/lib/elm_atspi_bridge.eo
@@ -4,7 +4,7 @@ class Elm.Atspi_Bridge (Eo.Base)
methods {
   @property connected {
  get {
-/*@ Indicate if connection with AT-SPI2 bus has been established. 
*/
+[[Indicate if connection with AT-SPI2 bus has been established.]]
 values {
ret: bool;
 }
@@ -12,7 +12,7 @@ class Elm.Atspi_Bridge (Eo.Base)
   }
   @property root {
  get {
-/*@ Get root object of accessible object hierarchy */
+[[Get root object of accessible object hierarchy]]
 values {
ret: Eo*;
 }
diff --git a/src/lib/elm_bg.eo b/src/lib/elm_bg.eo
index 2492926..01a9677 100644
--- a/src/lib/elm_bg.eo
+++ b/src/lib/elm_bg.eo
@@ -19,95 +19,88 @@ class Elm.Bg (Elm.Layout, Efl.File)
methods {
   @property option {
  set {
-/*@
-Set the mode of display for a given background widget's image
+[[Set the mode of display for a given background widget's image
 
-This sets how the background widget will display its image. This
-will only work if the elm_bg_file_set() was previously called with
-an image file on @a obj. The image can be display tiled, scaled,
-centered or stretched. #ELM_BG_OPTION_SCALE by default.
+  This sets how the background widget will display its image. This
+  will only work if the @Efl.File.file.set was previously called 
with
+  an image file on $obj. The image can be display tiled, scaled,
+  centered or stretched. @Elm.Bg.Option.scale by default.
 
-@see elm_bg_option_get()
+  See also @.option.get.
+]]
 
-@ingroup Bg */
  }
  get {
-/*@
-Get the mode of display for a given background widget's image
+[[Get the mode of display for a given background widget's image.
 
-@return The image displaying mode in use for @a obj or 
#ELM_BG_OPTION_LAST,
-on errors.
+  See also @.option.set for more details
 
-@see elm_bg_option_set() for more details
-
-@ingroup Bg */
+]]
  }
  values {
-option: Elm.Bg.Option; /*@ The desired background option (see 
#Elm_Bg_Option) */
+option: Elm.Bg.Option; [[The desired background option(See 
@Elm.Bg.Option)]]
  }
   }
   @property color {
  set {
-/*@
-Set the color on a given background widget
+[[Set the color on a given background widget
 
-This sets the color used for the background rectangle, in RGB
-format. Each color component's range is from 0 to 255.
+  This sets the color used for the background rectangle, in RGB
+  format. Each color component's range is from 0 to 255.
 
-@note You probably only want to use this function if you haven't
-previously called elm_bg_file_set(), so that you just want a solid
-color background.
+  Note: You probably only want to use this function if you haven't
+  previously called @Efl.File.file.set, so that you just want a 
solid
+  color background.
 
-@note You can reset the color by setting @p r, @p g, @p b as -1, 
-1, -1.
+  Note: You can reset the color by setting $r, $g, $b as -1, -1, 
-1.
 
-@see elm_bg_color_get()
+  See also @.color.get
 
-@ingroup Bg */
+]]
 legacy: null; /* legacy doesn't have 'a' param */
  }
  get {
-/*@
-Get the color set on a given background widget
+[[Get the color set on a given background widget
 
-@note Use @c NULL pointers on the file components you're not
-interested in: they'll be ignored by the function.
+  Note: Use $null pointers on the file components you'r

[EGIT] [core/elementary] master 02/02: elm_app_server: Convert eo docs to new format

2015-07-14 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=8014a35d56457372ef7e4822749b82860199d769

commit 8014a35d56457372ef7e4822749b82860199d769
Author: Vivek Ellur 
Date:   Tue Jul 14 14:44:25 2015 +0100

elm_app_server: Convert eo docs to new format

Summary:
Converted docs of elm_app_server.eo and elm_app_server_view.eo to
new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2816
---
 src/lib/elm_app_server.eo  | 41 ++---
 src/lib/elm_app_server_view.eo |  4 ++--
 2 files changed, 24 insertions(+), 21 deletions(-)

diff --git a/src/lib/elm_app_server.eo b/src/lib/elm_app_server.eo
index 353ac03..4cd82ea 100644
--- a/src/lib/elm_app_server.eo
+++ b/src/lib/elm_app_server.eo
@@ -9,15 +9,15 @@ class Elm_App_Server (Eo.Base)
  get {
  }
  values {
-icon: Eina_Stringshare *; /*@ title of icon */
+icon: Eina_Stringshare *; [[title of icon]]
  }
   }
   @property views {
  get {
-/*@ Return a iterator with all views of application */
+[[Return a iterator with all views of application]]
  }
  values {
-ret: free(own(iterator *), 
eina_iterator_free); /*@ Iterator with all views of application, you must free 
iterator after use */
+ret: free(own(iterator *), 
eina_iterator_free); [[Iterator with all views of application, you must free 
iterator after use]]
  }
   }
   @property path {
@@ -36,10 +36,10 @@ class Elm_App_Server (Eo.Base)
   }
   @property pixels {
  get {
-/*@ Get application raw icon. */
+[[Get application raw icon.]]
  }
  set {
-/*@ Set icon to application, using the raw pixels of image. */
+[[Set icon to application, using the raw pixels of image.]]
  }
  values {
 w: uint;
@@ -49,41 +49,44 @@ class Elm_App_Server (Eo.Base)
  }
   }
   constructor {
- /*@ Class constructor of elm_app_server */
+ [[Class constructor of elm_app_server]]
  legacy: null;
  params {
-@in packageid: const(char)*; /*@ package of application */
-@in create_view_cb: Elm_App_Server_Create_View_Cb; /*@ callback to 
be called when user whants to open some application view */
+@in packageid: const(char)*; [[package of application]]
+@in create_view_cb: Elm_App_Server_Create_View_Cb; [[callback to 
be called when user whants to open some application view]]
  }
   }
   close_all {
- /*@ Close all views of application */
+ [[Close all views of application]]
   }
   view_check {
- /*@ If view id is available and unique, return the full DBus object 
path of view */
+ [[If view id is available and unique, return the full DBus object 
path of view]]
  params {
-@in id: const(char)*; /*@ view identifier */
+@in id: const(char)*; [[view identifier]]
  }
- return: bool; /*@ @c EINA_TRUE if id is valid or @c EINA_FALSE if not 
*/
+ return: bool; [[true if id is valid or false if not]]
   }
   view_add {
- /*@ Add a view to elm_app_server. This should only be used if the 
application open a view that was not requested by create_view_cb. */
+ [[Add a view to elm_app_server. This should only be
+   used if the application open a view that was not
+   requested by create_view_cb.
+ ]]
  params {
-@in view: Elm_App_Server_View *; /*@ elm_app_server_view */
+@in view: Elm_App_Server_View *; [[elm_app_server_view]]
  }
   }
   title_set {
- /*@ Set a title to application. */
+ [[Set a title to application.]]
  params {
-@in title: const(char)* @nullable; /*@ title of application */
+@in title: const(char)* @nullable; [[title of application]]
  }
   }
   title_get {
- /*@ Get title of application */
- return: Eina_Stringshare *; /*@ title of application */
+ [[Get title of application]]
+ return: Eina_Stringshare *; [[title of application]]
   }
   save {
- /*@ Save the state of all views */
+ [[Save the state of all views]]
   }
}
implements {
diff --git a/src/lib/elm_app_server_view.eo b/src/lib/elm_app_server_view.eo
index dc48472..51e32c5 100644
--- a/src/lib/elm_app_server_view.eo
+++ b/src/lib/elm_app_server_view.eo
@@ -71,10 +71,10 @@ class Elm_App_Server_View (Eo.Base)
   }
   @property pixels {
  get {
-/*@ Get application raw icon. */
+[[Get application raw icon.]]
  }
  set

[EGIT] [core/elementary] master 01/02: elm_app_client: convert eo docs to new format

2015-07-14 Thread Vivek Ellur
q66 pushed a commit to branch master.

http://git.enlightenment.org/core/elementary.git/commit/?id=39a1e98bf65bdb5f58421a5aecdc9396118745b5

commit 39a1e98bf65bdb5f58421a5aecdc9396118745b5
Author: Vivek Ellur 
Date:   Tue Jul 14 14:44:02 2015 +0100

elm_app_client: convert eo docs to new format

Summary:
Converted docs of elm_app_client.eo and elm_app_client_view.eo to
new format

Signed-off-by: Vivek Ellur 

Reviewers: cedric, q66

Reviewed By: q66

Differential Revision: https://phab.enlightenment.org/D2814
---
 src/lib/elm_app_client.eo  | 30 ++--
 src/lib/elm_app_client_view.eo | 64 ++
 2 files changed, 49 insertions(+), 45 deletions(-)

diff --git a/src/lib/elm_app_client.eo b/src/lib/elm_app_client.eo
index b4cfa51..d58ec3a 100644
--- a/src/lib/elm_app_client.eo
+++ b/src/lib/elm_app_client.eo
@@ -5,46 +5,46 @@ class Elm_App_Client (Eo.Base)
methods {
   @property views {
  get {
-/*@ Return a iterator with all views of application. */
+[[Return a iterator with all views of application.]]
  }
  values {
-ret: free(own(iterator *), 
eina_iterator_free); /*@ The iterator with all views, must be freed after use */
+ret: free(own(iterator *), 
eina_iterator_free); [[The iterator with all views, must be freed after use.]]
  }
   }
   @property package {
  get {
-/*@ Return the application package. */
+[[Return the application package.]]
  }
  values {
-ret: const(char)*; /*@ application package */
+ret: const(char)*; [[application package]]
  }
   }
   constructor {
- /*@ Class constructor of elm_app_client. */
+ [[Class constructor of elm_app_client.]]
  legacy: null;
  params {
-@in package: const(char)*; /*@ Package of application */
+@in package: const(char)*; [[Package of application]]
  }
   }
   view_all_close {
- /*@ Close all views of application. */
+ [[Close all views of application.]]
   }
   terminate {
- /*@ Terminate application. */
+ [[Terminate application.]]
   }
   view_open {
- /*@ Open an application view. */
+ [[Open an application view.]]
  params {
-@in args: Eina_Value * @optional; /*@ an array of */
-@in view_open_cb: Elm_App_Client_Open_View_Cb @optional; /*@ 
callback to be called when view open */
-@in data: const(void)* @optional; /*@ calback user data */
+@in args: Eina_Value * @optional; [[an array of.]]
+@in view_open_cb: Elm_App_Client_Open_View_Cb @optional; 
[[callback to be called when view open]]
+@in data: const(void)* @optional; [[callback user data]]
  }
- return: Elm_App_Client_Pending *; /*@ handler to cancel the view 
opening if it takes to long */
+ return: Elm_App_Client_Pending *; [[handler to cancel the view 
opening if it takes to long ]]
   }
   view_open_cancel {
- /*@ Cancel a pending elm_app_client_view_open(). */
+ [[Cancel a pending elm_app_client_view_open().]]
  params {
-@in pending: Elm_App_Client_Pending *; /*@ the view open handler */
+@in pending: Elm_App_Client_Pending *; [[the view open handler]]
  }
   }
}
diff --git a/src/lib/elm_app_client_view.eo b/src/lib/elm_app_client_view.eo
index 8534a3f..3a2e4e5 100644
--- a/src/lib/elm_app_client_view.eo
+++ b/src/lib/elm_app_client_view.eo
@@ -5,37 +5,39 @@ class Elm_App_Client_View (Eo.Base)
methods {
   @property state {
  get {
-/*@ Get state of view */
+[[Get state of view]]
  }
  values {
-state: Elm_App_View_State; /*@ state of view */
+state: Elm_App_View_State; [[state of view]]
  }
   }
   @property new_events {
  get {
-/*@ Get new events of view */
+[[Get new events of view]]
  }
  values {
-events: int; /*@ number of events of view */
+events: int; [[number of events of view]]
  }
   }
   @property window {
  get {
-/*@ Get window of view */
+[[Get window of view]]
  }
  values {
-window: int; /*@ window of view */
+window: int; [[window of view]]
  }
   }
   @property icon_pixels {
  get {
-/*@ Get icon pixels of view, view could have a icon in raw format 
not saved in disk. */
+[[Get icon pixels of view, view could have a icon 
+  in raw format not saved in disk.
+]]
  }
  values {
-w: uint; /*@ icon width */
-h: uint

[EGIT] [core/efl] master 01/02: eina: fix failure of eina_quaternion_test.

2015-07-03 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=68d9c3d6f0595179199afbe37f07a3f2be5d6df4

commit 68d9c3d6f0595179199afbe37f07a3f2be5d6df4
Author: Vivek Ellur 
Date:   Sat Jul 4 02:30:40 2015 +0200

eina: fix failure of eina_quaternion_test.

Summary:
eina_test_quaternion_norm was failing. so fixed it.

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2781

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_quaternion.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index 9ca60d4..9759355 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -59,10 +59,11 @@ eina_matrix3_cmp(const Eina_Matrix3 *a, const Eina_Matrix3 
*b)
 START_TEST(eina_test_quaternion_norm)
 {
static const Eina_Quaternion q = { 1, 3, 4, 5 };
+   double result = eina_quaternion_norm(&q);
 
eina_init();
 
-   fail_if(!FLOAT_CMP(eina_quaternion_norm(&q), sqrt(51)));
+   fail_if(!FLOAT_CMP(result, sqrt(51)));
 
eina_shutdown();
 }

-- 




[EGIT] [core/efl] master 03/03: eina: add quaternion test in eina suite and fix the test case errors

2015-06-29 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=1155176aa50e3ecdaf7affaf0b86d4b1ffd51ce9

commit 1155176aa50e3ecdaf7affaf0b86d4b1ffd51ce9
Author: Vivek Ellur 
Date:   Mon Jun 29 14:24:21 2015 +0200

eina: add quaternion test in eina suite and fix the test case errors

Summary:
Fixed eina_test_quaternion_conjugate and eina_test_quaternion_matrix which
were failing and also added the quaternion test in eina suite

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2775

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_suite.c   |  1 +
 src/tests/eina/eina_suite.h   |  1 +
 src/tests/eina/eina_test_quaternion.c | 10 +-
 3 files changed, 7 insertions(+), 5 deletions(-)

diff --git a/src/tests/eina/eina_suite.c b/src/tests/eina/eina_suite.c
index 5383bb6..5caa536 100644
--- a/src/tests/eina/eina_suite.c
+++ b/src/tests/eina/eina_suite.c
@@ -82,6 +82,7 @@ static const Eina_Test_Case etc[] = {
{ "Crc", eina_test_crc },
{ "Quad", eina_test_quad },
{ "Matrix", eina_test_matrix },
+   { "Quaternion", eina_test_quaternion },
{ NULL, NULL }
 };
 
diff --git a/src/tests/eina/eina_suite.h b/src/tests/eina/eina_suite.h
index d39254d..845ec9d 100644
--- a/src/tests/eina/eina_suite.h
+++ b/src/tests/eina/eina_suite.h
@@ -67,5 +67,6 @@ void eina_test_xattr(TCase *tc);
 void eina_test_crc(TCase *tc);
 void eina_test_quad(TCase *tc);
 void eina_test_matrix(TCase *tc);
+void eina_test_quaternion(TCase *tc);
 
 #endif /* EINA_SUITE_H_ */
diff --git a/src/tests/eina/eina_test_quaternion.c 
b/src/tests/eina/eina_test_quaternion.c
index 55e1c9f..9ca60d4 100644
--- a/src/tests/eina/eina_test_quaternion.c
+++ b/src/tests/eina/eina_test_quaternion.c
@@ -62,7 +62,7 @@ START_TEST(eina_test_quaternion_norm)
 
eina_init();
 
-   fail_if(FLOAT_CMP(eina_quaternion_norm(&q), sqrt(51)));
+   fail_if(!FLOAT_CMP(eina_quaternion_norm(&q), sqrt(51)));
 
eina_shutdown();
 }
@@ -71,7 +71,7 @@ END_TEST
 START_TEST(eina_test_quaternion_conjugate)
 {
static const Eina_Quaternion q1 = { 1, -1, -1, 3 }, q2 = { 1, 3, 4, 3 };
-   static const Eina_Quaternion r1 = { 1, 1, 1, -3 }, r2 = { 1, -3, -4, -3 };
+   static const Eina_Quaternion r1 = { -1, 1, 1, 3 }, r2 = { -1, -3, -4, 3 };
Eina_Quaternion t1, t2;
 
eina_init();
@@ -90,9 +90,9 @@ START_TEST(eina_test_quaternion_matrix)
 {
Eina_Quaternion q = { 7, 9, 5, 1 };
Eina_Matrix3 m = {
- 104, 76, 8,
- 104, -8, -116,
- -52, 136, -56
+ -211, 136, 52,
+ 116, -147, 104,
+ 88, 76, -259
};
Eina_Quaternion tq;
Eina_Matrix3 tm;

-- 




[EGIT] [core/efl] master 01/04: eina: add test cases for eina matrix map and transform functions

2015-06-25 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=70e5687f6c43ffd3a66de592e625ff59c15909b6

commit 70e5687f6c43ffd3a66de592e625ff59c15909b6
Author: Vivek Ellur 
Date:   Thu Jun 25 15:35:33 2015 +0200

eina: add test cases for eina matrix map and transform functions

Summary:
Added test cases for matrix map and transform functions

Signed-off-by: Vivek Ellur 

Reviewers: stefan_schmidt, cedric

Reviewed By: stefan_schmidt, cedric

Subscribers: stefan_schmidt, cedric

Differential Revision: https://phab.enlightenment.org/D2758

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_matrix.c | 53 +++
 1 file changed, 53 insertions(+)

diff --git a/src/tests/eina/eina_test_matrix.c 
b/src/tests/eina/eina_test_matrix.c
index ed7c7ef..af1e005 100644
--- a/src/tests/eina/eina_test_matrix.c
+++ b/src/tests/eina/eina_test_matrix.c
@@ -366,6 +366,58 @@ START_TEST(eina_matrix3_f16p16)
 }
 END_TEST
 
+START_TEST(eina_matrix3_map_transform)
+{
+   double x = 2, y = 3, x1, y1;
+   Eina_Matrix3 m;
+   Eina_Rectangle r;
+   Eina_Quad q;
+   Eina_Bool ret;
+
+   eina_init();
+
+   eina_matrix3_values_set(&m,
+   0, 1, 0,
+   1, 0, 0,
+   0, 0, 1);
+
+   eina_matrix3_point_transform(&m,
+x, y,
+&x1, &y1);
+   fail_if(x1 != 3 || y1 != 2);
+
+   EINA_RECTANGLE_SET(&r, 0, 0, 3, 4);
+   eina_matrix3_rectangle_transform(&m, &r, &q);
+
+   fail_if(q.x0 != 0 || q.y0 != 0 ||
+   q.x1 != 0 || q.y1 != 3 ||
+   q.x2 != 4 || q.y2 != 3 ||
+   q.x3 != 4 || q.y3 != 0);
+
+   eina_quad_coords_set(&q,
+0.0, 0.0,
+3.0, 0.0,
+3.0, 3.0,
+0.0, 3.0);
+   ret = eina_matrix3_square_quad_map(&m, &q);
+   fail_if(ret != EINA_TRUE);
+
+   fail_if(m.xx != 3 || m.xy != 0 || m.xz != 0 ||
+   m.yx != 0 || m.yy != 3 || m.yz != 0 ||
+   m.zx != 0 || m.zy != 0 || m.zz != 1);
+
+   ret = eina_matrix3_quad_square_map(&m, &q);
+   fail_if(ret != EINA_TRUE);
+
+   fail_if(q.x0 != 0 || q.y0 != 0 ||
+   q.x1 != 3 || q.y1 != 0 ||
+   q.x2 != 3 || q.y2 != 3 ||
+   q.x3 != 0 || q.y3 != 3);
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_matrix(TCase *tc)
 {
@@ -374,4 +426,5 @@ eina_test_matrix(TCase *tc)
tcase_add_test(tc, eina_matrix3);
tcase_add_test(tc, eina_matrix3_operations);
tcase_add_test(tc, eina_matrix3_f16p16);
+   tcase_add_test(tc, eina_matrix3_map_transform);
 }

-- 




[EGIT] [core/efl] master 01/01: eina_matrix: Added test cases for fixed point matrix functions

2015-06-16 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=f2606fe151a786d89fe8f27ba1e4da4e14823e66

commit f2606fe151a786d89fe8f27ba1e4da4e14823e66
Author: Vivek Ellur 
Date:   Tue Jun 16 17:47:29 2015 +0200

eina_matrix: Added test cases for fixed point matrix functions

Summary:
Added test cases for all the functions related to fixed point matrix

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2701
---
 src/tests/eina/eina_test_matrix.c | 65 +++
 1 file changed, 65 insertions(+)

diff --git a/src/tests/eina/eina_test_matrix.c 
b/src/tests/eina/eina_test_matrix.c
index 36d3963..ed7c7ef 100644
--- a/src/tests/eina/eina_test_matrix.c
+++ b/src/tests/eina/eina_test_matrix.c
@@ -302,6 +302,70 @@ START_TEST(eina_matrix3_operations)
 }
 END_TEST
 
+START_TEST(eina_matrix3_f16p16)
+{
+   Eina_Matrix3_F16p16 m1;
+   Eina_Matrix3 m2;
+   Eina_F16p16 xx, xy, xz,
+   yx, yy, yz,
+   zx, zy, zz;
+   Eina_Matrix3_F16p16 m3;
+
+   eina_init();
+
+   eina_matrix3_values_set(&m2,
+   1, 0, 0,
+   0, 1, 0,
+   0, 0, 1);
+   eina_matrix3_matrix3_f16p16_to(&m2, &m1);
+   fail_if (eina_matrix3_f16p16_type_get(&m1) != EINA_MATRIX_TYPE_IDENTITY);
+
+   eina_matrix3_fixed_values_get(&m2,
+ &xx, &xy, &xz,
+ &yx, &yy, &yz,
+ &zx, &zy, &zz);
+   fail_if(xx != yy ||
+   yy != zz ||
+   zz != 65536);
+
+   fail_if(xy != xz ||
+   yx != yz ||
+   zx != zy ||
+   zy != 0);
+
+   eina_matrix3_values_set(&m2,
+   1, 2, 3,
+   4, 5, 6,
+   7, 8, 9);
+   eina_matrix3_matrix3_f16p16_to(&m2, &m1);
+   eina_matrix3_f16p16_identity(&m1);
+   fail_if(m1.xx != m1.yy ||
+   m1.yy != m1.zz ||
+   m1.zz != 65536);
+
+   fail_if(m1.xy != m1.xz ||
+   m1.yx != m1.yz ||
+   m1.zx != m1.zy ||
+   m1.zy != 0);
+
+   eina_matrix3_values_set(&m2,
+   1, 1, 1,
+   1, 1, 1,
+   1, 1, 1);
+   eina_matrix3_matrix3_f16p16_to(&m2, &m1);
+   eina_matrix3_f16p16_compose(&m1, &m1, &m3);
+   fail_if (m3.xx != m3.xy ||
+m3.xy != m3.xz ||
+m3.yx != m3.yy ||
+m3.yy != m3.yz ||
+m3.zx != m3.zy ||
+m3.zy != m3.zz ||
+m3.zz != 196608);
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_matrix(TCase *tc)
 {
@@ -309,4 +373,5 @@ eina_test_matrix(TCase *tc)
tcase_add_test(tc, eina_matrix4_2_3);
tcase_add_test(tc, eina_matrix3);
tcase_add_test(tc, eina_matrix3_operations);
+   tcase_add_test(tc, eina_matrix3_f16p16);
 }

-- 




[EGIT] [core/efl] master 01/01: eina: add test cases for various eina_matrix operations

2015-06-15 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=02d825f116e3b791c784791b10cfe613c5f18385

commit 02d825f116e3b791c784791b10cfe613c5f18385
Author: Vivek Ellur 
Date:   Mon Jun 15 09:06:31 2015 +0200

eina: add test cases for various eina_matrix operations

Summary:
Added test cases for various operations of 3*3 matrix

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2692

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_matrix.c | 161 ++
 1 file changed, 161 insertions(+)

diff --git a/src/tests/eina/eina_test_matrix.c 
b/src/tests/eina/eina_test_matrix.c
index 28983e1..36d3963 100644
--- a/src/tests/eina/eina_test_matrix.c
+++ b/src/tests/eina/eina_test_matrix.c
@@ -142,10 +142,171 @@ START_TEST(eina_matrix3)
 }
 END_TEST
 
+START_TEST(eina_matrix3_operations)
+{
+   Eina_Matrix3 m1, m2, m3;
+   double xx, xy, xz,
+  yx, yy, yz,
+  zx, zy, zz;
+   double tx = 20, ty = 30, ret;
+
+   eina_matrix3_values_set(&m1,
+   1, 0, 0,
+   0, 1, 0,
+   0, 0, 1);
+
+   eina_matrix3_values_set(&m2,
+   1, 1, 1,
+   1, 1, 1,
+   1, 1, 1);
+
+
+   eina_matrix3_compose(&m1, &m2, &m3);
+   eina_matrix3_values_get(&m3,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+   fail_if (xx != xy ||
+xy != xz ||
+yx != yy ||
+yy != yz ||
+zx != zy ||
+zy != zz ||
+zz != 1);
+
+   eina_matrix3_translate(&m1, tx, ty);
+   eina_matrix3_values_get(&m1,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+   fail_if (xx != 1 || xy != 0 || xz != tx ||
+yx != 0 || yy != 1 || yz != ty ||
+zx != 0 || zy != 0 || zz != 1);
+
+   eina_matrix3_values_set(&m1,
+   1, 0, 0,
+   0, 1, 0,
+   0, 0, 1);
+
+   eina_matrix3_scale(&m1, tx, ty);
+   eina_matrix3_values_get(&m1,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+   fail_if (xx != tx || xy != 0 || xz != 0 ||
+yx != 0 || yy != ty || yz != 0 ||
+zx != 0 || zy != 0 || zz != 1);
+
+   eina_matrix3_values_set(&m1,
+   1, 0, 0,
+   0, 1, 0,
+   0, 0, 1);
+   eina_matrix3_rotate(&m1, M_PI/2);
+
+   fail_if (round(m1.xx) != 0 || round(m1.xy) != -1 ||
+round(m1.xz) != 0 || round(m1.yx) != 1 ||
+round(m1.yy) != 0 || round(m1.yz) !=0 ||
+round(m1.zx) != 0 || round(m1.zy) != 0||
+round(m1.zz) != 1);
+
+   eina_matrix3_values_set(&m1,
+   1, 1, 1,
+   1, 1, 1,
+   1, 1, 1);
+   eina_matrix3_identity(&m1);
+   eina_matrix3_values_get(&m1,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+   fail_if(xx != yy ||
+   yy != zz ||
+   zz != 1);
+
+   fail_if(xy != xz ||
+   yx != yz ||
+   zx != zy ||
+   zy != 0);
+
+
+   eina_matrix3_values_set(&m1,
+   1, 2, 1,
+   2, 1, 1,
+   1, 2, 2);
+   ret = eina_matrix3_determinant(&m1);
+   fail_if(ret != -3);
+
+   eina_matrix3_values_set(&m1,
+   3, 3, 3,
+   3, 3, 3,
+   3, 3, 3);
+   eina_matrix3_divide(&m1, 2);
+   eina_matrix3_values_get(&m1,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+   fail_if (xx != xy ||
+xy != xz ||
+yx != yy ||
+yy != yz ||
+zx != zy ||
+zy != zz ||
+zz != 1.5);
+
+   eina_matrix3_values_set(&m1,
+   0, 2, 0,
+   2, 2, 0,
+   2, 1, 2);
+   eina_matrix3_inverse(&m1, &m2);
+   eina_matrix3_values_get(&m2,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+   fail_if (xx !=

[EGIT] [core/efl] master 01/01: eina: add test cases for eina_matrix3 APIs

2015-06-04 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=81b52fb0ec452989e5f17072669bb8a1bb9b40a1

commit 81b52fb0ec452989e5f17072669bb8a1bb9b40a1
Author: Vivek Ellur 
Date:   Thu Jun 4 10:32:01 2015 +0200

eina: add test cases for eina_matrix3 APIs

Summary:
Added test cases for eina_matrix3_values_get, eina_matrix3_values_set,
eina_matrix3_equal, eina_matrix3_type_get APIs

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2623

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_matrix.c | 51 +++
 1 file changed, 51 insertions(+)

diff --git a/src/tests/eina/eina_test_matrix.c 
b/src/tests/eina/eina_test_matrix.c
index 046876d..28983e1 100644
--- a/src/tests/eina/eina_test_matrix.c
+++ b/src/tests/eina/eina_test_matrix.c
@@ -92,9 +92,60 @@ START_TEST(eina_matrix4_2_3)
 }
 END_TEST
 
+START_TEST(eina_matrix3)
+{
+   Eina_Bool ret;
+   Eina_Matrix3 m, m1, m2;
+   double xx, xy, xz,
+  yx, yy, yz,
+  zx, zy, zz;
+
+  eina_init();
+
+   eina_matrix3_values_set(&m,
+   1, 0, 0,
+   0, 1, 0,
+   0, 0, 1);
+
+   eina_matrix3_values_set(&m1,
+   1, 0, 0,
+   0, 1, 0,
+   0, 0, 1);
+
+   eina_matrix3_values_set(&m2,
+   1, 1, 1,
+   0, 1, 0,
+   0, 0, 1);
+   fail_if(eina_matrix3_type_get(&m) != EINA_MATRIX_TYPE_IDENTITY);
+
+   eina_matrix3_values_get(&m,
+   &xx, &xy, &xz,
+   &yx, &yy, &yz,
+   &zx, &zy, &zz);
+
+   fail_if(xx != yy ||
+   yy != zz ||
+   zz != 1);
+
+   fail_if(xy != xz ||
+   yx != yz ||
+   zx != zy ||
+   zy != 0);
+
+   ret = eina_matrix3_equal(&m, &m1);
+   fail_if(ret != EINA_TRUE);
+
+   ret = eina_matrix3_equal(&m1, &m2);
+   fail_if(ret != EINA_FALSE);
+
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_matrix(TCase *tc)
 {
tcase_add_test(tc, eina_matrix4);
tcase_add_test(tc, eina_matrix4_2_3);
+   tcase_add_test(tc, eina_matrix3);
 }

-- 




[EGIT] [core/efl] master 03/08: eina: add test cases for various APIs in eina_quad module.

2015-05-26 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=331068b84a9f2adb1bed55d2c44278315a8de061

commit 331068b84a9f2adb1bed55d2c44278315a8de061
Author: Vivek Ellur 
Date:   Tue May 26 18:31:55 2015 +0200

eina: add test cases for various APIs in eina_quad module.

Summary:
Added test cases for all the APIs in einq_quad module

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2551

Signed-off-by: Cedric BAIL 
---
 src/Makefile_Eina.am|  3 +-
 src/tests/eina/eina_suite.c |  1 +
 src/tests/eina/eina_suite.h |  1 +
 src/tests/eina/eina_test_quad.c | 64 +
 4 files changed, 68 insertions(+), 1 deletion(-)

diff --git a/src/Makefile_Eina.am b/src/Makefile_Eina.am
index 3a181b6..532c051 100644
--- a/src/Makefile_Eina.am
+++ b/src/Makefile_Eina.am
@@ -309,7 +309,8 @@ tests/eina/eina_test_tmpstr.c \
 tests/eina/eina_test_trash.c \
 tests/eina/eina_test_lock.c \
 tests/eina/eina_test_xattr.c \
-tests/eina/eina_test_crc.c
+tests/eina/eina_test_crc.c \
+tests/eina/eina_test_quad.c
 # tests/eina/eina_test_model.c
 
 
diff --git a/src/tests/eina/eina_suite.c b/src/tests/eina/eina_suite.c
index 1a08480..17706cf 100644
--- a/src/tests/eina/eina_suite.c
+++ b/src/tests/eina/eina_suite.c
@@ -80,6 +80,7 @@ static const Eina_Test_Case etc[] = {
{ "Xattr", eina_test_xattr },
 #endif
{"Crc", eina_test_crc },
+   {"Quad", eina_test_quad },
{ NULL, NULL }
 };
 
diff --git a/src/tests/eina/eina_suite.h b/src/tests/eina/eina_suite.h
index 63d76cd..9f3df3a 100644
--- a/src/tests/eina/eina_suite.h
+++ b/src/tests/eina/eina_suite.h
@@ -65,5 +65,6 @@ void eina_test_abi(TCase *tc);
 void eina_test_trash(TCase *tc);
 void eina_test_xattr(TCase *tc);
 void eina_test_crc(TCase *tc);
+void eina_test_quad(TCase *tc);
 
 #endif /* EINA_SUITE_H_ */
diff --git a/src/tests/eina/eina_test_quad.c b/src/tests/eina/eina_test_quad.c
new file mode 100644
index 000..eb1638f
--- /dev/null
+++ b/src/tests/eina/eina_test_quad.c
@@ -0,0 +1,64 @@
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
+
+#include 
+#include 
+
+#include "eina_suite.h"
+#include "Eina.h"
+
+START_TEST(eina_quad_simple)
+{
+   Eina_Rectangle r1;
+   Eina_Quad r2;
+   double x0, x1, x2, x3;
+   double y0, y1, y2, y3;
+
+   fail_if(!eina_init());
+
+   eina_quad_coords_set(&r2,
+0.0, 5.0,
+5.0, 0.0,
+   10.0, 5.0,
+5.0, 10.0);
+
+   eina_quad_coords_get(&r2,
+&x0, &y0,
+&x1, &y1,
+&x2, &y2,
+&x3, &y3);
+   fail_if (r2.x0 != 0.0 || r2.y0 != 5.0
+   || r2.x1 != 5.0 || r2.y1 != 0.0
+   || r2.x2 != 10.0 || r2.y2 != 5.0
+   || r2.x3 != 5.0 || r2.y3 != 10.0);
+
+   eina_quad_rectangle_to(&r2, &r1);
+   fail_if (r1.x != 0 || r1.y != 0
+   || r1.w != 10 || r1.h != 10);
+
+   eina_quad_coords_set(&r2,
+0.0, 0.0,
+0.0, 0.0,
+0.0, 0.0,
+0.0, 0.0);
+   eina_quad_rectangle_to(&r2, &r1);
+   fail_if (r1.x != 0 || r1.y != 0
+   || r1.w != 0 || r1.h != 0);
+
+   EINA_RECTANGLE_SET(&r1, 5, 10, 20, 30);
+   eina_quad_rectangle_from(&r2, &r1);
+   fail_if (r2.x0 != 5.0 || r2.y0 != 10.0
+   || r2.x1 != 25.0 || r2.y1 != 10.0
+   || r2.x2 != 25.0 || r2.y2 != 40.0
+   || r2.x3 != 5.0 || r2.y3 != 40.0);
+
+   eina_shutdown();
+}
+END_TEST
+
+void
+eina_test_quad(TCase *tc)
+{
+   tcase_add_test(tc, eina_quad_simple);
+}

-- 




[EGIT] [core/efl] master 05/05: eina: add tests cases for eina_rectangle and eina_str functions

2015-05-19 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=6c33bbfa807e3935ba36d9956c24fd8a93d928fa

commit 6c33bbfa807e3935ba36d9956c24fd8a93d928fa
Author: Vivek Ellur 
Date:   Tue May 19 14:13:17 2015 +0200

eina: add tests cases for eina_rectangle and eina_str functions

Summary:
Added test cases for eina_rectangle_union and eina_streq functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2541

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_rectangle.c | 36 +---
 src/tests/eina/eina_test_str.c   |  7 +++
 2 files changed, 40 insertions(+), 3 deletions(-)

diff --git a/src/tests/eina/eina_test_rectangle.c 
b/src/tests/eina/eina_test_rectangle.c
index e5a211f..089b68b 100644
--- a/src/tests/eina/eina_test_rectangle.c
+++ b/src/tests/eina/eina_test_rectangle.c
@@ -72,9 +72,9 @@ START_TEST(eina_rectangle_pool)
 }
 END_TEST
 
-START_TEST(eina_rectangle_intersect)
+START_TEST(eina_rectangle_union_intersect)
 {
-   Eina_Rectangle r1, r2, r3, r4, rd;
+   Eina_Rectangle r1, r2, r3, r4, r5, r6, rd;
 
fail_if(!eina_init());
 
@@ -82,6 +82,8 @@ START_TEST(eina_rectangle_intersect)
EINA_RECTANGLE_SET(&r2, 20, 20, 20, 20);
EINA_RECTANGLE_SET(&r3, 0,   0, 10, 10);
EINA_RECTANGLE_SET(&r4, 30, 30, 50, 50);
+   EINA_RECTANGLE_SET(&r5, 10, 10, 0, 0);
+   EINA_RECTANGLE_SET(&r6, 30, 30, 0, 0);
 
rd = r1;
 
@@ -102,6 +104,34 @@ START_TEST(eina_rectangle_intersect)
|| rd.w != 30
|| rd.h != 30);
 
+   rd = r1;
+   eina_rectangle_union(&rd, &r2);
+   fail_if(rd.x != r1.x
+   || rd.y != r1.y
+   || rd.w != r1.w
+   || rd.h != r1.h);
+
+   rd = r1;
+   eina_rectangle_union(&rd, &r3);
+   fail_if(rd.x != 0
+   || rd.y != 0
+   || rd.w != 60
+   || rd.h != 60);
+
+   rd = r3;
+   eina_rectangle_union(&rd, &r4);
+   fail_if(rd.x != 0
+   || rd.y != 0
+   || rd.w != 80
+   || rd.h != 80);
+
+   rd = r5;
+   eina_rectangle_union(&rd, &r6);
+   fail_if(rd.x != 10
+   || rd.y != 10
+   || rd.w != 20
+   || rd.h != 20);
+
eina_shutdown();
 }
 END_TEST
@@ -110,6 +140,6 @@ void
 eina_test_rectangle(TCase *tc)
 {
tcase_add_test(tc, eina_rectangle_pool);
-   tcase_add_test(tc, eina_rectangle_intersect);
+   tcase_add_test(tc, eina_rectangle_union_intersect);
 }
 
diff --git a/src/tests/eina/eina_test_str.c b/src/tests/eina/eina_test_str.c
index be879db..8233dd8 100644
--- a/src/tests/eina/eina_test_str.c
+++ b/src/tests/eina/eina_test_str.c
@@ -80,6 +80,13 @@
fail_if(eina_str_has_extension("xab", "xYz"));
fail_if(eina_str_has_extension("", "x"));
 
+   fail_if(eina_streq("xab", NULL));
+   fail_if(eina_streq(NULL, "xab"));
+   fail_if(eina_streq("x", "xab"));
+   fail_if(eina_streq("xab", "XAB"));
+   fail_if(eina_streq("x", "x "));
+   fail_if(!eina_streq("xab", "xab"));
+
fail_if(eina_strlen_bounded("abc", 1024) != strlen("abc"));
fail_if(eina_strlen_bounded("abc", 2) != (size_t)-1);
 

-- 




[EGIT] [core/efl] master 01/02: eina: add test cases for various eina_inarray functions

2015-05-14 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=c3a644acff12751891ef0ee78c1bba5106c7d9a4

commit c3a644acff12751891ef0ee78c1bba5106c7d9a4
Author: Vivek Ellur 
Date:   Thu May 14 19:36:25 2015 +0200

eina: add test cases for various eina_inarray functions

Summary:
Added test cases for eina_inarray_search, eina_inarray_search_sorted,
eina_inarray_foreach_remove and other functions

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Reviewed By: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2504

Signed-off-by: Cedric BAIL 
---
 src/tests/eina/eina_test_inarray.c | 105 +
 1 file changed, 105 insertions(+)

diff --git a/src/tests/eina/eina_test_inarray.c 
b/src/tests/eina/eina_test_inarray.c
index 5139770..e54a5cf 100644
--- a/src/tests/eina/eina_test_inarray.c
+++ b/src/tests/eina/eina_test_inarray.c
@@ -389,6 +389,110 @@ START_TEST(eina_inarray_test_itr)
 }
 END_TEST
 
+int
+cmp(const void *a, const void *b)
+{
+   return *(int *)a - *(int *)b;
+}
+
+int
+compare(const void *a, const void *b)
+{
+   return *(int *)a > *(int *)b;
+}
+
+static Eina_Bool
+match_cb(const void *array EINA_UNUSED, void *p, void *user_data)
+{
+   int *member = p;
+   int *i = user_data;
+
+   if (*i == *member)
+ return EINA_TRUE;
+   else
+ return EINA_FALSE;
+}
+
+START_TEST(eina_inarray_test_search)
+{
+   Eina_Inarray *iarr;
+   int i, ret, temp=92, ret1;
+   int arr[] = {91, 95, 97, 93, 92, 94};
+   int arr_size = sizeof(arr)/sizeof(arr[0]);
+   unsigned int curr_len;
+   Eina_Bool rval;
+
+   eina_init();
+
+   iarr = eina_inarray_new(sizeof(int), 0);
+
+   //search in empty array
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   fail_if(ret != -1);
+
+   //search in single element array
+   temp = 91;
+   eina_inarray_push(iarr, &arr[0]);
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   fail_if(ret != 0);
+
+   temp = 92;
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   fail_if(ret != -1);
+
+   for (i = 1; i < arr_size; ++i)
+ {
+ret = eina_inarray_push(iarr, &arr[i]);
+fail_unless(ret == i);
+ }
+   temp = 92;
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   fail_if(ret != 4);
+
+   temp = 100;
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   fail_if(ret != -1);
+
+   rval = eina_inarray_replace_at(iarr, 3, &temp);
+   fail_if(rval != EINA_TRUE);
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   fail_if(ret != 3);
+
+   rval = eina_inarray_replace_at(iarr, arr_size+1, &temp);
+   fail_if(rval != EINA_FALSE);
+
+   ret = eina_inarray_remove(iarr, &temp);
+   fail_if(ret != 3);
+
+   temp = 101;
+   ret = eina_inarray_remove(iarr, &temp);
+   fail_if(ret != -1);
+
+   eina_inarray_sort(iarr, compare);
+
+   temp = 92;
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   ret1 = eina_inarray_search_sorted(iarr, &temp, cmp);
+   fail_if(ret != ret1);
+
+   temp = 100;
+   ret = eina_inarray_search(iarr, &temp, cmp);
+   ret1 = eina_inarray_search_sorted(iarr, &temp, cmp);
+   fail_if(ret != ret1);
+
+   temp = 105;
+   ret = eina_inarray_push(iarr, &temp);
+   ret = eina_inarray_push(iarr, &temp);
+   ret = eina_inarray_push(iarr, &temp);
+   curr_len = iarr->len;
+   rval = eina_inarray_foreach_remove(iarr, match_cb, &temp);
+   fail_if(iarr->len != curr_len-3);
+
+   eina_inarray_free(iarr);
+   eina_shutdown();
+}
+END_TEST
+
 void
 eina_test_inarray(TCase *tc)
 {
@@ -398,4 +502,5 @@ eina_test_inarray(TCase *tc)
tcase_add_test(tc, eina_inarray_test_sort);
tcase_add_test(tc, eina_inarray_test_reverse);
tcase_add_test(tc, eina_inarray_test_itr);
+   tcase_add_test(tc, eina_inarray_test_search);
 }

-- 




[EGIT] [core/efl] master 02/02: eina: fix eina_inarray_search to do linear search

2015-05-14 Thread Vivek Ellur
cedric pushed a commit to branch master.

http://git.enlightenment.org/core/efl.git/commit/?id=756ed9ccbabc202027ab2c15f25f48483e800b86

commit 756ed9ccbabc202027ab2c15f25f48483e800b86
Author: Vivek Ellur 
Date:   Thu May 14 19:37:06 2015 +0200

eina: fix eina_inarray_search to do linear search

Summary:
Currently eina_inarray_search was using binary search to search elements
which would not work on unsorted array so modified it to work as linear 
search.
There is already a function eina_inarray_search_sorted to work on sorted 
array.

Signed-off-by: Vivek Ellur 

Reviewers: cedric

Subscribers: cedric

Differential Revision: https://phab.enlightenment.org/D2501

Signed-off-by: Cedric BAIL 
---
 src/lib/eina/eina_inarray.c | 21 -
 1 file changed, 16 insertions(+), 5 deletions(-)

diff --git a/src/lib/eina/eina_inarray.c b/src/lib/eina/eina_inarray.c
index 2d52e6f..a216f02 100644
--- a/src/lib/eina/eina_inarray.c
+++ b/src/lib/eina/eina_inarray.c
@@ -150,12 +150,23 @@ _eina_inarray_get(const Eina_Inarray *array, unsigned int 
position)
 static int
 _eina_inarray_search(const Eina_Inarray *array, const void *data, 
Eina_Compare_Cb compare)
 {
-   const unsigned char *start, *found;
-   start = array->members;
-   found = bsearch(data, start, array->len, array->member_size, compare);
-   if (!found)
+   unsigned int found, pos=-1;
+   void *p;
+
+   if (array->len == 0)
+ return -1;
+
+   for (pos = 0; pos < array->len; ++pos)
+ {
+p = _eina_inarray_get(array, pos);
+found = compare(data, p);
+if (found == 0)
+  break;
+ }
+   if (pos < array->len)
+ return pos;
+   else
  return -1;
-   return (found - start) / array->member_size;
 }
 
 static unsigned int

--