[FFmpeg-soc] [soc]: r815 - in dirac: ffmpeg.diff libavcodec/dirac.c

Michael Niedermayer michaelni at gmx.at
Wed Aug 15 17:09:51 CEST 2007


Hi

On Wed, Aug 15, 2007 at 04:49:09PM +0200, Marco Gerards wrote:
> Diego Biurrun <diego at biurrun.de> writes:
> 
> > On Wed, Aug 15, 2007 at 04:03:59PM +0200, marco wrote:
> >> 
> >> Log:
> >> basic code for the encoder
> >> 
> >> --- dirac/ffmpeg.diff	(original)
> >> +++ dirac/ffmpeg.diff	Wed Aug 15 16:03:59 2007
> >> @@ -2937,14 +2958,14 @@ AVCodec dirac_decoder = {
> >>  
> >> +#ifdef CONFIG_ENCODERS
> >> +AVCodec dirac_encoder = {
> >> +    "dirac",
> >> +    CODEC_TYPE_VIDEO,
> >> +    CODEC_ID_DIRAC,
> >> +    sizeof(DiracContext),
> >> +    encode_init,
> >> +    encode_frame,
> >> +    encode_end,
> >> +};
> >> +#endif
> >
> > The #ifdefs remind me: Please split the encoder off into a separate
> > file.
> 
> Would you mind if I waited a bit with this?  Because I am not familiar
> with encoders (and thus do not know which code can be shared), I do
> not actually know what should be split off.  So personally I rather
> wait a bit instead of moving functions back and forth.

ok

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

Republics decline into democracies and democracies degenerate into
despotisms. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20070815/b0f5662b/attachment.pgp>


More information about the FFmpeg-soc mailing list