[FFmpeg-devel] [PATCH][RFC] variable frame sizes

[Eric Buehl]

v5c.patch

On Thu, Jun 18, 2009 at 10:46 AM, Michael Niedermayer <michaelni at gmx.at>wrote:

> On Thu, Jun 18, 2009 at 09:50:05AM -0700, Eric Buehl wrote:
> > v5b.patch
> >
> > Blocks of statements elsewhere are not always aligned.  When do you
> normally
> > do this?
>
> i do when i notice and i notice more code in your patch that woul be
> more readable where it aligned ...
> please fix this, id like to see this one commited as a bug related to
> changing pix_fmt will change the same code ...
>
> also i think we dont need a MAKE_EVEN macro, all people here are suppossed
> to know what &~1 does (the ones not knowing are not going to contribute
> usefull code yet)
>
> [...]
>
>
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> In fact, the RIAA has been known to suggest that students drop out
> of college or go to community college in order to be able to afford
> settlements. -- The RIAA
>
> -----BEGIN PGP SIGNATURE-----
> Version: GnuPG v1.4.9 (GNU/Linux)
>
> iD8DBQFKOn1rYR7HhwQLD6sRAsMSAJ9gmYf8yH1GnRFLFQ378d535i5+swCgiZkO
> 6+BTJR1zmoyjCKhunJY0ZKE=
> =WwZX
> -----END PGP SIGNATURE-----
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/ffmpeg-devel
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dynamic_frame_height_adjust_v5c.patch
Type: text/x-diff
Size: 5427 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090618/1dd63995/attachment.patch>