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

List:       helix-datatype-cvs
Subject:    [Datatype-cvs] xps/fileformat CXPSFileformat.cpp, 1.1.1.1.2.10,
From:       joaquincab () helixcommunity ! org
Date:       2009-01-29 15:48:56
Message-ID: 200901291551.n0TFpu4k007638 () mailer ! progressive-comp ! com
[Download RAW message or body]

Update of /cvsroot/datatype/xps/fileformat
In directory cvs01.internal.helixcommunity.org:/tmp/cvs-serv26372

Modified Files:
      Tag: hxclient_2_1_0_cayennes
	CXPSFileformat.cpp 
Log Message:
CR: AKRY-7MC8BL: Getting CMDFDevVideoClient::Pause after streamEnd

"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:  ext-joaquin.1.cabrera@nokia.com; rajesh.rathinasamy@nokia.com           \
 Reviewed by:
Date: 01/28/2009
ErrorId: AKRY-7MC8BL
Project: SymbianMmf_rel
             
Synopsis: 
Getting CMDFDevVideoClient::Pause after streamEnd

Overview:
There is a CMDFDevVideoClient::Pause because a buffer is dry. The last happens \
because there is an active stream in XPS.

The DispatchPkt() function dispatches the packets of all streams. But there is a case \
in which one stream blocks the PacketReady  call from the other. The blocking state \
occurs near the end of the clip when one stream is empty and done (ie, stream 0), and \
the next stream (ie. 1) is tried to be dispatched before is called the EndOfStream() \
of the first stream (ie, 0).


Code Change:
Add a DispatchPkt() function in EndStream() to dispatch any outstanding packet \
requests.


Files modified & changes:
\datatype\xps\fileformat\CXPSFileformat.cpp


Image Size and Heap Use impact: None

Module Release testing (STIF) : Passed

Test case(s) Added  : None

File(s) Added  : None

Memory leak check performed : Passed, No leaks found
             
Platforms and Profiles Build Verified: helix-client-s60-32-mmf-mdf-arm

Platforms and Profiles Functionality verified: armv5
             
Branch: HEAD, 221CayS, 210CayS



--- CXPSFileformat.cpp	Wed Aug 06 12:57:21 2008
+++ CXPSFileformat.cpp	Wed Jan 28 15:33:14 2009
@@ -698,6 +698,9 @@
     {
         m_pFFResponse->StreamDone(uStreamNumber);
     }
+	
+	//  Dispatch if there is any outstanding packet requests.
+	DispatchPkt(0);
 }
 


Thanks,
Joaquin 


Index: CXPSFileformat.cpp
===================================================================
RCS file: /cvsroot/datatype/xps/fileformat/CXPSFileformat.cpp,v
retrieving revision 1.1.1.1.2.10
retrieving revision 1.1.1.1.2.11
diff -u -d -r1.1.1.1.2.10 -r1.1.1.1.2.11
--- CXPSFileformat.cpp	19 Oct 2008 04:57:28 -0000	1.1.1.1.2.10
+++ CXPSFileformat.cpp	29 Jan 2009 15:48:54 -0000	1.1.1.1.2.11
@@ -716,6 +716,9 @@
     {
         m_pFFResponse->StreamDone(uStreamNumber);
     }
+	
+	//  Dispatch if there is any outstanding packet requests.
+	DispatchPkt(0);
 }
 
 //


_______________________________________________
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]