Apply new attribute API of libmm-common 20/192320/3 accepted/tizen/unified/20181128.074044 submit/tizen/20181127.074455
authorSangchul Lee <sc11.lee@samsung.com>
Fri, 2 Nov 2018 07:16:15 +0000 (16:16 +0900)
committerSangchul Lee <sc11.lee@samsung.com>
Wed, 14 Nov 2018 03:06:22 +0000 (12:06 +0900)
- Remove dependency of mm_attrs_private.h.
- Use new API set in mm_attrs.h instead of those.

Change-Id: I235197cfa1d89ea656238709ac25b66ba02af19b
Signed-off-by: Sangchul Lee <sc11.lee@samsung.com>
src/include/mm_streamrecorder_attribute.h
src/include/mm_streamrecorder_internal.h
src/mm_streamrecorder.c
src/mm_streamrecorder_attribute.c

index 9879e7c..8114b00 100644 (file)
@@ -27,7 +27,6 @@
 ========================================================================================*/
 #include <mm_types.h>
 #include <mm_attrs.h>
-#include <mm_attrs_private.h>
 #include <stdarg.h>
 #include "mm_streamrecorder_internal.h"
 
@@ -83,7 +82,7 @@ typedef enum {
 /*=======================================================================================
 | TYPE DEFINITIONS                                                                     |
 ========================================================================================*/
-typedef bool(*mmf_streamrecorder_commit_func_t) (MMHandleType handle, int attr_idx, const mmf_value_t *value);
+typedef bool(*mmf_streamrecorder_commit_func_t) (MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
 /*=======================================================================================
 | STRUCTURE DEFINITIONS                                                                        |
@@ -208,23 +207,23 @@ int _mmstreamrecorder_get_attribute_info(MMHandleType handle, const char *attr_n
  * @see
  *
  */
-bool _mmstreamrecorder_commit_streamrecorder_attrs(int attr_idx, const char *attr_name, const mmf_value_t *value, void *commit_param);
+bool _mmstreamrecorder_commit_streamrecorder_attrs(int attr_idx, const char *attr_name, const MMAttrsValue *value, void *commit_param);
 
-bool _mmstreamrecorder_commit_video_enable(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_video_enable(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_video_encoder(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_video_encoder(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_audio_enable(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_audio_enable(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_audio_encoder(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_audio_encoder(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_audio_samplingrate(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_audio_samplingrate(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_audio_bitformat(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_audio_bitformat(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_audio_channel(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_audio_channel(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
-bool _mmstreamrecorder_commit_audio_bitrate(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_audio_bitrate(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
 /**
  * check whether supported or not
@@ -235,7 +234,7 @@ bool _mmstreamrecorder_commit_audio_bitrate(MMHandleType handle, int attr_idx, c
  */
 bool _mmstreamrecorder_check_supported_attribute(MMHandleType handle, int attr_index);
 
-bool _mmstreamrecorder_commit_video_bitrate(MMHandleType handle, int attr_idx, const mmf_value_t *value);
+bool _mmstreamrecorder_commit_video_bitrate(MMHandleType handle, int attr_idx, const MMAttrsValue *value);
 
 #ifdef __cplusplus
 }
index a29c3cf..61cb31e 100644 (file)
@@ -33,7 +33,6 @@
 
 #include <mm_types.h>
 #include <mm_attrs.h>
-#include <mm_attrs_private.h>
 #include <mm_message.h>
 
 #include "mm_streamrecorder.h"
index ec4c9b6..d2b70b8 100644 (file)
@@ -27,7 +27,6 @@
 
 #include <mm_error.h>
 
-#include <mm_attrs_private.h>
 #include "mm_streamrecorder.h"
 #include "mm_streamrecorder_internal.h"
 #include "mm_streamrecorder_attribute.h"
index e908d0f..f64b0dd 100644 (file)
@@ -44,7 +44,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_BUFFER_TYPE,      /* ID */
         (char *)"videobuffer-type",            /* Name */
-        MMF_VALUE_TYPE_INT,            /* Type */
+        MM_ATTRS_TYPE_INT,             /* Type */
         MM_ATTRS_FLAG_RW,                      /* Flag */
         {(void *)MM_STREAMRECORDER_VIDEO_TYPE_TBM_BO}, /* Default value */
         MM_ATTRS_VALID_TYPE_INT_RANGE, /* Validity type */
@@ -55,7 +55,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        /* 1  */
        {MM_STR_VIDEO_FORMAT,
         (char *)"videosource-format",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)MM_STREAMRECORDER_INPUT_FORMAT_NV12},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -67,7 +67,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_FRAMERATE,
         (char *)"video-framerate",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -79,7 +79,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_ENCODER_BITRATE,
         (char *)"video-bitrate",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -91,7 +91,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_RESOLUTION_WIDTH,
         (char *)"video-resolution-width",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_ARRAY,
@@ -103,7 +103,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_RESOLUTION_HEIGHT,
         (char *)"video-resolution-height",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_ARRAY,
@@ -115,7 +115,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_AUDIO_FORMAT,
         (char *)"audio-source-format",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)MM_STREAMRECORDER_AUDIO_FORMAT_PCM_S16_LE},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -127,7 +127,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_AUDIO_ENCODER_BITRATE,
         (char *)"audio-bitrate",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)128000},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -139,7 +139,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_AUDIO_SAMPLERATE,
         (char *)"audio-samplerate",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -151,7 +151,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_ENCODER,
         (char *)"video-encoder",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_ARRAY,
@@ -163,7 +163,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_AUDIO_ENCODER,
         (char *)"audio-encoder",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_ARRAY,
@@ -175,7 +175,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_AUDIO_CHENNEL_COUNT,
         (char *)"audio-channel-count",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)2},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -187,7 +187,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_FILE_FORMAT,
         (char *)"file-format",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_ARRAY,
@@ -199,7 +199,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_TARGET_FILE_NAME,
         (char *)"filename",
-        MMF_VALUE_TYPE_STRING,
+        MM_ATTRS_TYPE_STRING,
         MM_ATTRS_FLAG_RW,
         {NULL},
         MM_ATTRS_VALID_TYPE_NONE,
@@ -211,7 +211,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_VIDEO_ENABLE,
         (char *)"video-enable",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)FALSE},
         MM_ATTRS_VALID_TYPE_NONE,
@@ -223,7 +223,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_AUDIO_ENABLE,
         (char *)"audio-enable",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)FALSE},
         MM_ATTRS_VALID_TYPE_NONE,
@@ -235,7 +235,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_MODE,
         (char *)"recorder-mode",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)MM_STREAMRECORDER_MODE_MEDIABUFFER},
         MM_ATTRS_VALID_TYPE_NONE,
@@ -247,7 +247,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_TARGET_MAX_SIZE,
         (char *)"target-max-size",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -259,7 +259,7 @@ mm_streamrecorder_attr_construct_info stream_attrs_const_info[] = {
        {
         MM_STR_TARGET_TIME_LIMIT,
         (char *)"target-time-limit",
-        MMF_VALUE_TYPE_INT,
+        MM_ATTRS_TYPE_INT,
         MM_ATTRS_FLAG_RW,
         {(void *)0},
         MM_ATTRS_VALID_TYPE_INT_RANGE,
@@ -362,7 +362,6 @@ static int __mmstreamrecorder_release_conf_valid_info(MMHandleType handle)
 #if 0
 static bool __mmstreamrecorder_attrs_is_supported(MMHandleType handle, int idx)
 {
-       mmf_attrs_t *attr = (mmf_attrs_t *) handle;
        int flag;
 
        if (mm_attrs_get_flags(handle, idx, &flag) == MM_ERROR_NONE) {
@@ -372,15 +371,15 @@ static bool __mmstreamrecorder_attrs_is_supported(MMHandleType handle, int idx)
                return FALSE;
        }
 
-       if (attr->items[idx].value_spec.type == MM_ATTRS_VALID_TYPE_INT_RANGE) {
+       if (mm_attrs_get_valid_type(handle, idx) == MM_ATTRS_VALID_TYPE_INT_RANGE) {
                int min, max;
-               mm_attrs_get_valid_range((MMHandleType) attr, idx, &min, &max);
+               mm_attrs_get_valid_range(handle, idx, &min, &max);
                if (max < min)
                        return FALSE;
-       } else if (attr->items[idx].value_spec.type == MM_ATTRS_VALID_TYPE_INT_ARRAY) {
+       } else if (mm_attrs_get_valid_type(handle, idx) == MM_ATTRS_VALID_TYPE_INT_ARRAY) {
                int count;
                int *array;
-               mm_attrs_get_valid_array((MMHandleType) attr, idx, &count, &array);
+               mm_attrs_get_valid_array(handle, idx, &count, &array);
                if (count == 0)
                        return FALSE;
        }
@@ -525,19 +524,19 @@ static int __mmstreamrecorder_check_valid_pair(MMHandleType handle, char **err_a
 }
 
 /* attribute commiter */
-void __mmstreamrecorder_print_attrs(const char *attr_name, const mmf_value_t * value, const char *cmt_way)
+void __mmstreamrecorder_print_attrs(const char *attr_name, const MMAttrsValue * value, const char *cmt_way)
 {
        switch (value->type) {
-       case MMF_VALUE_TYPE_INT:
+       case MM_ATTRS_TYPE_INT:
                _mmstreamrec_dbg_log("%s :(%s:%d)", cmt_way, attr_name, value->value.i_val);
                break;
-       case MMF_VALUE_TYPE_DOUBLE:
+       case MM_ATTRS_TYPE_DOUBLE:
                _mmstreamrec_dbg_log("%s :(%s:%f)", cmt_way, attr_name, value->value.d_val);
                break;
-       case MMF_VALUE_TYPE_STRING:
+       case MM_ATTRS_TYPE_STRING:
                _mmstreamrec_dbg_log("%s :(%s:%s)", cmt_way, attr_name, value->value.s_val);
                break;
-       case MMF_VALUE_TYPE_DATA:
+       case MM_ATTRS_TYPE_DATA:
                _mmstreamrec_dbg_log("%s :(%s:%p)", cmt_way, attr_name, value->value.p_val);
                break;
        default:
@@ -738,17 +737,18 @@ MMHandleType _mmstreamrecorder_alloc_attribute(MMHandleType handle)
 {
        _mmstreamrec_dbg_log("");
 
-       MMHandleType attrs = 0;
-       mmf_attrs_construct_info_t *attrs_const_info = NULL;
+       MMHandleType attrs = NULL;
+       MMAttrsConstructInfo *attrs_const_info = NULL;
        unsigned int attr_count = 0;
        unsigned int idx;
+       int ret = MM_ERROR_NONE;
 
        /* Create attribute constructor */
        _mmstreamrec_dbg_log("start");
 
-       /* alloc 'mmf_attrs_construct_info_t' */
+       /* alloc 'mm_attrs_construct_info_t' */
        attr_count = ARRAY_SIZE(stream_attrs_const_info);
-       attrs_const_info = malloc(attr_count * sizeof(mmf_attrs_construct_info_t));
+       attrs_const_info = malloc(attr_count * sizeof(MMAttrsConstructInfo));
 
        if (!attrs_const_info) {
                _mmstreamrec_dbg_err("Fail to alloc constructor.");
@@ -772,12 +772,12 @@ MMHandleType _mmstreamrecorder_alloc_attribute(MMHandleType handle)
 
        _mmstreamrec_dbg_log("Create Streamrecorder Attributes[%p, %d]", attrs_const_info, attr_count);
 
-       attrs = mmf_attrs_new_from_data("Streamrecorder_Attributes", attrs_const_info, attr_count, _mmstreamrecorder_commit_streamrecorder_attrs, (void *)handle);
+       ret = mm_attrs_new(attrs_const_info, attr_count, "Streamrecorder_Attributes", _mmstreamrecorder_commit_streamrecorder_attrs, (void *)handle, &attrs);
 
        free(attrs_const_info);
        attrs_const_info = NULL;
 
-       if (attrs == 0) {
+       if (ret != MM_ERROR_NONE) {
                _mmstreamrec_dbg_err("Fail to alloc attribute handle");
                return 0;
        }
@@ -785,19 +785,19 @@ MMHandleType _mmstreamrecorder_alloc_attribute(MMHandleType handle)
        __mmstreamrecorder_set_conf_to_valid_info(handle);
 
        for (idx = 0; idx < attr_count; idx++) {
-               mmf_attrs_set_valid_type(attrs, idx, stream_attrs_const_info[idx].validity_type);
+               mm_attrs_set_valid_type(attrs, idx, stream_attrs_const_info[idx].validity_type);
 
                switch (stream_attrs_const_info[idx].validity_type) {
                case MM_ATTRS_VALID_TYPE_INT_ARRAY:
                        if (stream_attrs_const_info[idx].validity_value_1.int_array &&
                                        stream_attrs_const_info[idx].validity_value_2.count > 0)
-                               mmf_attrs_set_valid_array(attrs, idx,
+                               mm_attrs_set_valid_array(attrs, idx,
                                                (const int *)(stream_attrs_const_info[idx].validity_value_1.int_array),
                                                stream_attrs_const_info[idx].validity_value_2.count,
                                                stream_attrs_const_info[idx].default_value.value_int);
                        break;
                case MM_ATTRS_VALID_TYPE_INT_RANGE:
-                       mmf_attrs_set_valid_range(attrs, idx,
+                       mm_attrs_set_valid_range(attrs, idx,
                                        stream_attrs_const_info[idx].validity_value_1.int_min,
                                        stream_attrs_const_info[idx].validity_value_2.int_max,
                                        stream_attrs_const_info[idx].default_value.value_int);
@@ -805,13 +805,13 @@ MMHandleType _mmstreamrecorder_alloc_attribute(MMHandleType handle)
                case MM_ATTRS_VALID_TYPE_DOUBLE_ARRAY:
                        if (stream_attrs_const_info[idx].validity_value_1.double_array &&
                                        stream_attrs_const_info[idx].validity_value_2.count > 0)
-                               mmf_attrs_set_valid_double_array(attrs, idx,
+                               mm_attrs_set_valid_double_array(attrs, idx,
                                                (const double *)(stream_attrs_const_info[idx].validity_value_1.double_array),
                                                stream_attrs_const_info[idx].validity_value_2.count,
                                                stream_attrs_const_info[idx].default_value.value_double);
                        break;
                case MM_ATTRS_VALID_TYPE_DOUBLE_RANGE:
-                       mmf_attrs_set_valid_double_range(attrs, idx,
+                       mm_attrs_set_valid_double_range(attrs, idx,
                                        stream_attrs_const_info[idx].validity_value_1.double_min,
                                        stream_attrs_const_info[idx].validity_value_2.double_max,
                                        stream_attrs_const_info[idx].default_value.value_double);
@@ -835,7 +835,7 @@ void _mmstreamrecorder_dealloc_attribute(MMHandleType attrs)
        _mmstreamrec_dbg_log("");
 
        if (attrs) {
-               mmf_attrs_free(attrs);
+               mm_attrs_free(attrs);
 
                _mmstreamrec_dbg_log("released attribute");
        }
@@ -940,7 +940,7 @@ int _mmstreamrecorder_get_attribute_info(MMHandleType handle, const char *attr_n
        return ret;
 }
 
-bool _mmstreamrecorder_commit_streamrecorder_attrs(int attr_idx, const char *attr_name, const mmf_value_t * value, void *commit_param)
+bool _mmstreamrecorder_commit_streamrecorder_attrs(int attr_idx, const char *attr_name, const MMAttrsValue * value, void *commit_param)
 {
        bool bret = FALSE;
 
@@ -962,7 +962,7 @@ bool _mmstreamrecorder_commit_streamrecorder_attrs(int attr_idx, const char *att
        return bret;
 }
 
-bool _mmstreamrecorder_commit_video_enable(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_video_enable(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        /* mmf_streamrecorder_t *hstreamrecorder = MMF_STREAMRECORDER(handle); */
@@ -988,7 +988,7 @@ bool _mmstreamrecorder_commit_video_enable(MMHandleType handle, int attr_idx, co
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_video_encoder(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_video_encoder(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;
@@ -1010,7 +1010,7 @@ bool _mmstreamrecorder_commit_video_encoder(MMHandleType handle, int attr_idx, c
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_audio_enable(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_audio_enable(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        /* mmf_streamrecorder_t *hstreamrecorder = MMF_STREAMRECORDER(handle); */
@@ -1035,7 +1035,7 @@ bool _mmstreamrecorder_commit_audio_enable(MMHandleType handle, int attr_idx, co
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_audio_encoder(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_audio_encoder(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;
@@ -1057,7 +1057,7 @@ bool _mmstreamrecorder_commit_audio_encoder(MMHandleType handle, int attr_idx, c
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_audio_samplingrate(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_audio_samplingrate(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;
@@ -1079,7 +1079,7 @@ bool _mmstreamrecorder_commit_audio_samplingrate(MMHandleType handle, int attr_i
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_audio_bitformat(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_audio_bitformat(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;
@@ -1104,7 +1104,7 @@ bool _mmstreamrecorder_commit_audio_bitformat(MMHandleType handle, int attr_idx,
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_video_bitrate(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_video_bitrate(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;
@@ -1127,7 +1127,7 @@ bool _mmstreamrecorder_commit_video_bitrate(MMHandleType handle, int attr_idx, c
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_audio_bitrate(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_audio_bitrate(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;
@@ -1149,7 +1149,7 @@ bool _mmstreamrecorder_commit_audio_bitrate(MMHandleType handle, int attr_idx, c
        return TRUE;
 }
 
-bool _mmstreamrecorder_commit_audio_channel(MMHandleType handle, int attr_idx, const mmf_value_t *value)
+bool _mmstreamrecorder_commit_audio_channel(MMHandleType handle, int attr_idx, const MMAttrsValue *value)
 {
        MMHandleType attr = 0;
        _MMStreamRecorderSubContext *sc = NULL;