[Ffmpeg-devel] [PATCH] Basic H.264 encoder

[Guillaume POIRIER]

Hi,

On 4/12/06, Panagiotis Issaris <takis.issaris at uhasselt.be> wrote:

> We would like some suggestions about how we should proceed
> further to integrate a LGPL H.264 encoder into libavcodec. Is
> there an encoder we can use as an example about how the code
> should be structured? For example, we've noticed a large
> difference in the way the H.261 (and also H.263) and Snow codecs
> are organized.

I haven't tested you patch [yet], but here's a suggestion after having
a quick look at your patch: Could you maybe generate the big tables on
runtime? That would significantly reduce the size of your patch,
simplify it, and probably reduce the likelyhood of typos.

just my 2 cents...

Guillaume
--
I am disillusioned enough to know that no man's opinion on any subject
is worth a damn unless backed up with enough genuine information to
make him really know what he's talking about.

-- H. P. Lovecraft (about the flamewars on FFmpeg and MPlayer-dev mailing lists)
http://www.brainyquote.com/quotes/quotes/h/hplovecr278144.html