From 0ba7d42fed573a1179a3617298db04f6d203d5ae Mon Sep 17 00:00:00 2001
From: Carsten Haitzler <raster@rasterman.com>
Date: Fri, 17 Feb 2012 11:05:17 +0000
Subject: [PATCH] elm video review comments.

SVN revision: 68066
---
 legacy/elementary/src/lib/Elementary.h.in | 2 +-
 legacy/elementary/src/lib/elm_video.h     | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/legacy/elementary/src/lib/Elementary.h.in b/legacy/elementary/src/lib/Elementary.h.in
index e14e3e4964..d044b56206 100644
--- a/legacy/elementary/src/lib/Elementary.h.in
+++ b/legacy/elementary/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/legacy/elementary/src/lib/elm_video.h b/legacy/elementary/src/lib/elm_video.h
index d9f2f3387f..986b655e57 100644
--- a/legacy/elementary/src/lib/elm_video.h
+++ b/legacy/elementary/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);