Resend - CR: MDF Video Renderer - always appending videostreaming header to first frame

Looks good.

--greg.


john.weinokia.com wrote:
> "Nokia submits this code under the terms of a
commercial contribution
> agreement with Real Networks, and I am authorized to
contribute this
> code under said agreement."
> 
> Modified by:  John Wei
> 
> Reviewed by: 
> 
> Date: 14-09-2006.
> 
> Error ID: JWEI-6TKUDJ
> 
> Project: Helix plugin for Symbian
> 
> Synopsis: MDF Video Renderer - always appending video
streaming header
> to first frame
> 
> This CR proposes following error corrections or/and
improvements:
> 
> Current implementation does not send video stream
header to decoder
> along with first frame when header is sent by a
seperate function call
> (i.e. ConfigureDecoderL()). This is not what DevVideo
guideline says.
> Guideline asks that this header be attached to first
frame always
> regardless ConfigureDecoderL() is used or not.
> 
> Files Modified:
>
/cvsroot/clientapps/symbiancommon/config/R1_Mobile_4_0_Facto
ry.cfg
>
/cvsroot/datatype/mdf/video/format/common/pub/mdfpayloadform
at.h
>
/cvsroot/datatype/mdf/video/format/common/mdfpayloadformat.c
pp
>
/cvsroot/datatype/mdf/video/format/h263/pub/mdfh263payloadfo
rmat.h
>
/cvsroot/datatype/mdf/video/format/h263/mdfh263payloadformat
.cpp
>
/cvsroot/datatype/mdf/video/format/h264/pub/mdfh264payloadfo
rmat.h
>
/cvsroot/datatype/mdf/video/format/h264/mdfh264payloadformat
.cpp
>
/cvsroot/datatype/mdf/video/format/mp4/pub/mdfmp4payloadform
at.h
>
/cvsroot/datatype/mdf/video/format/mp4/mdfmp4payloadformat.c
pp
>
/cvsroot/datatype/mdf/video/format/rm/pub/mdfrvxpayloadforma
t.h
>
/cvsroot/datatype/mdf/video/format/rm/mdfrvspayloadformat.cp
p
>
/cvsroot/datatype/mdf/video/renderer/pub/mdfvideoadapter.h
>
/cvsroot/datatype/mdf/video/renderer/mdfvideoadapter.cpp
> 
> Image Size and Heap Use impact: minor
> 
> Platforms and Profiles Build Verified:
helix-client-s60-mmf-mdf-arm
> 
> Platforms and Profiles Functionality verified: winscw,
armv5
> 
> Branch: hxclient_2_1_0_cayennes_restricted

_______________________________________________
Clientapps-dev mailing list
Clientapps-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/
clientapps-dev

 
Thanks Greg,

Changes have checked into both head and
hxclient_2_1_0_cayennes

Br,

--john

-----Original Message-----
From: ext Greg Wright [mailto:gwrightreal.com] 
Sent: Monday, September 25, 2006 3:23 PM
To: Wei John (Nokia-TP-MSW/Dallas)
Cc: datatype-devhelixcommunity.org; clientapps-devhelixcommunity.org
Subject: Re: Resend - CR: MDF Video Renderer - always
appending
videostreaming header to first frame

Looks good.

--greg.


john.weinokia.com wrote:
> "Nokia submits this code under the terms of a
commercial contribution 
> agreement with Real Networks, and I am authorized to
contribute this 
> code under said agreement."
> 
> Modified by:  John Wei
> 
> Reviewed by: 
> 
> Date: 14-09-2006.
> 
> Error ID: JWEI-6TKUDJ
> 
> Project: Helix plugin for Symbian
> 
> Synopsis: MDF Video Renderer - always appending video
streaming header

> to first frame
> 
> This CR proposes following error corrections or/and
improvements:
> 
> Current implementation does not send video stream
header to decoder 
> along with first frame when header is sent by a
seperate function call

> (i.e. ConfigureDecoderL()). This is not what DevVideo
guideline says.
> Guideline asks that this header be attached to first
frame always 
> regardless ConfigureDecoderL() is used or not.
> 
> Files Modified:
>
/cvsroot/clientapps/symbiancommon/config/R1_Mobile_4_0_Facto
ry.cfg
>
/cvsroot/datatype/mdf/video/format/common/pub/mdfpayloadform
at.h
>
/cvsroot/datatype/mdf/video/format/common/mdfpayloadformat.c
pp
>
/cvsroot/datatype/mdf/video/format/h263/pub/mdfh263payloadfo
rmat.h
>
/cvsroot/datatype/mdf/video/format/h263/mdfh263payloadformat
.cpp
>
/cvsroot/datatype/mdf/video/format/h264/pub/mdfh264payloadfo
rmat.h
>
/cvsroot/datatype/mdf/video/format/h264/mdfh264payloadformat
.cpp
>
/cvsroot/datatype/mdf/video/format/mp4/pub/mdfmp4payloadform
at.h
>
/cvsroot/datatype/mdf/video/format/mp4/mdfmp4payloadformat.c
pp
>
/cvsroot/datatype/mdf/video/format/rm/pub/mdfrvxpayloadforma
t.h
>
/cvsroot/datatype/mdf/video/format/rm/mdfrvspayloadformat.cp
p
>
/cvsroot/datatype/mdf/video/renderer/pub/mdfvideoadapter.h
>
/cvsroot/datatype/mdf/video/renderer/mdfvideoadapter.cpp
> 
> Image Size and Heap Use impact: minor
> 
> Platforms and Profiles Build Verified:
helix-client-s60-mmf-mdf-arm
> 
> Platforms and Profiles Functionality verified: winscw,
armv5
> 
> Branch: hxclient_2_1_0_cayennes_restricted

_______________________________________________
Clientapps-dev mailing list
Clientapps-devhelixcommunity.org
http://lists.helixcommunity.org/mailman/listinfo/
clientapps-dev