diff -r bc06d8566074 -r d5f04de580b7 mmlibs/mmfw/src/Client/Video/mediaclientvideodisplaybody.cpp --- a/mmlibs/mmfw/src/Client/Video/mediaclientvideodisplaybody.cpp Mon Mar 15 12:46:07 2010 +0200 +++ b/mmlibs/mmfw/src/Client/Video/mediaclientvideodisplaybody.cpp Wed Mar 31 23:56:23 2010 +0300 @@ -695,8 +695,8 @@ break; default: // Should never get to default unless there's been some programming error. - User::Invariant(); - break; + DEBUG_PRINTF2("CMediaClientVideoDisplayBody::SetBackgroundSurface --- Returned with error %d", KErrArgument); + return KErrArgument; } TReal32 viewportAspect = pixelAspectRatio * inputWidth / inputHeight; @@ -867,8 +867,8 @@ break; default: // Should never get to default unless there's been some programming error. - User::Invariant(); - break; + DEBUG_PRINTF2("CMediaClientVideoDisplayBody::SetBackgroundSurface --- Returned with error %d", KErrArgument); + return KErrArgument; } DEBUG_PRINTF5("CMediaClientVideoDisplayBody::SetBackgroundSurface - viewport2 %d,%d - %d,%d", viewport.iTl.iX, viewport.iTl.iY, viewport.iBr.iX, viewport.iBr.iY); @@ -963,8 +963,9 @@ break; default: // Should never get to default unless there's been some programming error. - User::Invariant(); - // This is never reached, just to keep the compiler happy + DEBUG_PRINTF2("CMediaClientVideoDisplayBody::ConvertRotation --- Failed due to %d bad aRotation argument", aRotation); + __ASSERT_DEBUG(FALSE, User::Invariant()); + // Use the normal option otherwise orientation = CFbsBitGc::EGraphicsOrientationNormal; } DEBUG_PRINTF2("CMediaClientVideoDisplayBody::ConvertRotation --- return %d", orientation);