diff -r c1f20ce4abcf -r 3e88ff8f41d5 kerneltest/e32test/heap/t_hcomp.cpp
--- a/kerneltest/e32test/heap/t_hcomp.cpp	Tue Aug 31 16:34:26 2010 +0300
+++ b/kerneltest/e32test/heap/t_hcomp.cpp	Wed Sep 01 12:34:56 2010 +0100
@@ -39,12 +39,7 @@
 	if (heap1==NULL)
 		return KErrNoMemory;
 
-	// Need this horrible construct because RVCT4 complains that the return is
-	// unreachable code.  Without it, though, other code analysers will complain
-	// that there is a missing return value!	
-	volatile TInt forever = 1;
-	
-	while(forever)
+	FOREVER
 		{
 		TUint8* ptr=(TUint8*)heap1->Alloc(heapsize);	// fail, compress, fail
 		User::After(1000);	// quite soon