[prev in list] [next in list] [prev in thread] [next in thread] 

List:       helix-datatype-cvs
Subject:    [Datatype-cvs] mdf/video/renderer mdfvideoadapter.cpp,1.71,1.72
From:       gashish () helixcommunity ! org
Date:       2007-09-27 21:49:59
Message-ID: 200709272150.l8RLoO4L010062 () mailer ! progressive-comp ! com
[Download RAW message or body]

Update of /cvsroot/datatype/mdf/video/renderer
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv14781

Modified Files:
	mdfvideoadapter.cpp 
Log Message:
Nokia submits this code under the terms of a commercial contribution agreement with \
RealNetworks, and I am authorized to contribute this code under said agreement."

Modified by:  ashish.as.gupta@nokia.com

Reviewed by: greg.wright@real.com

Date: 27-Sept-2007

Project: SymbianMmf_Rel

TSW: JAIN-76YJ3K

Synopsis: 

Using EColor16MU may not be correct on some devices. In case of 16M colors following \
modes are possible.

EColor16MU,   EColor16MA,    EColor16MAP. Correct method to get the 16M color mode is \
by calling CFbsDevice::DisplayMode16M().

For the completeness, missing low color values are also added.


Root Cause of the problem: OS implementation


Files Modified:
    datatype/mdf/video/renderer/mdfvideoadapter.cpp

Image Size and Heap Use impact: 
    none

Module Release testing (STIF): Yes.

Test case(s) Added: none

Memory leak check performed: Yes, no new leaks introduced.

Platforms and Profiles Build Verified: 
    armv5 / helix-client-s60-32-mmf-mdf-arm
    winscw / helix-client-s60-32-mmf-mdf-arm

Platforms and Profiles Functionality verified: 
    armv5 / helix-client-s60-32-mmf-mdf-arm

Branch: 
    HEAD and hxclient_2_1_0_cayennes



Index: mdfvideoadapter.cpp
===================================================================
RCS file: /cvsroot/datatype/mdf/video/renderer/mdfvideoadapter.cpp,v
retrieving revision 1.71
retrieving revision 1.72
diff -u -d -r1.71 -r1.72
--- mdfvideoadapter.cpp	15 Aug 2007 22:37:40 -0000	1.71
+++ mdfvideoadapter.cpp	27 Sep 2007 21:49:56 -0000	1.72
@@ -495,12 +495,21 @@
         {
             switch (colors)
             {
+            case 16:
+                m_DisplayMode = EColor16;
+                break;
+            case 256:
+                m_DisplayMode = EColor256;
+                break;
+            case 4096:
+                m_DisplayMode = EColor4K;
+                break;
             case 65536:
                 m_DisplayMode = EColor64K;
                 break;
             case 16777216:
             default:
-                m_DisplayMode = EColor16MU;
+                m_DisplayMode = CFbsDevice::DisplayMode16M();
             }
             retVal = HXR_OK;
         }


_______________________________________________
Datatype-cvs mailing list
Datatype-cvs@helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/datatype-cvs


[prev in list] [next in list] [prev in thread] [next in thread]