Thanks. Changes have been checked into Head & 210CayS.
Br,
--john
-----Original Message-----
From: ext Greg Wright [mailto:gwright real.com]
Sent: Monday, January 29, 2007 1:52 PM
To: Wei John (Nokia-TP-MSW/Dallas)
Cc: datatype-dev helixcommunity.org; clientapps-dev helixcommunity.org
Subject: Re: [datatype-dev] CR: MDF renderer - Creating a
config flag to
determine SynchronizeDecoding status
looks g00d.
--greg.
john.wei nokia.com wrote:
> Hi, I made some changes since last CR. I tied
configuration flag to
> each mimetype Thanks for review.
>
> --john
>
>
============================================================
==========
> ==
> ==============
> 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: john.wei nokia.com
>
> Reviewed by:
>
> Date: 25-Jan-2007
>
> Project: SymbianMmf_Rel
>
> ErrorId: EOVL-6XQPGR
>
> Synopsis: MDF renderer - Creating a config flag to
determine
> SynchronizeDecoding status
>
> Currently, SynchronizeDecoding is set to be True.
Decoder skips some
> b-frames when it lacks of speed. However, turning
SynchronizeDecoding
> to be False could fix this visual defect.
>
> Some people want to see good quality video frames even
though they are
> dropped heavily due to slowness of decoding. Other
people want to see
> less dropped video frames even though it means they
have visual
defects.
> Setting SynchronizeDecoding to be false satisfies the
first group
> while setting to be true satisfies the second group. In
order to keep
> this flexiblity, this CR proposes to create a
configuration flag to
> determine the SynchronizeDecoding status.
>
> Files Modified:
>
>
/cvsroot/clientapps/symbiancommon/config/R1_Mobile_4_0_Facto
ry.cfg
>
/cvsroot/datatype/mdf/video/renderer/mdfvideoadapter.cpp
>
/cvsroot/datatype/mdf/video/format/common/mdfpayloadformat.c
pp
>
/cvsroot/datatype/mdf/video/format/common/pub/mdfpayloadform
at.h
>
/cvsroot/datatype/mdf/video/format/h263/mdfh263payloadformat
.cpp
>
/cvsroot/datatype/mdf/video/format/h263/pub/mdfh263payloadfo
rmat.h
>
/cvsroot/datatype/mdf/video/format/h264/mdfh264payloadformat
.cpp
>
/cvsroot/datatype/mdf/video/format/h264/pub/mdfh264payloadfo
rmat.h
>
/cvsroot/datatype/mdf/video/format/mp4/mdfmp4payloadformat.c
pp
>
/cvsroot/datatype/mdf/video/format/mp4/pub/mdfmp4payloadform
at.h
>
/cvsroot/datatype/mdf/video/format/rm/mdfrvxpayloadformat.cp
p
>
/cvsroot/datatype/mdf/video/format/rm/pub/mdfrvxpayloadforma
t.h
>
> Files Added: None
>
> Image Size and Heap Use impact: minor
>
> Module Release testing (STIF) : MRT test cases run and
passed.
>
> Root Cause: Implementation.
>
> Test case(s) Added : No
>
> Memory leak check performed : Yes. No new leaks
introduced.
>
> Platforms and Profiles Build Verified:
helix-client-s60-32-mmf-mdf-arm
>
> Platforms and Profiles Functionality verified: armv5,
winscw
>
> Branch: Head & 210CayS
>
>
>
>
------------------------------------------------------------
----------
> --
>
> _______________________________________________
> Datatype-dev mailing list
> Datatype-dev helixcommunity.org
> http://lists.helixcommunity.org/mailman/listinfo/da
tatype-dev
_______________________________________________
Clientapps-dev mailing list
Clientapps-dev helixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/
clientapps-dev
|