userlibandfileserver/fileserver/sfsrv/cl_fman.cpp
branchRCL_3
changeset 257 3e88ff8f41d5
parent 256 c1f20ce4abcf
child 268 345b1ca54e88
--- a/userlibandfileserver/fileserver/sfsrv/cl_fman.cpp	Tue Aug 31 16:34:26 2010 +0300
+++ b/userlibandfileserver/fileserver/sfsrv/cl_fman.cpp	Wed Sep 01 12:34:56 2010 +0100
@@ -15,10 +15,21 @@
 
 #include "cl_std.h"
 
-#ifdef OST_TRACE_COMPILER_IN_USE
-#include "cl_fmanTraces.h"
-#endif
-
+#define RETURNIFERROR(a,b,t)  					\
+	{											\
+	if ((a=b)!=KErrNone)						\
+		{										\
+		if(iStatus)								\
+			User::RequestComplete(iStatus,a);	\
+		TInt _t = t;							\
+		if (_t)	{TRACE1(UTF::EBorder, t, MODULEUID, a);}	\
+		return(a);								\
+		}										\
+	}
+
+#define RETURNIFERRORD(a,b,t)  		 			\
+	TInt a; 									\
+	RETURNIFERROR(a,b,t)
 
 const TUint KRecurseFlag	=	0x40000000;
 const TUint KScanDownFlag	=	0x20000000;
@@ -517,14 +528,14 @@
 @return Newly created CFileMan object.
 */
 	{
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANNEWL1, "sess %x", aFs.Handle());
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManNewL1, MODULEUID, aFs.Handle());
 
 	CFileMan* fileMan=new(ELeave) CFileMan(aFs);
 	CleanupStack::PushL(fileMan);
 	fileMan->CFileBase::ConstructL();
 	CleanupStack::Pop();
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANNEWL1RETURN, "CFileMan* %x", fileMan);
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManNewL1Return, MODULEUID, fileMan);
 	return fileMan;
 	}
 
@@ -541,7 +552,7 @@
 @return Newly created CFileMan object.
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANNEWL2, "sess %x anObserver %x", (TUint) aFs.Handle(), (TUint) anObserver);
+	TRACE2(UTF::EBorder, UTraceModuleEfsrv::ECFileManNewL2, MODULEUID, aFs.Handle(), anObserver);
 
 	CFileMan* fileMan=new(ELeave) CFileMan(aFs);
 	CleanupStack::PushL(fileMan);
@@ -549,7 +560,7 @@
 	CleanupStack::Pop();
 	fileMan->SetObserver(anObserver);
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANNEWL2RETURN, "CFileMan* %x", fileMan);
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManNewL2Return, MODULEUID, fileMan);
 	return fileMan;
 	}
 
@@ -565,9 +576,9 @@
 	}
 CFileMan::~CFileMan()
 	{
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANDESTRUCTOR, "this %x", this);
-
-	OstTrace0(TRACE_BORDER, EFSRV_ECFILEMANDESTRUCTORRETURN, "");
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManDestructor, MODULEUID, this);
+
+	TRACE0(UTF::EBorder, UTraceModuleEfsrv::ECFileManDestructorReturn, MODULEUID);
 	}
 
 
@@ -578,7 +589,7 @@
 @return The action which CFileMan is carrying out.
 */
 	{
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCURRENTACTION, "this %x", this);
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCurrentAction, MODULEUID, this);
 
 	TAction action = ENone;
 
@@ -617,7 +628,7 @@
 		Panic(EFManUnknownAction);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCURRENTACTIONRETURN, "action %d", action);
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCurrentActionReturn, MODULEUID, action);
 	return (action);
 	}
 
@@ -635,12 +646,11 @@
                 the current CFileMan operation
 */
 	{
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANGETCURRENTTARGET, "this %x", this);
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManGetCurrentTarget, MODULEUID, this);
 
 	GetSrcAndTrg(iTmpParse, aTrgName);
 
-	OstTrace0(TRACE_BORDER, EFSRV_ECFILEMANGETCURRENTTARGETRETURN, "");
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANGETCURRENTTARGET_EFILENAME, "FileName %S", aTrgName.Ptr(), aTrgName.Length()<<1);
+	TRACEMULT1(UTF::EBorder, UTraceModuleEfsrv::ECFileManGetCurrentTargetReturn, MODULEUID, aTrgName);
 	}
 
 
@@ -657,14 +667,13 @@
                 CFileMan operation.
 */
 	{
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANGETCURRENTSOURCE, "this %x", this);
+	TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManGetCurrentSource, MODULEUID, this);
 
     TPtrC fullPath(FullPath());
 	iTmpParse.Set(CurrentEntry().iName, &fullPath, NULL);
 	aSrcName = iTmpParse.FullName();
 
-	OstTrace0(TRACE_BORDER, EFSRV_ECFILEMANGETCURRENTSOURCERETURN, "");
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANGETCURRENTSOURCE_EFILENAME, "FileName %S", aSrcName.Ptr(), aSrcName.Length()<<1);
+	TRACEMULT1(UTF::EBorder, UTraceModuleEfsrv::ECFileManGetCurrentSourceReturn, MODULEUID, aSrcName);
 	}
 
 void CFileMan::GetSrcAndTrg(TParse& aSrcName,TFileName& aTrgName)
@@ -725,7 +734,7 @@
 @return The number of bytes transferred.
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANBYTESTRANSFERREDBYCOPYSTEP, "this %x BytesTransferred %d", (TUint) this, (TUint) iBytesTransferred);
+	TRACE2(UTF::EBorder, UTraceModuleEfsrv::ECFileManBytesTransferredByCopyStep, MODULEUID, this, iBytesTransferred);
 
 	return(iBytesTransferred);
 	}
@@ -895,9 +904,8 @@
 
 */
 	{
-	OstTraceExt5(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS1A, "this %x aSetMask %x aClearMask %x aSwitches %d status %x", (TUint) this, (TUint) aSetMask, (TUint) aClearMask, (TUint) aSwitches, (TUint) &aStatus);
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS1B, "aTime %x:%x ", (TUint) I64HIGH(aTime.Int64()), (TUint) I64LOW(aTime.Int64()));
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS1A_EFILEPATH, "FilePath %S", aName.Ptr(), aName.Length()<<1);
+	TRACEMULT8(UTF::EBorder, UTraceModuleEfsrv::ECFileManAttribs1, MODULEUID,
+		(TUint) this, aName, aSetMask, aClearMask, I64LOW(aTime.Int64()), I64HIGH(aTime.Int64()), aSwitches, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -910,8 +918,7 @@
 		r = Attribs(aName,aSetMask,aClearMask,aTime,aSwitches);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS1RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManAttribs1Return, MODULEUID, r);
 	return r;
 	}
 
@@ -967,9 +974,8 @@
 
 */
 	{
-	OstTraceExt4(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS2A, "this %x aSetMask %x aClearMask %x aSwitches %x", (TUint) this, (TUint) aSetMask, (TUint) aClearMask, (TUint) aSwitches);
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS2B, "aTime %x:%x ", (TUint) I64HIGH(aTime.Int64()), (TUint) I64LOW(aTime.Int64()));
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS2A_EFILEPATH, "FilePath %S", aName.Ptr(), aName.Length()<<1);
+	TRACEMULT7(UTF::EBorder, UTraceModuleEfsrv::ECFileManAttribs2, MODULEUID,
+		(TUint) this, aName, aSetMask, aClearMask, I64LOW(aTime.Int64()), I64HIGH(aTime.Int64()), aSwitches);
 
 	TInt ret;
 	if (iSwitches&KFManBusyFlag)
@@ -979,15 +985,7 @@
 	else
 		{
 		SetFlags(aSwitches&EOverWrite,aSwitches&ERecurse,ETrue,EFalse);
-		TInt r;
-		if ((r = iFs.Parse(aName,iSrcFile)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS2RETURN1, "r %d", r);
-			return r;
-			}
-
+		RETURNIFERRORD(r,iFs.Parse(aName,iSrcFile),UTraceModuleEfsrv::ECFileManAttribs2Return);
 		iSetMask=aSetMask;
 		iClearMask=aClearMask;
 		iTime=aTime;
@@ -999,8 +997,7 @@
 		DoSynchronize(r);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANATTRIBS2RETURN2, "r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManAttribs2Return, MODULEUID, ret);
 	return(ret);
 	}
 
@@ -1040,9 +1037,8 @@
 
 */
 	{
-	OstTraceExt3(TRACE_BORDER, EFSRV_ECFILEMANCOPY1, "this %x aSwitches %x status %x", (TUint) this, (TUint) aSwitches, (TUint) &aStatus);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANCOPY1_EOLDNAME, "OldName %S", anOld.Ptr(), anOld.Length()<<1);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANCOPY1_ENEWNAME, "NewName %S", aNew.Ptr(), aNew.Length()<<1);
+	TRACEMULT5(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy1, MODULEUID,
+		(TUint) this, anOld, aNew, aSwitches, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -1053,8 +1049,7 @@
 		r = Copy(anOld,aNew,aSwitches);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY1RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy1Return, MODULEUID, r);
 	return(r);
 	}
 
@@ -1094,7 +1089,7 @@
 including any directories in the path specified by aNew which do not
 already exist.
 
-If the source (anOld) is a FILE and the recursive operation is set,
+If the source (anOld) is a file and the recursive operation is set,
 then all the files with the same name as anOld in the source directory
 including those in subdirectories will be copied to the destination.
 
@@ -1183,38 +1178,21 @@
 
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANCOPY2, "this %x aSwitches %d", (TUint) this, (TUint) aSwitches);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANCOPY2_EOLDNAME, "OldName %S", anOld.Ptr(), anOld.Length()<<1);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANCOPY2_ENEWNAME, "NewName %S", aNew.Ptr(), aNew.Length()<<1);
+	TRACEMULT4(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy2, MODULEUID, (TUint) this, anOld, aNew, aSwitches);
 
 	if (iSwitches&KFManBusyFlag)
 		{
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY2RETURN1, "r %d", KErrInUse);
+		TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy2Return, MODULEUID, KErrInUse);
 		return(KErrInUse);
 		}
 	SetFlags(aSwitches&EOverWrite,aSwitches&ERecurse,ETrue,EFalse);
-	TInt r;
-	if ((r = iFs.Parse(anOld,iSrcFile)) != KErrNone)
-		{
-		if(iStatus)
-			User::RequestComplete(iStatus,r);
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY2RETURN2, "r %d", r);
-		return r;
-		}
-
-	if ((r = iFs.Parse(aNew,_L("*"),iTrgFile)) != KErrNone)
-		{
-		if(iStatus)
-			User::RequestComplete(iStatus,r);
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY2RETURN3, "r %d", r);
-		return r;
-		}
-
+	RETURNIFERRORD(r,iFs.Parse(anOld,iSrcFile),UTraceModuleEfsrv::ECFileManCopy2Return);
+	RETURNIFERROR(r,iFs.Parse(aNew,_L("*"),iTrgFile),UTraceModuleEfsrv::ECFileManCopy2Return);
 	CheckForDirectory();
 
 	if((iSwitches&KRecurseFlag) && iTrgFile.DriveAndPath().MatchF(iSrcFile.FullName()) != KErrNotFound)
 		{
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY2RETURN4, "r %d", KErrArgument);
+		TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy2Return, MODULEUID, KErrArgument);
 		return(KErrArgument);
 		}
 
@@ -1225,8 +1203,7 @@
 	TInt ret=(r==KErrNone) ? iLastError : r;
 	DoSynchronize(r);
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY2RETURN5, "r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy2Return, MODULEUID, ret);
 	return(ret);
 	}
 
@@ -1266,8 +1243,7 @@
 @see KNullDesC
 */
 	{
-	OstTraceExt3(TRACE_BORDER, EFSRV_ECFILEMANDELETE1, "this %x aSwitches %x status %x", (TUint) this, (TUint) aSwitches, (TUint) &aStatus);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANDELETE1_EFILEPATH, "FilePath %S", aName.Ptr(), aName.Length()<<1);
+	TRACEMULT4(UTF::EBorder, UTraceModuleEfsrv::ECFileManDelete1, MODULEUID, (TUint) this, aName, aSwitches, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -1280,8 +1256,7 @@
 		r = Delete(aName,aSwitches);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANDELETE1RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManDelete1Return, MODULEUID, r);
 	return(r);
 	}
 
@@ -1328,8 +1303,7 @@
 @see KNullDesC
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANDELETE2, "this %x aSwitches %d", (TUint) this, (TUint) aSwitches);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANDELETE2_EFILEPATH, "FilePath %S", aName.Ptr(), aName.Length()<<1);
+	TRACEMULT3(UTF::EBorder, UTraceModuleEfsrv::ECFileManDelete2, MODULEUID, (TUint) this, aName, aSwitches);
 
 	TInt ret;
 	if (iSwitches&KFManBusyFlag)
@@ -1339,15 +1313,7 @@
 	else
 		{
 		SetFlags(aSwitches&EOverWrite,aSwitches&ERecurse,ETrue,EFalse);
-		TInt r;
-		if ((r = iFs.Parse(aName,iSrcFile)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANDELETE2RETURN1, "() r %d", r);
-			return r;
-			}
-
+		RETURNIFERRORD(r,iFs.Parse(aName,iSrcFile),UTraceModuleEfsrv::ECFileManDelete2Return);
 		iAction = EInternalDelete;
 		iMatchEntry=KEntryAttHidden|KEntryAttMatchExclude|KEntryAttDir;
 	//	Exclude directories and system files - include hidden files
@@ -1357,8 +1323,7 @@
 		DoSynchronize(r);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANDELETE2RETURN2, "() r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManDelete2Return, MODULEUID, ret);
 	return(ret);
 	}
 
@@ -1402,9 +1367,8 @@
 
 */
 	{
-	OstTraceExt3(TRACE_BORDER, EFSRV_ECFILEMANMOVE1, "this %x aSwitches %x status %x", (TUint) this, (TUint) aSwitches, (TUint) &aStatus);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANMOVE1_EOLDNAME, "OldName %S", anOld.Ptr(), anOld.Length()<<1);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANMOVE1_ENEWNAME, "NewName %S", aNew.Ptr(), aNew.Length()<<1);
+	TRACEMULT5(UTF::EBorder, UTraceModuleEfsrv::ECFileManMove1, MODULEUID,
+		(TUint) this, anOld, aNew, aSwitches, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -1417,8 +1381,7 @@
 		r = Move(anOld,aNew,aSwitches);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANMOVE1RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManMove1Return, MODULEUID, r);
 	return r;
 	}
 
@@ -1498,7 +1461,7 @@
 fm->Move(_L("C:\\a\\b"), _L("C:\\x\\y\\"), CFileMan::ERecurse);
 @endcode
 
-If the source (anOld) is a FILE and the recursive operation is set,
+If the source (anOld) is a file and the recursive operation is set,
 then all the files with the same name as anOld in the source directory
 including those in subdirectories will be moved to the destination.
 
@@ -1558,36 +1521,20 @@
 @see CFileBase::GetLastError()
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANMOVE2, "this %x aSwitches %d", (TUint) this, (TUint) aSwitches);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANMOVE2_EOLDNAME, "OldName %S", anOld.Ptr(), anOld.Length()<<1);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANMOVE2_ENEWNAME, "NewName %S", aNew.Ptr(), aNew.Length()<<1);
+	TRACEMULT4(UTF::EBorder, UTraceModuleEfsrv::ECFileManMove2, MODULEUID,
+		(TUint) this, anOld, aNew, aSwitches);
 
 
 	if (iSwitches&KFManBusyFlag)
 		{
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANMOVE2RETURN1, "r %d", KErrInUse);
+		TRACE1(UTF::EBorder, UTraceModuleEfsrv::ECFileManMove2Return, MODULEUID, KErrInUse);
 		return(KErrInUse);
 		}
 
 	iNumberOfFilesProcessed = 0;
 
-	TInt r;
-	if ((r = iFs.Parse(anOld,iSrcFile)) != KErrNone)
-		{
-		if(iStatus)
-			User::RequestComplete(iStatus,r);
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANMOVE2RETURN2, "r %d", r);
-		return r;
-		}
-
-	if ((r = iFs.Parse(aNew,_L("*"),iTrgFile)) != KErrNone)
-		{
-		if(iStatus)
-			User::RequestComplete(iStatus,r);
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANMOVE2RETURN3, "r %d", r);
-		return r;
-		}
-
+	RETURNIFERRORD(r,iFs.Parse(anOld,iSrcFile),UTraceModuleEfsrv::ECFileManMove2Return);
+	RETURNIFERROR(r,iFs.Parse(aNew,_L("*"),iTrgFile),UTraceModuleEfsrv::ECFileManMove2Return);
 
 	TInt ret = KErrNone;
 	TBool aComplete = EFalse;
@@ -1606,7 +1553,7 @@
 			{
 			User::RequestComplete(iStatus, ret);
 			}
-		OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANMOVE2RETURN4, "r %d", ret);
+		TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManMove2Return, MODULEUID, ret);
 		return(ret);
 		}
 
@@ -1621,8 +1568,7 @@
 	ret = (r==KErrNone) ? iLastError : r;
 	DoSynchronize(r);
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANMOVE2RETURN5, "r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManMove2Return, MODULEUID, ret);
 	return(ret);
 	}
 
@@ -1926,6 +1872,15 @@
 	return(KErrNone);
 	}
 
+TInt CFileMan::RenameInvalidEntry(const TDesC& /*aName*/,const TDesC& /*aNewName*/,TUint /*aSwitches*/)
+//
+// Start rename operation
+//
+	{
+	return KErrNotSupported;
+	}
+
+
 
 
 EXPORT_C TInt CFileMan::Rename(const TDesC& aName,const TDesC& aNewName,TUint aSwitches,TRequestStatus& aStatus)
@@ -1960,9 +1915,8 @@
 
 */
 	{
-	OstTraceExt3(TRACE_BORDER, EFSRV_ECFILEMANRENAME1, "this %x aSwitches %x status %x", (TUint) this, (TUint) aSwitches, (TUint) &aStatus);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANRENAME1_EOLDNAME, "OldName %S", aName.Ptr(), aName.Length()<<1);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANRENAME1_ENEWNAME, "NewName %S", aNewName.Ptr(), aNewName.Length()<<1);
+	TRACEMULT5(UTF::EBorder, UTraceModuleEfsrv::ECFileManRename1, MODULEUID,
+		(TUint) this, aName, aNewName, aSwitches, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -1975,8 +1929,7 @@
 		r = Rename(aName,aNewName,aSwitches);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRENAME1RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManRename1Return, MODULEUID, r);
 	return(r);
 	}
 
@@ -2142,9 +2095,8 @@
 
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANRENAME2, "this %x aSwitches %d", (TUint) this, (TUint) aSwitches);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANRENAME2_EOLDNAME, "OldName %S", aName.Ptr(), aName.Length()<<1);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANRENAME2_ENEWNAME, "NewName %S", aNewName.Ptr(), aNewName.Length()<<1);
+	TRACEMULT4(UTF::EBorder, UTraceModuleEfsrv::ECFileManRename2, MODULEUID,
+		(TUint) this, aName, aNewName, aSwitches);
 
 	TInt ret;
 	if (iSwitches&KFManBusyFlag)
@@ -2154,22 +2106,8 @@
 	else
 		{
 		SetFlags(aSwitches&EOverWrite,EFalse,ETrue,EFalse);
-		TInt r;
-		if ((r = iFs.Parse(aName,iSrcFile)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRENAME2RETURN1, "r %d", r);
-			return r;
-			}
-
-		if ((r = iFs.Parse(aNewName,_L("*"),iTrgFile)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRENAME2RETURN2, "r %d", r);
-			return r;
-			}
+		RETURNIFERRORD(r,iFs.Parse(aName,iSrcFile),UTraceModuleEfsrv::ECFileManRename2Return);
+		RETURNIFERROR(r,iFs.Parse(aNewName,_L("*"),iTrgFile),UTraceModuleEfsrv::ECFileManRename2Return);
 
 		iAction = EInternalRename;
 		iMatchEntry=KEntryAttMaskSupported;
@@ -2179,8 +2117,7 @@
 		DoSynchronize(r);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRENAME2RETURN3, "r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManRename2Return, MODULEUID, ret);
 	return(ret);
 	}
 
@@ -2210,8 +2147,7 @@
 
 */
 	{
-	OstTraceExt2(TRACE_BORDER, EFSRV_ECFILEMANRMDIR1, "this %x status %x", (TUint) this, (TUint) &aStatus);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANRMDIR1_EDIRNAME, "Dir %S", aDirName.Ptr(), aDirName.Length()<<1);
+	TRACEMULT3(UTF::EBorder, UTraceModuleEfsrv::ECFileManRmDir1, MODULEUID, (TUint) this, aDirName, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -2224,8 +2160,7 @@
 		r = RmDir(aDirName);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRMDIR1RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManRmDir1Return, MODULEUID, r);
 	return r;
 	}
 
@@ -2260,8 +2195,7 @@
 
 */
 	{
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRMDIR2, "this %x", (TUint) this);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANRMDIR2_EDIRNAME, "Dir %S", aDirName.Ptr(), aDirName.Length()<<1);
+	TRACEMULT2(UTF::EBorder, UTraceModuleEfsrv::ECFileManRmDir2, MODULEUID, (TUint) this, aDirName);
 
 	TInt ret;
 	if (iSwitches&KFManBusyFlag)
@@ -2271,15 +2205,7 @@
 	else
 		{
 		SetFlags(ETrue,ETrue,EFalse,EFalse);
-		TInt r;
-		if ((r = iFs.Parse(aDirName,iTrgFile)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRMDIR2RETURN1, "r %d", r);
-			return r;
-			}
-
+		RETURNIFERRORD(r,iFs.Parse(aDirName,iTrgFile),UTraceModuleEfsrv::ECFileManRmDir2Return);
 		iSrcFile.Set(iTrgFile.DriveAndPath(),NULL,NULL);
 		iAction = EInternalRmDir;
 		iMatchEntry=KEntryAttMaskSupported;
@@ -2289,8 +2215,7 @@
 		ret = (r!=KErrNone) ? iLastError : KErrNone;
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANRMDIR2RETURN2, "r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManRmDir2Return, MODULEUID, ret);
 	return ret;
 	}
 
@@ -2688,8 +2613,8 @@
 @capability Dependent If the path for aNew begins with /Resource then Tcb capability is required.
 */
 	{
-	OstTraceExt3(TRACE_BORDER, EFSRV_ECFILEMANCOPY3, "this %x anOldSubs %x aSwitches %x", (TUint) this, (TUint) anOld.SubSessionHandle(), (TUint) aSwitches);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANCOPY3_ENEWNAME, "NewName %S", aNew.Ptr(), aNew.Length()<<1);
+	TRACEMULT4(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy3, MODULEUID,
+		(TUint) this, anOld.SubSessionHandle(), aNew, aSwitches);
 
 	TInt ret;
 	if (iSwitches&KFManBusyFlag)
@@ -2711,23 +2636,11 @@
 		iSwitches|= KCopyFromHandle;
 
 		TInt r;
-		if ((r = iFs.Parse(aNew, iTrgFile)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY3RETURN1, "r %d", r);
-			return r;
-			}
+		RETURNIFERROR(r, iFs.Parse(aNew, iTrgFile),UTraceModuleEfsrv::ECFileManCopy3Return);
 
 		// Need to duplicate the RFile handle so that any threads owned
 		// by this process can use it - i.e. the worker thread
-		if ((r = iSrcFileHandle.Duplicate(anOld, EOwnerProcess)) != KErrNone)
-			{
-			if(iStatus)
-				User::RequestComplete(iStatus,r);
-			OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY3RETURN2, "r %d", r);
-			return r;
-			}
+		RETURNIFERROR(r, iSrcFileHandle.Duplicate(anOld, EOwnerProcess),UTraceModuleEfsrv::ECFileManCopy3Return);
 
 		iAction = EInternalCopyFromHandle;
 		iNumberOfFilesProcessed = 0;
@@ -2736,8 +2649,7 @@
 		DoSynchronize(r);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY3RETURN3, "r %d", ret);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy3Return, MODULEUID, ret);
 	return(ret);
 	}
 
@@ -2773,8 +2685,8 @@
 @capability Dependent If the path for aNew begins with /Resource then Tcb capability is required.
 */
 	{
-	OstTraceExt4(TRACE_BORDER, EFSRV_ECFILEMANCOPY4, "this %x anOldSubs %x aSwitches %dstatus %x", (TUint) this, (TUint) anOld.SubSessionHandle(), (TUint) aSwitches, (TUint) &aStatus);
-	OstTraceData(TRACE_BORDER, EFSRV_ECFILEMANCOPY4_ENEWNAME, "NewName %S", aNew.Ptr(), aNew.Length()<<1);
+	TRACEMULT5(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy4, MODULEUID,
+		(TUint) this, anOld.SubSessionHandle(), aNew, aSwitches, (TUint) &aStatus);
 
 	TInt r;
 	if (iSwitches&KFManBusyFlag)
@@ -2787,8 +2699,7 @@
 		r = Copy(anOld,aNew,aSwitches);
 		}
 
-	OstTrace1(TRACE_BORDER, EFSRV_ECFILEMANCOPY4RETURN, "r %d", r);
-
+	TRACERET1(UTF::EBorder, UTraceModuleEfsrv::ECFileManCopy4Return, MODULEUID, r);
 	return(r);
 	}
 
@@ -2850,21 +2761,8 @@
 	{
 	TInt64 rem;
 #endif
-	TInt r;
-	if ((r = aSrcFile.Size(rem)) != KErrNone)
-		{
-		if(iStatus)
-			User::RequestComplete(iStatus,r);
-		return r;
-		}
-
-
-	if ((r = aDstFile.SetSize(rem)) != KErrNone)
-		{
-		if(iStatus)
-			User::RequestComplete(iStatus,r);
-		return r;
-		}
+	RETURNIFERRORD(r,aSrcFile.Size(rem),EFalse);
+	RETURNIFERROR(r, aDstFile.SetSize(rem),EFalse);
 
 	HBufC8* bufPtr = NULL;
 	bufPtr = AllocateBuffer(rem);