API Description optimize
authorVBS <vdtizen.sds@samsung.com>
Thu, 17 Oct 2024 06:36:32 +0000 (14:36 +0800)
committerVBS <vdtizen.sds@samsung.com>
Thu, 17 Oct 2024 06:36:32 +0000 (14:36 +0800)
Signed-off-by: VBS <vdtizen.sds@samsung.com>
include/esplusplayer/decodedvideopacketex.h [changed mode: 0644->0755]
include/esplusplayer/elementary_stream.h
include/esplusplayer/es_eventlistener.h [changed mode: 0644->0755]
include/esplusplayer/espacket.h [changed mode: 0644->0755]
include/esplusplayer/esplusplayer.h [changed mode: 0644->0755]
include/esplusplayer/external_drm.h [changed mode: 0644->0755]
include/esplusplayer/track.h [changed mode: 0644->0755]
include/esplusplayer_capi/esplusplayer_capi.h
include/esplusplayer_capi/esplusplayer_internal.h [changed mode: 0644->0755]
include/mixer/mixer.h [changed mode: 0644->0755]
include/mixer_capi/mixer_capi.h [changed mode: 0644->0755]

old mode 100644 (file)
new mode 100755 (executable)
index 26f69c0..66bf9bf
@@ -36,21 +36,61 @@ namespace esplusplayer {
 class DecodedVideoPacketEx : private boost::noncopyable {
  public:
   using Ptr = std::unique_ptr<DecodedVideoPacketEx>;
-
+  /**
+   * @brief      Create a new DecodedVideoPacketEx object.
+   * @param     [in] pts : timestamp of decoded packet.
+   * @param     [in] duration : duration of decoded packet.
+   * @param     [in] surface_data : surface data address which store the decoded data.
+   * @param     [in] scaler_index : the scaler index.
+   * @exception  None
+   * @return     DecodedVideoPacketEx object. 
+   */
   static Ptr Create(const uint64_t pts = 0, const uint64_t duration = 0,
                     tbm_surface_h surface_data = nullptr,
                     const void* scaler_index = nullptr);
-
+  /**
+   * @brief     Constructor of DecodedVideoPacketEx without params.
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       DecodedVideoPacketEx::Create()
+   */
   DecodedVideoPacketEx() = delete;
-
+  /**
+   * @brief     Destructor of DecodedVideoPacketEx object
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       DecodedVideoPacketEx::Create()
+   */
   virtual ~DecodedVideoPacketEx();
-
+  /**
+   * @brief     Get pts of decoded packet.
+   */
   uint64_t GetPts() const { return pts_; }
+  /**
+   * @brief     Get duration of decoded packet.
+   */
   uint64_t GetDuration() const { return duration_; }
+  /**
+   * @brief     Get tbm surface data of decoded packet.
+   */
   const tbm_surface_h GetTbmSurface() const { return surface_data_; }
+  /**
+   * @brief     Get scaler index of decoded packet.
+   */
   const void* GetScalerIndex() const { return scaler_index_; }
 
  protected:
+  /**
+   * @brief  Constructor of structure DecodedVideoPacketEx.
+   * @param     [in] pts : timestamp of decoded packet.
+   * @param     [in] duration : duration of decoded packet.
+   * @param     [in] surface_data : surface data address which store the decoded data.
+   * @param     [in] scaler_index : the scaler index.
+   */
   explicit DecodedVideoPacketEx(const uint64_t pts, const uint64_t duration,
                                 tbm_surface_h surface_data,
                                 const void* scaler_index)
index 9034f8bf421d154e58e0e740df51553bea492540..3412800355bc8eaebbbb6f2cb41dfa23cd0b22ad 100755 (executable)
@@ -98,9 +98,23 @@ class AudioStream : private boost::noncopyable {
    * @return    audio stream object (unique_ptr<AudioStream>)
    */
   static Ptr Create() { return Ptr(new AudioStream); }
-
+  /**
+   * @brief     Constructor of AudioStream without params.
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       AudioStream::Create()
+   */
   AudioStream() noexcept;
-
+  /**
+   * @brief     Deonstructor of AudioStream.
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       AudioStream::Create()
+   */
   ~AudioStream() {}
   /**
    * @brief     Set mime type for the associated audio stream
@@ -213,8 +227,23 @@ class VideoStream : private boost::noncopyable {
    */
   static Ptr Create() { return Ptr(new VideoStream); }
 
+  /**
+   * @brief     Constructor of VideoStream without params
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       VideoStream::Create()
+   */
   VideoStream() noexcept;
-
+  /**
+   * @brief     Destructor of VideoStream
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       VideoStream::Create()
+   */
   ~VideoStream() {}
 
   /**
old mode 100644 (file)
new mode 100755 (executable)
index 95a0d32..2263b96
@@ -42,8 +42,22 @@ namespace esplusplayer {
  */
 class EsEventListener {
  public:
+  /**
+   * @brief     Constructor of EsEventListener object
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   */
   EsEventListener() noexcept {}
   using UserData = void*;
+  /**
+   * @brief     Destructor of EsEventListener object
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   */
   virtual ~EsEventListener() {}
 // LCOV_EXCL_START
   /**
@@ -168,10 +182,17 @@ class EsEventListener {
    * @see       EsPlusPlayer::Flush()
    */
   virtual void OnFlushDone(UserData userdata) {}
-
+  /**
+   * @brief     It will be invoked when player gets closed caption data from
+   *            decoder.
+   * @param     [in] event_type : the type of event.
+   * @param     [in] msg_data : the event message data.
+   */
   virtual void OnEvent(const EventType& event_type, const EventMsg& msg_data,
                        UserData userdata) {}
-
+  /**
+   * @brief     It will be invoked when the first frame decode out.
+   */
   virtual void OnFirstDecodingDone(UserData userdata) {}
 
   /**
old mode 100644 (file)
new mode 100755 (executable)
index 979fd22..04220b5
@@ -92,7 +92,14 @@ class EsPacket : private boost::noncopyable {
    */
   static Ptr CreateEos(const StreamType type = StreamType::kMax);
   EsPacket() = delete;
-
+  /**
+   * @brief     Destructor of EsPacket object
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       EsPacket::Create()
+   */
   ~EsPacket() {}
   /**
    * @brief     Get the stream type for the associated packet
@@ -165,6 +172,17 @@ class EsPacket : private boost::noncopyable {
   bool IsEosPacket() const { return buffer_size_ == 0 && buffer_ == nullptr; }
 
  private:
+  /**
+   * @brief     Constructor of EsPacket object.
+   * @pre       None
+   * @post      None
+   * @param     [in] type : stream type of packet.
+   * @param     [in] buffer_size : buffer size of packet.
+   * @param     [in] pts : timestamp of this packet.
+   * @param     [in] duration : duration of this packet.
+   * @return    None
+   * @see       EsPacket::Create()
+   */
   explicit EsPacket(const StreamType type, std::shared_ptr<char> buffer,
                     const uint32_t buffer_size, const uint64_t pts,
                     const uint64_t duration, const uint32_t hdr10p_size,
old mode 100644 (file)
new mode 100755 (executable)
index 756cdcb..593af67
@@ -253,6 +253,14 @@ class EsPlusPlayer : private boost::noncopyable {
   static Ptr Create();
 
  public:
+  /**
+   * @brief     Destructor of EsPlusPlayer object
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       EsPlusPlayer::Create()
+   */
   virtual ~EsPlusPlayer() {}
   /**
    * @brief     Make player get ready to set playback mode
@@ -361,7 +369,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual bool PrepareAsync() { return false; }
   /**
-   * @brief     Start playback
+   * @brief     Start a media content playback.
    * @pre       The player state should be #EsState::kReady
    * @post      The player state will be #EsState::kPlaying
    * @return    @c True on success, otherwise @c False
@@ -398,7 +406,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual bool Pause() { return false; }
   /**
-   * @brief     Resume a media content
+   * @brief     Resume a media content after pause.
    * @pre       The player state must be one of #EsState::kPlaying or
    * #EsState::kPaused
    * @post      The player state will be #EsState::kPlaying
@@ -408,7 +416,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual bool Resume() { return false; }
   /**
-   * @brief     SetAppInfo
+   * @brief     Set the app info of player.
    * @remarks   Set app_id to resource manager. Resource manager check the
    * priority to control resource.
    * @param     [in] app_info : application id, version, type
@@ -417,7 +425,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual void SetAppInfo(const PlayerAppInfo& app_info) { return; }
   /**
-   * @brief     SetAppInfoEx
+   * @brief     Set more app info of player.
    * @remarks   Set app_id to resource manager. Resource manager check the
    * priority to control resource.
    * @param     [in] app_info : application id, version, type, runtitle
@@ -426,7 +434,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual void SetAppInfoEx(const PlayerAppInfoEx& app_info) { return; }
   /**
-   * @brief     SetPlaybackRate.
+   * @brief     Set the playback rate of player.
    * @remarks   Set playback rate from 0.0 to 2.0.
    * @param     [in] rate : The playback rate from 0.0 to 2.0. EsPlayer isn't
    * support trick play.
@@ -455,7 +463,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual bool Seek(const uint64_t time) { return false; }
   /**
-   * @brief     Set the video display
+   * @brief     Set the video display with obj.
    * @remarks   We are not supporting changing display.
    * @remarks   This API have to be called before calling the
    * EsPlusPlayer::PrepareAsync() to reflect the display type.
@@ -471,7 +479,7 @@ class EsPlusPlayer : private boost::noncopyable {
   virtual bool SetDisplay(const DisplayType& type, void* obj) { return false; }
 #ifdef USE_MIXER
   /**
-   * @brief     Set the video display
+   * @brief     Set the video display of mixer player.
    * @remarks   We are not supporting changing display.
    * @remarks   This API have to be called before calling the
    * PlusPlayer::Prepare()
@@ -491,7 +499,7 @@ class EsPlusPlayer : private boost::noncopyable {
   }
 #endif
   /**
-   * @brief     Set the video display
+   * @brief     Set the video display with ecore window.
    * @remarks   We are not supporting changing display.
    * @remarks   This API have to be called before calling the
    * EsPlusPlayer::PrepareAsync() to reflect the display type.
@@ -536,7 +544,7 @@ class EsPlusPlayer : private boost::noncopyable {
     return false;
   }
   /**
-   * @brief     Set the video display
+   * @brief     Set the video display with surface id.
    * @remarks   We are not supporting changing display.
    * @remarks   This API have to be called before calling the
    * EsPlusPlayer::PrepareAsync() to reflect the display type.
@@ -866,14 +874,14 @@ class EsPlusPlayer : private boost::noncopyable {
     return;
   }
   /**
-   * @brief     Set volume to player
+   * @brief     Set volume value of player
    * @param     [in] volume : volume level
    * @pre       The player state can be all of #EsState except #EsState::kNone
    * @return       @c True on success, otherwise @c False
    */
   virtual bool SetVolume(const int& volume) { return false; }
   /**
-   * @brief     Get volume to player
+   * @brief     Get volume value of player
    * @param     [out] volume : volume ptr
    * @pre       The player state can be all of #EsState except #EsState::kNone
    * @return       @c True on success, otherwise @c False
@@ -887,7 +895,7 @@ class EsPlusPlayer : private boost::noncopyable {
    */
   virtual bool Flush(const StreamType& type) { return false; }
   /**
-   * @brief     Set the buffer size
+   * @brief     Set the type of buffer size.
    * @param     [in] option : A type of Buffer Option
    * @pre       The player state must be set to #EsState::kIdle
    * @return       @c True on success, otherwise @c False
@@ -1196,7 +1204,7 @@ class EsPlusPlayer : private boost::noncopyable {
   virtual bool StopAudioEasing() { return false; }
 
   /**
-   * @brief     Get virtual resource id
+   * @brief     Get virtual render resource id.
    * @param     [in] type : The resource type of virtual id.
    * @param     [out] virtual_id : Stored virtual resource id value.
    * @pre       The player state should be #EsState::kReady, #EsState::kPlaying
@@ -1337,6 +1345,14 @@ class EsPlusPlayer : private boost::noncopyable {
   // LCOV_EXCL_STOP
 
  protected:
+  /**
+   * @brief     Constructor of EsPlusPlayer object
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   * @see       EsPlusPlayer::Create()
+   */
   EsPlusPlayer() noexcept {};
 };  // class EsPlusPlayer
 
old mode 100644 (file)
new mode 100755 (executable)
index 44ababb..a2936d1
@@ -68,6 +68,9 @@ enum class DrmbEsCipherPhase : int {
  * @brief  Structure of subsample information for drm
  */
 struct DrmbEsSubSampleInfo {
+  /**
+   * @brief  Constructor of structure DrmbEsSubSampleInfo.
+   */
   explicit DrmbEsSubSampleInfo(const uint32_t _bytes_of_clear_data,
                                const uint32_t _bytes_of_encrypted_data)
       : bytes_of_clear_data(_bytes_of_clear_data),
old mode 100644 (file)
new mode 100755 (executable)
index 05d38e6..28a3c78
@@ -73,7 +73,24 @@ struct Track {
   bool use_swdecoder = false;
   std::string language_code;
   std::string subtitle_format;
+  /**
+   * @brief     Constructor of Track
+   * @pre       None
+   * @post      None
+   * @exception None
+   * @return    None
+   */
   Track() {};
+  /**
+   * @brief     Constructor of Track with params
+   * @pre       None
+   * @post      None
+   * @param     [in] _index : track index
+   * @param     [in] _mimetype : codec type of video/audio
+   * @param     [in] _streamtype : audio/video/subtile
+   * @exception None
+   * @return    None
+   */
   Track(int _index, int _id, std::string _mimetype, std::string _streamtype, std::string _container_type,
     TrackType _type, std::shared_ptr<char> _codec_data, unsigned int _codec_tag,  int _codec_data_len,
     int _width, int _height, int _maxwidth, int _maxheight, int _framerate_num, int _framerate_den,
@@ -136,6 +153,17 @@ enum class SubtitleType {
 };
 
 struct SubtitleAttr {
+  /**
+   * @brief     Constructor of SubtitleAttr with params
+   * @pre       None
+   * @post      None
+   * @param     [in] _type : see SubtitleAttrType, set attribute such as pos, width,height, size and so on.
+   * @param     [in] _start_time : start time of the attr value.
+   * @param     [in] _stop_time : stop time of the attr value.
+   * @param     [in] _value : value of the attr value.
+   * @exception None
+   * @return    None
+   */
   explicit SubtitleAttr(const SubtitleAttrType _type,
                         const uint32_t _start_time, const uint32_t _stop_time,
                         const boost::any _value, const int _extsub_index)
index 4f26314596f8988e50a31d9dd68bd60dae452f5b..649a19d905ca9e0e5a432acde965a00d3bb277b1 100755 (executable)
@@ -545,7 +545,7 @@ int esplusplayer_open(esplusplayer_handle handle);
 int esplusplayer_close(esplusplayer_handle handle);
 
 /**
- * @brief     Release player handle.
+ * @brief     Release a player handle after finish playing.
  * @param     [in] handle : esplusplayer handle.
  * @return    @c ESPLUSPLAYER_ERROR_TYPE_NONE on success,otherwise @c one of
  * esplusplayer_error_type
@@ -759,7 +759,7 @@ int esplusplayer_activate_audio(esplusplayer_handle handle);
 int esplusplayer_prepare_async(esplusplayer_handle handle);
 
 /**
- * @brief     Start playback.
+ * @brief     Start playback after prepare done.
  * @param     [in] handle : esplusplayer handle.
  * @return    @c ESPLUSPLAYER_ERROR_TYPE_NONE on success,otherwise @c one of
  * esplusplayer_error_type
@@ -872,7 +872,7 @@ int esplusplayer_pause(esplusplayer_handle handle);
 int esplusplayer_resume(esplusplayer_handle handle);
 
 /**
- * @brief     Set playback rate.
+ * @brief     Set the video playback rate.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] playback_rate :  the playback rate from 0.0 to 2.0.
  * @param     [in] audio_mute :  the audio is mute on/off, true: mute on, false:
@@ -974,7 +974,7 @@ int esplusplayer_set_app_info(esplusplayer_handle handle,
                               const esplusplayer_app_info* app_info);
 
 /**
- * @brief     Set the video display.
+ * @brief     Set the video display with window object.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] type : display type.
  * @param     [in] window : the handle to display window.
@@ -1005,7 +1005,7 @@ int esplusplayer_set_app_info(esplusplayer_handle handle,
 int esplusplayer_set_display(esplusplayer_handle handle,
                              esplusplayer_display_type type, void* window);
 /**
- * @brief     Set the video display.
+ * @brief     Set the video display with subsurface.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] type : display type.
  * @param     [in] subsurface : the ecore wayland subsurface handle.
@@ -1027,7 +1027,7 @@ int esplusplayer_set_display_ecore_subsurface(esplusplayer_handle handle,
                                               void* subsurface, int x, int y,
                                               int width, int height);
 /**
- * @brief     Set the video display.
+ * @brief     Set the video display with window position.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] type : display type.
  * @param     [in] window : the ecore wayland2 window handle.
@@ -1764,7 +1764,7 @@ int esplusplayer_get_adaptive_info(
     esplusplayer_adaptive_info_type adaptive_type);
 
 /**
- * @brief     Set volume to player
+ * @brief     Set volume value to player.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] volume : volume level(0 ~ 100).
  * @return    @c ESPLUSPLAYER_ERROR_TYPE_NONE on success,otherwise @c one of
@@ -1789,7 +1789,7 @@ int esplusplayer_get_adaptive_info(
 int esplusplayer_set_volume(esplusplayer_handle handle, const int volume);
 
 /**
- * @brief     Get volume from player
+ * @brief     Get volume value from player.
  * @param     [in] handle : esplusplayer handle.
  * @param     [out] volume : volume ptr.
  * @return    @c ESPLUSPLAYER_ERROR_TYPE_NONE on success,otherwise @c one of
@@ -3464,7 +3464,7 @@ int esplusplayer_get_audio_easing_info(
 int esplusplayer_start_audio_easing(esplusplayer_handle handle);
 
 /**
- * @brief     Stop audio easing
+ * @brief     Stop an audio easing handle which have been started.
  * @param     [in] handle : esplusplayer handle.
  * @return    @c ESPLUSPLAYER_ERROR_TYPE_NONE on success,otherwise @c one of
  * esplusplayer_error_type
@@ -3493,7 +3493,7 @@ int esplusplayer_start_audio_easing(esplusplayer_handle handle);
 int esplusplayer_stop_audio_easing(esplusplayer_handle handle);
 
 /**
- * @brief     Get virtual resource id
+ * @brief     Get virtual render resource id of player.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] type : The resource type of virtual id.
  * @param     [out] virtual_id : Stored virtual resource id value.
@@ -3712,7 +3712,7 @@ int esplusplayer_set_video_frame_dropped_cb(
     esplusplayer_video_frame_dropped_cb video_frame_dropped_cb, void* userdata);
 
 /**
- * @brief     Set video scan type.
+ * @brief     Set video scan type(progressive/interlaced).
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] type : The video scan type.
  * @pre       The player state must be set to #ESPLUSPLAYER_STATE_IDLE.
old mode 100644 (file)
new mode 100755 (executable)
index e2109e9..be89e0e
@@ -42,7 +42,7 @@ typedef void* esplusplayer_handle;
 typedef void (*esplusplayer_first_video_decoding_done_cb)(void*);
 
 /**
- * @brief     Set the video display.
+ * @brief     Set the video display by surface id.
  * @param     [in] handle : esplusplayer handle.
  * @param     [in] type : display type.
  * @param     [in] surface_id : resource id of window.
@@ -63,7 +63,18 @@ int esplusplayer_set_surface_display(esplusplayer_handle handle,
                                      esplusplayer_display_type type,
                                      unsigned int surface_id, int x, int y,
                                      int width, int height);
-
+/**
+ * @brief     Set a callback function to be invoked when the first buffer is
+ *            output from a video decoder.
+ * @param     [in] handle : esplusplayer handle.
+ * @param     [in] callback : the callback function to register.
+ * @param     [in] userdata : userdata of esplusplayer_decoder_underrun_cb()
+ * @return    @c one of esplusplayer_error_type values will be returned.
+ * @pre       The player state must be set to #ESPLUSPLAYER_STATE_NONE or
+ *            #ESPLUSPLAYER_STATE_IDLE.
+ * @post      esplusplayer_first_video_decoding_done_cb() will be invoked.
+ * @exception   None
+ */
 int esplusplayer_set_first_video_decoding_done_cb(
     esplusplayer_handle handle,
     esplusplayer_first_video_decoding_done_cb first_video_decoding_done_cb,
old mode 100644 (file)
new mode 100755 (executable)
index 4d74a10..3dffe04
@@ -55,7 +55,7 @@ class Mixer : private boost::noncopyable {
  public:
   using Ptr = std::unique_ptr<Mixer>;
   /**
-   * @brief      Create a mixer object
+   * @brief      Create a new mixer object
    * @remarks    You must use this to get mixer object
    * @pre        None
    * @post       None
@@ -76,7 +76,7 @@ class Mixer : private boost::noncopyable {
 
  public:
   /**
-   * @brief     Destructor of Mixer
+   * @brief     Destructor of Mixer object
    * @pre       None
    * @post      None
    * @exception None
@@ -85,7 +85,7 @@ class Mixer : private boost::noncopyable {
    */
   virtual ~Mixer(){};
   /**
-   * @brief     Starts Mixer
+   * @brief     Starts Mixer when all players are ready.
    * @pre       None
    * @post      Black frame or mixed frame will be displayed on the screen
    * @exception None
@@ -95,7 +95,7 @@ class Mixer : private boost::noncopyable {
 // LCOV_EXCL_START
   virtual bool Start() { return false; }
   /**
-   * @brief     Stops Mixer
+   * @brief     Stops Mixer when playing finished.
    * @pre       Mixer::Start() was called
    * @post      None
    * @exception None
@@ -112,7 +112,7 @@ class Mixer : private boost::noncopyable {
    */
   virtual int GetMaximumAllowedNumberOfPlayer() { return 0; }
   /**
-   * @brief     Sets the video display
+   * @brief     Sets the video display with params
    * @remarks   We are not supporting changing display.
    * @remarks   This API have to be called before calling the
    *            Mixer::Start() to reflect the display type.
@@ -127,7 +127,7 @@ class Mixer : private boost::noncopyable {
    */
   virtual bool SetDisplay(const DisplayType type, void* obj) { return false; }
   /**
-   * @brief     Sets the video display
+   * @brief     Sets the video display with params
    * @remarks   We are not supporting changing display.
    * @remarks   This API have to be called before calling the
    *            Mixer::Start() to reflect the display type.
@@ -258,7 +258,7 @@ class Mixer : private boost::noncopyable {
 
  protected:
   /**
-   * @brief     Constructor of Mixer
+   * @brief     Constructor of Mixer object
    * @pre       None
    * @post      None
    * @exception None
old mode 100644 (file)
new mode 100755 (executable)
index 10796d2..c33f7d5
@@ -53,7 +53,7 @@ typedef enum mixer_rsc_alloc_mode {
 } mixer_rsc_alloc_mode;
 
 /**
- * @brief     Create a mixer handle
+ * @brief     Create a new mixer handle.
  * @param     None
  * @return    return mixer handle pointer
  * @code
@@ -88,7 +88,7 @@ typedef enum mixer_rsc_alloc_mode {
 mixer_handle mixer_create();
 
 /**
- * @brief     Release mixer handle
+ * @brief     Release the handle of mixer player.
  * @param     [in] handle : mixer handle
  * @return    @c one of mixer_error_type values will be returned
  * @pre       None
@@ -99,7 +99,7 @@ mixer_handle mixer_create();
 int mixer_destroy(mixer_handle handle);
 
 /**
- * @brief     Starts Mixer.
+ * @brief     Starts a Mixer video playing.
  * @param     [in] handle : mixer handle
  * @return    @c one of mixer_error_type values will be returned
  * @pre       None
@@ -110,7 +110,7 @@ int mixer_destroy(mixer_handle handle);
 int mixer_start(mixer_handle handle);
 
 /**
- * @brief     Stops Mixer
+ * @brief     Stops a Mixer video playing.
  * @param     [in] handle : Mixer handle
  * @return    @c one of mixer_error_type values will be returned
  * @pre       mixer_start() should be called
@@ -131,9 +131,9 @@ int mixer_stop(mixer_handle handle);
 int mixer_get_max_allowed_number_of_player(mixer_handle handle);
 
 /**
- * @brief     Sets the video display
+ * @brief     Sets the video display of mixer.
  * @param     [in] handle : Mixer handle
- * @param     [in] type : display type
+ * @param     [in] type : mixer display type
  * @param     [in] window : The handle to display window
  * @return    @c one of mixer_error_type values will be returned
  * @pre       This API have to be called before calling the