[FFmpeg-soc] [soc]: r815 - in dirac: ffmpeg.diff libavcodec/dirac.c
Marco Gerards
mgerards at xs4all.nl
Wed Aug 15 17:34:51 CEST 2007
Aurelien Jacobs <aurel at gnuage.org> writes:
> On Wed, 15 Aug 2007 16:54:09 +0200
> Diego Biurrun <diego at biurrun.de> wrote:
>
>> 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.
>>
>> I'm confident that the task is well within your capabilities :)
>>
>> It's really not hard, move everything that is within CONFIG_ENCODERS
>> (which should really be CONFIG_DIRAC_ENCODER, btw) to a new file. Do
>> the same for CONFIG_DECODERS, keep the rest in the common file. If you
>> have trouble, we'll be here to give you a hand.
>
> I think Marco has no problem about the splitting itself. His problem is
> that the encoder is a work in progress and that he currently still don't
> know what will really be shared between then encoder and the decoder.
> So I agree that the encoder and decoder must be split, but I think this
> can be done a little bit later, when the encoder will be more advanced.
Yes, that is exactly what I meant :-)
I really want to split this. The file is getting huge already... But
first I am to sort out what has to be split.
Thanks,
Marco
More information about the FFmpeg-soc
mailing list