From ca3986b37fdeaca4e347c5bb93b900f4e4040f6c Mon Sep 17 00:00:00 2001 From: Seokpil Park Date: Wed, 21 Aug 2013 16:48:08 +0900 Subject: [PATCH] [content] Fix a sample app fail Change-Id: I07a9e076f4728fe435862ce59a52b4fb5275ae0e Signed-off-by: Seokpil Park --- src/FCnt_ContentManagerImpl.cpp | 258 +++++++++++++++++++--------------------- 1 file changed, 122 insertions(+), 136 deletions(-) diff --git a/src/FCnt_ContentManagerImpl.cpp b/src/FCnt_ContentManagerImpl.cpp index 0a649fd..b7be808 100644 --- a/src/FCnt_ContentManagerImpl.cpp +++ b/src/FCnt_ContentManagerImpl.cpp @@ -967,78 +967,71 @@ _ContentManagerImpl::UpdateDataToDatabase(const _ContentInfoImpl* pContentInfoIm result r = E_SUCCESS; unique_ptr pValue(null); - if (!pContentInfoImpl->GetAuthor().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetAuthor())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + // author + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetAuthor())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); - val = media_info_set_author(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_author failed[%d].", val); - } - if (!pContentInfoImpl->GetCategory().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetCategory())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + val = media_info_set_author(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_author failed[%d].", val); - val = media_info_set_category(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_category failed[%d].", val); - } - if (!pContentInfoImpl->GetContentName().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetContentName())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + // category + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetCategory())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); - val = media_info_set_content_name(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_content_name failed[%d].", val); - } - if (!pContentInfoImpl->GetDescription().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetDescription())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + val = media_info_set_category(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_category failed[%d].", val); - val = media_info_set_description(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_description failed[%d].", val); - } - if (!pContentInfoImpl->GetKeyword().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetKeyword())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + // content name + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetContentName())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); - val = media_info_set_keyword(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_keyword failed[%d].", val); - } - if (!pContentInfoImpl->GetLocationTag().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetLocationTag())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + val = media_info_set_content_name(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_content_name failed[%d].", val); - val = media_info_set_location_tag(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_location_tag failed[%d].", val); - } - if (!pContentInfoImpl->GetProvider().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetProvider())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + // description + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetDescription())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); - val = media_info_set_provider(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_provider failed[%d].", val); - } - if (!pContentInfoImpl->GetRating().IsEmpty()) - { - pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetRating())); - SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + val = media_info_set_description(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_description failed[%d].", val); - val = media_info_set_age_rating(pMediaInfo.get(), pValue.get()); - SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), - "media_info_set_age_rating failed[%d].", val); - } + // keyword + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetKeyword())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + + val = media_info_set_keyword(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_keyword failed[%d].", val); + + // location tag + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetLocationTag())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + + val = media_info_set_location_tag(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_location_tag failed[%d].", val); + + // provider + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetProvider())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + + val = media_info_set_provider(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_provider failed[%d].", val); + + // rating + pValue.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetRating())); + SysTryReturnResult(NID_CNT, pValue != null, E_OUT_OF_MEMORY, "The memory is insufficient."); + + val = media_info_set_age_rating(pMediaInfo.get(), pValue.get()); + SysTryReturnResult(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, ConvertError(val), + "media_info_set_age_rating failed[%d].", val); + + // latitude, longitude, altitude if (Double::Compare(pContentInfoImpl->GetLatitude(), DEFAULT_COORDINATE) != 0 && Double::Compare(pContentInfoImpl->GetLongitude(), DEFAULT_COORDINATE) != 0) { @@ -1126,86 +1119,79 @@ _ContentManagerImpl::SaveDataToDatabase(const media_info_h pMediaInfo, int val = 0; unique_ptr pStr(null); - if (pContentInfoImpl->GetAuthor() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetAuthor())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + // author + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetAuthor())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); - val = media_info_set_author(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_author failed[%d].", val); - } - if (pContentInfoImpl->GetCategory() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetCategory())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + val = media_info_set_author(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_author failed[%d].", val); - val = media_info_set_category(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_category failed[%d].", val); - } - if (pContentInfoImpl->GetContentName() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetContentName())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + // category + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetCategory())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); - val = media_info_set_content_name(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_content_name failed[%d].", val); - } - if (pContentInfoImpl->GetDescription() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetDescription())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + val = media_info_set_category(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_category failed[%d].", val); - val = media_info_set_description(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_description failed[%d].", val); - } - if (pContentInfoImpl->GetKeyword() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetKeyword())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + // content name + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetContentName())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); - val = media_info_set_keyword(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_keyword failed[%d].", val); - } - if (pContentInfoImpl->GetLocationTag() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetLocationTag())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + val = media_info_set_content_name(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_content_name failed[%d].", val); - val = media_info_set_location_tag(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_location_tag failed[%d].", val); - } - if (pContentInfoImpl->GetProvider() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetProvider())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + // description + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetDescription())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); - val = media_info_set_provider(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_provider failed[%d].", val); - } - if (pContentInfoImpl->GetRating() != null) - { - pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetRating())); - SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, - "[E_OUT_OF_MEMORY] The memory is insufficient."); + val = media_info_set_description(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_description failed[%d].", val); - val = media_info_set_age_rating(pMediaInfo, pStr.get()); - SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), - "media_info_set_age_rating failed[%d].", val); - } + // keyword + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetKeyword())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); + + val = media_info_set_keyword(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_keyword failed[%d].", val); + + // location tag + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetLocationTag())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); + + val = media_info_set_location_tag(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_location_tag failed[%d].", val); + + // provider + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetProvider())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); + + val = media_info_set_provider(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_provider failed[%d].", val); + + // rating + pStr.reset(_StringConverter::CopyToCharArrayN(pContentInfoImpl->GetRating())); + SysTryReturn(NID_CNT, pStr != null, UuId::GetInvalidUuId(), E_OUT_OF_MEMORY, + "[E_OUT_OF_MEMORY] The memory is insufficient."); + + val = media_info_set_age_rating(pMediaInfo, pStr.get()); + SysTryReturn(NID_CNT, val == MEDIA_CONTENT_ERROR_NONE, UuId::GetInvalidUuId(), ConvertError(val), + "media_info_set_age_rating failed[%d].", val); + + // latitude, longitude, altitude if (Double::Compare(pContentInfoImpl->GetLatitude(), DEFAULT_COORDINATE) != 0 && Double::Compare(pContentInfoImpl->GetLongitude(), DEFAULT_COORDINATE) != 0) { -- 2.7.4