[FFmpeg-devel] [PATCH] Support playing SMV files.

[Michael Niedermayer]

On Mon, Oct 10, 2011 at 05:44:22PM +0200, Reimar Döffinger wrote:
> On Sun, Oct 09, 2011 at 07:44:49PM +0200, Michael Niedermayer wrote:
> > On Sun, Oct 09, 2011 at 04:02:20PM +0200, Reimar Döffinger wrote:
> > > Audio and video are interleaved via stream cur_dts - no idea how
> > > reliable this is.
> > > It also cannot display the video properly - it is stored with
> > > about 15 in a single JPEG frame, I cannot think of a reasonable
> > > way to implement this.
> > > Samples: http://samples.mplayerhq.hu/smv/
> > > Format description: http://wiki.multimedia.cx/index.php?title=SMV
> > > 
> > > Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> > > ---
> > >  libavformat/wav.c |   91 ++++++++++++++++++++++++++++++++++++++++++++++++++++-
> > >  1 files changed, 90 insertions(+), 1 deletions(-)
> > 
> > should be ok
> 
> Pushed.
> 
> > also remind me to look into the jpeg issue
> 
> As far as came was: separate codec-id, height of single frame in
> extradata.
> Then in JPEG decoder either
> 1) adjust data pointers in output, actually decoding only once in n
> frames. Might cause issues with custom get_buffer
> 2) copy each frame from internal to external buffer.

have you confirmed that these are normal jpegs?
if so i would suggest the following implementation:

create a new AVCodec smv_jpeg
in it avcodec_open() a jpeg decoder, decode the frame and then return
the frames one by one referencing data from the decoded frame and
do not set CODEC_CAP_DR1

this also keeps the jpeg decoder free of convoluted hacks to support
this

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I know you won't believe me, but the highest form of Human Excellence is
to question oneself and others. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20111010/e3295bc2/attachment.asc>