From: raster <raster@7cbeb6ba-43b4-40fd-8cce-4c39aea84d33>
Date: Fri, 17 Feb 2012 11:05:17 +0000 (+0000)
Subject: elm video review comments.
X-Git-Tag: REL_F_I9500_20120323_1~17^2~665
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=cde28544fd308bf81407c334f3fe1d71dff4a3de;p=framework%2Fuifw%2Felementary.git

elm video review comments.



git-svn-id: svn+ssh://svn.enlightenment.org/var/svn/e/trunk/elementary@68066 7cbeb6ba-43b4-40fd-8cce-4c39aea84d33
---

diff --git a/src/lib/Elementary.h.in b/src/lib/Elementary.h.in
index e14e3e4..d044b56 100644
--- a/src/lib/Elementary.h.in
+++ b/src/lib/Elementary.h.in
@@ -247,7 +247,7 @@ EAPI extern Elm_Version *elm_version;
 #include <elm_thumb.h> // OK
 #include <elm_toolbar.h> // OK
 #include <elm_transit.h> // OK
-#include <elm_video.h>
+#include <elm_video.h> // XXX: needs work. see elm_video.h
 #include <elm_web.h>
 #include <elm_win.h>
 
diff --git a/src/lib/elm_video.h b/src/lib/elm_video.h
index d9f2f33..986b655 100644
--- a/src/lib/elm_video.h
+++ b/src/lib/elm_video.h
@@ -95,6 +95,7 @@ EAPI void                 elm_video_file_set(Evas_Object *video, const char *fil
  *
  * @ingroup Video
  */
+/* XXX: remove this - just keep elm_video_file_set() */
 EAPI void                 elm_video_uri_set(Evas_Object *video, const char *uri);
 
 /**
@@ -151,6 +152,7 @@ EAPI void                 elm_video_stop(Evas_Object *video);
  *
  * @ingroup Video
  */
+/* XXX: elm_video_is_playing_get() */
 EAPI Eina_Bool            elm_video_is_playing(const Evas_Object *video);
 
 /**
@@ -161,6 +163,7 @@ EAPI Eina_Bool            elm_video_is_playing(const Evas_Object *video);
  *
  * @ingroup Video
  */
+/* XXX: elm_video_is_seekable_get() */
 EAPI Eina_Bool            elm_video_is_seekable(const Evas_Object *video);
 
 /**
@@ -203,6 +206,7 @@ EAPI double               elm_video_audio_level_get(const Evas_Object *video);
  */
 EAPI void                 elm_video_audio_level_set(Evas_Object *video, double volume);
 
+/* XXX: need docs for these */
 EAPI double               elm_video_play_position_get(const Evas_Object *video);
 EAPI void                 elm_video_play_position_set(Evas_Object *video, double position);
 EAPI double               elm_video_play_length_get(const Evas_Object *video);