From 92558b9389d4f5e0549f5d642135c2d00b67c317 Mon Sep 17 00:00:00 2001
From: caro <caro>
Date: Fri, 30 Dec 2011 14:17:56 +0000
Subject: [PATCH] Emotion: use eina_semaphore API and update EFL versions

git-svn-id: http://svn.enlightenment.org/svn/e/trunk/emotion@66696 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33
---
 m4/emotion_check.m4                          |  6 +++---
 src/modules/generic/Emotion_Generic_Plugin.h |  7 +------
 src/modules/generic/emotion_generic.c        | 31 ++++------------------------
 3 files changed, 8 insertions(+), 36 deletions(-)

diff --git a/m4/emotion_check.m4 b/m4/emotion_check.m4
index 8c0114c..8c43a68 100644
--- a/m4/emotion_check.m4
+++ b/m4/emotion_check.m4
@@ -7,7 +7,7 @@ AC_DEFUN([EMOTION_CHECK_DEP_XINE],
 requirement=""
 
 PKG_CHECK_MODULES([XINE],
-   [libxine >= 1.1.1 evas >= 0.9.9],
+   [libxine >= 1.1.1 evas >= 1.0.0],
    [
     have_dep="yes"
     requirement="libxine"
@@ -33,7 +33,7 @@ GST_MAJORMINOR=0.10
 
 requirement=""
 PKG_CHECK_MODULES([GSTREAMER],
-   [gstreamer-$GST_MAJORMINOR >= $GST_REQS gstreamer-plugins-base-$GST_MAJORMINOR >= $GSTPLUG_REQS gstreamer-video-$GST_MAJORMINOR >= $GSTPLUG_REQS evas >= 0.9.9],
+   [gstreamer-$GST_MAJORMINOR >= $GST_REQS gstreamer-plugins-base-$GST_MAJORMINOR >= $GSTPLUG_REQS gstreamer-video-$GST_MAJORMINOR >= $GSTPLUG_REQS evas >= 1.0.0 eina >= 1.1.99],
    [
     have_dep="yes"
     requirement="gstreamer-$GST_MAJORMINOR gstreamer-plugins-base-$GST_MAJORMINOR"
@@ -79,7 +79,7 @@ AC_DEFUN([EMOTION_CHECK_DEP_GENERIC],
 requirement=""
 
 PKG_CHECK_MODULES([GENERIC],
-   [evas >= 0.9.9],
+   [evas >= 1.0.0],
    [
     have_dep="yes"
    ],
diff --git a/src/modules/generic/Emotion_Generic_Plugin.h b/src/modules/generic/Emotion_Generic_Plugin.h
index 40a9b31..655201a 100644
--- a/src/modules/generic/Emotion_Generic_Plugin.h
+++ b/src/modules/generic/Emotion_Generic_Plugin.h
@@ -93,12 +93,7 @@ struct _Emotion_Generic_Video_Shared
 	int last;
 	int next;
    } frame;
-  /* FIXME: maybe abstracting that in Eina ? */
-#ifdef _WIN32
-   HANDLE lock;
-#else
-   sem_t lock;
-#endif
+   Eina_Semaphore lock;
    int frame_drop;
 };
 
diff --git a/src/modules/generic/emotion_generic.c b/src/modules/generic/emotion_generic.c
index ef7a4c2..bf440f4 100644
--- a/src/modules/generic/emotion_generic.c
+++ b/src/modules/generic/emotion_generic.c
@@ -178,17 +178,11 @@ _create_shm_data(Emotion_Generic_Video *ev, const char *shmname)
    vs->frame.last = 2;
    vs->frame.next = 2;
    vs->frame_drop = 0;
-#ifdef _WIN32
-   /* FIXME: maximum count for the semaphore: 10. Is it sufficient ? */
-   vs->lock = CreateSemaphore(NULL, 1, 10, NULL);
-   if (!vs->lock)
+   if (!eina_semaphore_new(&vs->lock, 1))
      {
 	ERR("can not create semaphore");
 	return EINA_FALSE;
      }
-#else
-   sem_init(&vs->lock, 1, 1);
-#endif
    ev->frame.frames[0] = (unsigned char *)vs + sizeof(*vs);
    ev->frame.frames[1] = (unsigned char *)vs + sizeof(*vs) + vs->height * vs->width * vs->pitch;
    ev->frame.frames[2] = (unsigned char *)vs + sizeof(*vs) + 2 * vs->height * vs->width * vs->pitch;
@@ -483,12 +477,7 @@ static void
 _player_file_closed(Emotion_Generic_Video *ev)
 {
    INF("Closed previous file.");
-#ifdef _WIN32
-   CloseHandle(ev->shared->lock);
-#else
-   sem_destroy(&ev->shared->lock);
-#endif
-
+   eina_semaphore_free(&ev->shared->lock);
    ev->closing = EINA_FALSE;
 
    if (ev->opening)
@@ -1257,21 +1246,13 @@ static int
 em_bgra_data_get(void *data, unsigned char **bgra_data)
 {
    Emotion_Generic_Video *ev = data;
-#ifdef _WIN32
-   DWORD res;
-#endif
 
    if (!ev || !ev->file_ready)
      return 0;
 
    // lock frame here
-#ifdef _WIN32
-   res = WaitForSingleObject(ev->shared->lock, 0L);
-   if (res != WAIT_OBJECT_0)
+   if (!eina_semaphore_lock(&ev->shared->lock))
      return 0;
-#else
-   sem_wait(&ev->shared->lock);
-#endif
 
    // send current frame to emotion
    if (ev->shared->frame.emotion != ev->shared->frame.last)
@@ -1286,11 +1267,7 @@ em_bgra_data_get(void *data, unsigned char **bgra_data)
    ev->shared->frame_drop = 0;
 
    // unlock frame here
-#ifdef _WIN32
-   ReleaseSemaphore(ev->shared->lock, 1, NULL);
-#else
-   sem_post(&ev->shared->lock);
-#endif
+   eina_semaphore_release(&ev->shared->lock, 1);
    ev->drop = 0;
 
    return 1;
-- 
2.7.4