diff -r 748ec5531811 -r 336bee5c2d35 charconvfw/charconvplugins/src/plugins/iscii.cpp
--- a/charconvfw/charconvplugins/src/plugins/iscii.cpp	Tue Aug 31 17:01:26 2010 +0300
+++ b/charconvfw/charconvplugins/src/plugins/iscii.cpp	Wed Sep 01 12:39:40 2010 +0100
@@ -420,8 +420,7 @@
 
 LOCAL_C void FindAndModifyBuffer(TDes8& aModifyBuffer)
 {
-	TInt ret = KErrNone;
-    RArray<TPtrC8> searchBuffer;
+	RArray<TPtrC8> searchBuffer;
 	RArray<TPtrC8> replaceBuffer;
 	
 	//If the passed buffer contains the replacement buffer,
@@ -429,62 +428,61 @@
 	//buffer rather it should get converted to replacement for
 	//unconvertible character.
 	
-	ret |= searchBuffer.Append(KReplacementForExplicitHalant().Right(1));
-	ret |= searchBuffer.Append(KReplacementForSoftHalant().Right(1));
-	ret |= searchBuffer.Append(KReplacementForOm().Right(1));
-	ret |= searchBuffer.Append(KReplacementForAvagraha().Right(1));
+	searchBuffer.Append(KReplacementForExplicitHalant().Right(1));
+	searchBuffer.Append(KReplacementForSoftHalant().Right(1));
+	searchBuffer.Append(KReplacementForOm().Right(1));
+	searchBuffer.Append(KReplacementForAvagraha().Right(1));
 	
-	ret |= searchBuffer.Append(KReplacementForVocalicRr().Right(1));
-	ret |= searchBuffer.Append(KReplacementForVocalicLl().Right(1));
-	ret |= searchBuffer.Append(KReplacementForVocalicLSign().Right(1));
-	ret |= searchBuffer.Append(KReplacementForVocalicLlSign().Right(1));
-	ret |= searchBuffer.Append(KReplacementForVocalicL().Right(1));
-	ret |= searchBuffer.Append(KReplacementForVocalicRrSign().Right(1));
+	searchBuffer.Append(KReplacementForVocalicRr().Right(1));
+	searchBuffer.Append(KReplacementForVocalicLl().Right(1));
+	searchBuffer.Append(KReplacementForVocalicLSign().Right(1));
+	searchBuffer.Append(KReplacementForVocalicLlSign().Right(1));
+	searchBuffer.Append(KReplacementForVocalicL().Right(1));
+	searchBuffer.Append(KReplacementForVocalicRrSign().Right(1));
 	
 	//All normal search buffers
-	ret |= searchBuffer.Append(KExplicitHalant().Mid(0));
-	ret |= searchBuffer.Append(KSoftHalant().Mid(0));
-	ret |= searchBuffer.Append(KOm().Mid(0));
-	ret |= searchBuffer.Append(KAvagraha().Mid(0));
+	searchBuffer.Append(KExplicitHalant().Mid(0));
+	searchBuffer.Append(KSoftHalant().Mid(0));
+	searchBuffer.Append(KOm().Mid(0));
+	searchBuffer.Append(KAvagraha().Mid(0));
 	
-	ret |= searchBuffer.Append(KVocalicRr().Mid(0));
-	ret |= searchBuffer.Append(KVocalicLl().Mid(0));
-	ret |= searchBuffer.Append(KVocalicLSign().Mid(0));
-	ret |= searchBuffer.Append(KVocalicLlSign().Mid(0));
-	ret |= searchBuffer.Append(KVocalicL().Mid(0));
-	ret |= searchBuffer.Append(KVocalicRrSign().Mid(0));
+	searchBuffer.Append(KVocalicRr().Mid(0));
+	searchBuffer.Append(KVocalicLl().Mid(0));
+	searchBuffer.Append(KVocalicLSign().Mid(0));
+	searchBuffer.Append(KVocalicLlSign().Mid(0));
+	searchBuffer.Append(KVocalicL().Mid(0));
+	searchBuffer.Append(KVocalicRrSign().Mid(0));
 	
 	//The replacement buffer for the odd cases to restrict the 
 	//replacement buffers not to get converted to the corresponding 
 	//unicode buffer
 	
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
 	
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
-	ret |= replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
+	replaceBuffer.Append(KIsciiUnconvertibleCharacter().Mid(0));
 	
 	//All normal replace buffers		
-	ret |= replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForOm().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
+	replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForOm().Mid(0));
+	replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
 	
 	
-	ret |= replaceBuffer.Append(KReplacementForVocalicRr().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicLl().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicLSign().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicLlSign().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicL().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicRrSign().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicRr().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicLl().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicLSign().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicLlSign().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicL().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicRrSign().Mid(0));
 	
-	__ASSERT_DEBUG(!ret, User::Panic(_L("RArray append failure"), ret));
 			
 	for(TInt counter=0;counter<searchBuffer.Count();counter++)
 	{
@@ -554,35 +552,32 @@
 
 LOCAL_C void NormalizeReturnValue(TUint& aReturnValue,const TDesC8& aBuffer)
 {
-	TInt ret = KErrNone;
-    RArray<TPtrC8> searchBuffer;
+	RArray<TPtrC8> searchBuffer;
 	RArray<TPtrC8> replaceBuffer;
 	
-	ret |= searchBuffer.Append(KExplicitHalant().Mid(0));
-	ret |= searchBuffer.Append(KSoftHalant().Mid(0));
-	ret |= searchBuffer.Append(KOm().Mid(0));
-	ret |= searchBuffer.Append(KAvagraha().Mid(0));
+	searchBuffer.Append(KExplicitHalant().Mid(0));
+	searchBuffer.Append(KSoftHalant().Mid(0));
+	searchBuffer.Append(KOm().Mid(0));
+	searchBuffer.Append(KAvagraha().Mid(0));
 	
-	ret |= searchBuffer.Append(KVocalicRr().Mid(0));
-	ret |= searchBuffer.Append(KVocalicLl().Mid(0));
-	ret |= searchBuffer.Append(KVocalicLSign().Mid(0));
-	ret |= searchBuffer.Append(KVocalicLlSign().Mid(0));
-	ret |= searchBuffer.Append(KVocalicL().Mid(0));
-	ret |= searchBuffer.Append(KVocalicRrSign().Mid(0));
+	searchBuffer.Append(KVocalicRr().Mid(0));
+	searchBuffer.Append(KVocalicLl().Mid(0));
+	searchBuffer.Append(KVocalicLSign().Mid(0));
+	searchBuffer.Append(KVocalicLlSign().Mid(0));
+	searchBuffer.Append(KVocalicL().Mid(0));
+	searchBuffer.Append(KVocalicRrSign().Mid(0));
 	
-	ret |= replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForOm().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
+	replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForOm().Mid(0));
+	replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
 	
-	ret |= replaceBuffer.Append(KReplacementForVocalicRr().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicLl().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicLSign().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicLlSign().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicL().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForVocalicRrSign().Mid(0));
-	
-	__ASSERT_DEBUG(!ret, User::Panic(_L("RArray append failure"), ret));
+	replaceBuffer.Append(KReplacementForVocalicRr().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicLl().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicLSign().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicLlSign().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicL().Mid(0));
+	replaceBuffer.Append(KReplacementForVocalicRrSign().Mid(0));
 	
 	DoNormalizeReturnValue(aReturnValue,aBuffer,searchBuffer,replaceBuffer);
 	searchBuffer.Reset();
@@ -688,14 +683,10 @@
 		return EFalse;
 	if(anIsciiBuffer[anIsciiBuffer.Length()-1] == 0xEF)
 	return ETrue;
-	
-	TInt appendret = KErrNone;
-	appendret |= searchBuffer.Append(KSoftHalant().Mid(0));
-	appendret |= searchBuffer.Append(KOm().Mid(0));
-	appendret |= searchBuffer.Append(KAvagraha().Mid(0));
-	appendret |= searchBuffer.Append(KExplicitHalant().Mid(0));
-	__ASSERT_DEBUG(!appendret, User::Panic(_L("RArray append failure"), appendret));
-	
+	searchBuffer.Append(KSoftHalant().Mid(0));
+	searchBuffer.Append(KOm().Mid(0));
+	searchBuffer.Append(KAvagraha().Mid(0));
+	searchBuffer.Append(KExplicitHalant().Mid(0));
 	TBool ret = EFalse;
 	TBool isNotTruncated =EFalse;
 	
@@ -1204,18 +1195,15 @@
 	RArray<TPtrC8> searchBuffer;
 	RArray<TPtrC8> replaceBuffer;
 	
-	TInt ret = KErrNone;
-	ret |= searchBuffer.Append(KExplicitHalant().Mid(0));
-	ret |= searchBuffer.Append(KSoftHalant().Mid(0));
-	ret |= searchBuffer.Append(KOm().Mid(0));
-	ret |= searchBuffer.Append(KAvagraha().Mid(0));
+	searchBuffer.Append(KExplicitHalant().Mid(0));
+	searchBuffer.Append(KSoftHalant().Mid(0));
+	searchBuffer.Append(KOm().Mid(0));
+	searchBuffer.Append(KAvagraha().Mid(0));
 	
-	ret |= replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForOm().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
-	
-	__ASSERT_DEBUG(!ret, User::Panic(_L("RArray append failure"), ret));
+	replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForOm().Mid(0));
+	replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
 	
 	for(TInt counter=0;counter<searchBuffer.Count();counter++)
 	{
@@ -1287,19 +1275,16 @@
 {
 	RArray<TPtrC8> searchBuffer;
 	RArray<TPtrC8> replaceBuffer;
-	TInt ret =KErrNone;
-	ret |= searchBuffer.Append(KExplicitHalant().Mid(0));
-	ret |= searchBuffer.Append(KSoftHalant().Mid(0));
-	ret |= searchBuffer.Append(KOm().Mid(0));
-	ret |= searchBuffer.Append(KAvagraha().Mid(0));
 	
-	ret |= replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForOm().Mid(0));
-	ret |= replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
+	searchBuffer.Append(KExplicitHalant().Mid(0));
+	searchBuffer.Append(KSoftHalant().Mid(0));
+	searchBuffer.Append(KOm().Mid(0));
+	searchBuffer.Append(KAvagraha().Mid(0));
 	
-	__ASSERT_DEBUG(!ret, User::Panic(_L("RArray append failure"), ret));
-	
+	replaceBuffer.Append(KReplacementForExplicitHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForSoftHalant().Mid(0));
+	replaceBuffer.Append(KReplacementForOm().Mid(0));
+	replaceBuffer.Append(KReplacementForAvagraha().Mid(0));
 	DoNormalizeReturnValue(aReturnValue,aBuffer,searchBuffer,replaceBuffer);
 	searchBuffer.Reset();
 	replaceBuffer.Reset();
@@ -1404,14 +1389,10 @@
 		return EFalse;
 	if(anIsciiBuffer[anIsciiBuffer.Length()-1] == 0xEF)
 	return ETrue;
-	
-	TInt appendret = KErrNone;
-	appendret |= searchBuffer.Append(KSoftHalant().Mid(0));
-	appendret |= searchBuffer.Append(KOm().Mid(0));
-	appendret |= searchBuffer.Append(KAvagraha().Mid(0));
-	appendret |= searchBuffer.Append(KExplicitHalant().Mid(0));
-	__ASSERT_DEBUG(!ret, User::Panic(_L("RArray append failure"), ret));
-	
+	searchBuffer.Append(KSoftHalant().Mid(0));
+	searchBuffer.Append(KOm().Mid(0));
+	searchBuffer.Append(KAvagraha().Mid(0));
+	searchBuffer.Append(KExplicitHalant().Mid(0));
 	TBool ret = EFalse;
 	TBool isNotTruncated =EFalse;