diff -r b42b9ce90ea9 -r 661475905584 kerneltest/e32test/system/t_ctrap.cpp --- a/kerneltest/e32test/system/t_ctrap.cpp Fri Apr 23 22:02:01 2010 +0100 +++ b/kerneltest/e32test/system/t_ctrap.cpp Fri Apr 23 22:08:41 2010 +0100 @@ -55,9 +55,11 @@ #include #include +#if defined(_DEBUG) + const TInt KInitialCount=2; + const TInt KInitialCountAll=3; +#endif -const TInt KInitialCount=2; -const TInt KInitialCountAll=3; const TInt KLeaveValue=0x12345678; const TInt KMaxAlloc=6; @@ -106,12 +108,10 @@ LOCAL_C void ReallocateStackL() { - TInt n = 0; for(TInt i = 0; i < KMaxAlloc; ++i) { - HBufC *p1 = HBufC::NewLC(4); //Stack re-allocation will be performed due to the additional objects pushed + (void)HBufC::NewLC(4); //Stack re-allocation will be performed due to the additional objects pushed //into the cleanup stack - n = p1->Length(); //include this line to avoid warnigs for unused "p1" variable } test.Printf(_L("ReallocateStackL(): PopAndDestroy KMaxAlloc pointers\n")); CleanupStack::PopAndDestroy(KMaxAlloc); @@ -130,12 +130,10 @@ { RDebug::Printf("~CTest3(): Modify Cleanup stack by pushing items"); - TInt n = 0; for(TInt i = 0; i < KMaxAlloc; ++i) { - HBufC *p1 = HBufC::NewLC(4); //Stack re-allocation will be performed due to the additional objects pushed + HBufC::NewLC(4); //Stack re-allocation will be performed due to the additional objects pushed //into the cleanup stack - n = p1->Length(); //include this line to avoid warnigs for unused "p1" variable } } @@ -148,13 +146,11 @@ CleanupStack::PopAndDestroy(); } -LOCAL_C TInt PanicStackModifiedFn(TAny* aNopFn) +LOCAL_C TInt PanicStackModifiedFn(TAny* /*aNopFn*/) { __UHEAP_MARK; CTrapCleanup* cleanup = CTrapCleanup::New(); - aNopFn = NULL; //avoid warnings for unused "aNopFn" variable - TInt err = KErrNoMemory; RDebug::Printf("PanicStackModifiedFn(): call TRAP(err, ModifyStack())"); @@ -1091,9 +1087,11 @@ // when we do the cleanup. This test only works in debug mode. // __UHEAP_FAILNEXT(1); +#if defined(_DEBUG) TRAPD(r,pC->PushL(p6)); -#if defined(_DEBUG) test(r==KErrNoMemory); +#else + TRAP_IGNORE(pC->PushL(p6)); #endif __UHEAP_CHECK(KInitialCount+6); pC->PopAndDestroyAll(); @@ -1300,7 +1298,7 @@ // A cleanup operation which uses a trap harness and the cleanup stack // { - TRAPD(ignore,useCleanupStackL()) + TRAP_IGNORE(useCleanupStackL()); } LOCAL_C void addReentrantItemL() @@ -1424,7 +1422,7 @@ test.Next(_L("Check the object has closed")); __KHEAP_CHECK(0); - TRAPD(r, testAutoCloseL()); + TRAP_IGNORE(testAutoCloseL()); test.Next(_L("Check object has been closed and cleaned up after leave")); __KHEAP_MARKEND; test.End();