diff -r b42b9ce90ea9 -r 661475905584 kerneltest/e32test/heap/t_hcomp.cpp
--- a/kerneltest/e32test/heap/t_hcomp.cpp	Fri Apr 23 22:02:01 2010 +0100
+++ b/kerneltest/e32test/heap/t_hcomp.cpp	Fri Apr 23 22:08:41 2010 +0100
@@ -39,7 +39,12 @@
 	if (heap1==NULL)
 		return KErrNoMemory;
 
-	FOREVER
+	// 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)
 		{
 		TUint8* ptr=(TUint8*)heap1->Alloc(heapsize);	// fail, compress, fail
 		User::After(1000);	// quite soon