info[i].conv_type);
*/
+ if (ret != MM_ERROR_NONE)
+ break;
+
if (info[i].type == CONFIGURE_TYPE_MAIN) {
conf_info = hcamcorder->conf_main;
/*_mmcam_dbg_log("MAIN configure [%s]", info[i].keyword);*/
if (!_mmcamcorder_conf_get_value_int(handle, conf_info, info[i].category, info[i].keyword, &ivalue)) {
ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
- break; /* skip to set */
+ break; /* skip to set */
}
ret = mm_attrs_set_int(MMF_CAMCORDER_ATTRS(hcamcorder), info[i].attr_idx, ivalue);
int idefault = 0;
type_int_array *tarray = NULL;
- if (!_mmcamcorder_conf_get_value_int_array(conf_info, info[i].category, info[i].keyword, &tarray)) {
- ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
- break; /* skip to set */
- }
+ if (!_mmcamcorder_conf_get_value_int_array(conf_info, info[i].category, info[i].keyword, &tarray))
+ break; /* skip to set, but not error */
if (tarray) {
idefault = tarray->default_value;
{
type_int_range *irange = NULL;
- if (!_mmcamcorder_conf_get_value_int_range(conf_info, info[i].category, info[i].keyword, &irange)) {
- ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
- break; /* skip to set */
- }
+ if (!_mmcamcorder_conf_get_value_int_range(conf_info, info[i].category, info[i].keyword, &irange))
+ break; /* skip to set, but not error */
if (irange) {
/* _mmcam_dbg_log("INT Range. m:%d, s:%d, min=%d, max=%d", info[i].main_key, info[i].sub_key1, irange->min, irange->max); */
if (!_mmcamcorder_conf_get_value_string(handle, conf_info, info[i].category, info[i].keyword, &cString)) {
ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
- break; /* skip to set */
+ break; /* skip to set */
}
/* _mmcam_dbg_log("String. m:%d, s:%d, cString=%s", info[i].main_key, info[i].sub_key1, cString); */
/*_mmcam_dbg_log("INT PAIR Array. type:%d, attr_idx:%d, attr_idx_pair:%d", info[i].type, info[i].attr_idx, info[i].attr_idx_pair);*/
- if (!_mmcamcorder_conf_get_value_int_pair_array(conf_info, info[i].category, info[i].keyword, &pair_array)) {
- ret = MM_ERROR_CAMCORDER_CREATE_CONFIGURE;
- break; /* skip to set */
- }
+ if (!_mmcamcorder_conf_get_value_int_pair_array(conf_info, info[i].category, info[i].keyword, &pair_array))
+ break; /* skip to set, but not error */
if (pair_array && pair_array->count > 0) {
/* "mmf_attrs_set_valid_type" initializes spec value in attribute, so allocated memory could be missed */
if (hcamcorder->caminfo_convert[i].type == CONFIGURE_TYPE_CTRL) {
switch (hcamcorder->caminfo_convert[i].category) {
case CONFIGURE_CATEGORY_CTRL_STROBE:
- if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "StrobeMode")) {
+ if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "StrobeMode"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_STROBE_MODE]);
- }
break;
case CONFIGURE_CATEGORY_CTRL_EFFECT:
- if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "WhiteBalance")) {
+ if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "WhiteBalance"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_WHITE_BALANCE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "ColorTone")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "ColorTone"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_COLOR_TONE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "WDR")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "WDR"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_WDR]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "Flip")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "Flip"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_FLIP]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "Rotation")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "Rotation"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_ROTATION]);
- }
break;
case CONFIGURE_CATEGORY_CTRL_PHOTOGRAPH:
- if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "FocusMode")) {
+ if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "FocusMode"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_FOCUS_MODE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "AFType")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "AFType"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_AF_RANGE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "AEType")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "AEType"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_EXPOSURE_MODE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "ISO")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "ISO"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_ISO]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "ProgramMode")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "ProgramMode"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_PROGRAM_MODE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "AntiHandshake")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "AntiHandshake"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_ANTI_HAND_SHAKE]);
- } else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "VideoStabilization")) {
+ else if (!strcmp(hcamcorder->caminfo_convert[i].keyword, "VideoStabilization"))
hcamcorder->caminfo_convert[i].enum_convert = &(hcamcorder->enum_conv[ENUM_CONVERT_VIDEO_STABILIZATION]);
- }
break;
default:
break;