diff -r d141fc1ad77b -r e61a04404bdf mmappcomponents/mmmtpdataprovider/mmmtpdprequestprocessor/src/ccopyobject.cpp --- a/mmappcomponents/mmmtpdataprovider/mmmtpdprequestprocessor/src/ccopyobject.cpp Tue Jul 06 14:48:59 2010 +0300 +++ b/mmappcomponents/mmmtpdataprovider/mmmtpdprequestprocessor/src/ccopyobject.cpp Wed Aug 18 10:16:02 2010 +0300 @@ -310,9 +310,9 @@ SetPreviousPropertiesL(); CFileMan* fileMan = CFileMan::NewL( iFramework.Fs() ); + CleanupStack::PushL( fileMan ); // + fileMan User::LeaveIfError( fileMan->Copy( oldFileName, aNewFileName ) ); - delete fileMan; - fileMan = NULL; + CleanupStack::PopAndDestroy( fileMan ); // - fileMan PRINT( _L( "MM MTP <= CCopyObject::CopyFileL" ) ); return handle; @@ -417,8 +417,6 @@ { PRINT( _L( "MM MTP => CCopyObject::SetPreviousPropertiesL" ) ); - TMTPResponseCode respcode = EMTPRespCodeOK; - iPropertyList->ResetCursor(); const TInt count = iPropertyList->NumberOfElements(); for ( TInt i = 0; i < count; i++ ) @@ -460,7 +458,7 @@ { CMTPTypeString *stringData = CMTPTypeString::NewLC( element.StringL( CMTPTypeObjectPropListElement::EValue ) ); // + stringData - respcode = iDpConfig.PropSettingUtility()->SetMetaDataToWrapper( iDpConfig, + iDpConfig.PropSettingUtility()->SetMetaDataToWrapper( iDpConfig, propertyCode, *stringData, *iTargetObject ); @@ -471,7 +469,7 @@ default: { - respcode = iDpConfig.PropSettingUtility()->SetSpecificObjectPropertyL( iDpConfig, + iDpConfig.PropSettingUtility()->SetSpecificObjectPropertyL( iDpConfig, propertyCode, *iTargetObject, element ); @@ -480,13 +478,7 @@ } } // end of for loop - // ignore errors - if ( respcode == EMTPRespCodeOK ) - { - // do nothing, just to get rid of build warning - } - - PRINT1( _L( "MM MTP <= CCopyObject::SetPreviousPropertiesL respcode = 0x%x" ), respcode ); + PRINT( _L( "MM MTP <= CCopyObject::SetPreviousPropertiesL" ) ); } // -----------------------------------------------------------------------------