Re: [FFmpeg-devel] [PATCH] Reorganized libpostproc code

2015-03-11 Thread Ronald S. Bultje
Hi, On Wed, Mar 11, 2015 at 2:06 AM, James Almer wrote: > On 11/03/15 12:57 AM, Tucker DiNapoli wrote: > > From: Tucker DiNapoli > > A couple comments below. > > > > > The only changes were formating and moving code. > > This needs to be split into several different patches. Also, why does > "m

Re: [FFmpeg-devel] [PATCH] Reorganized libpostproc code

2015-03-11 Thread Michael Niedermayer
Hi On Tue, Mar 10, 2015 at 11:57:56PM -0400, Tucker DiNapoli wrote: > From: Tucker DiNapoli > > The only changes were formating and moving code. > --- > libpostproc/postprocess.c | 436 ++-- > libpostproc/postprocess_c.c| 1328 > >

Re: [FFmpeg-devel] [PATCH] Reorganized libpostproc code

2015-03-10 Thread James Almer
On 11/03/15 12:57 AM, Tucker DiNapoli wrote: > From: Tucker DiNapoli A couple comments below. > > The only changes were formating and moving code. This needs to be split into several different patches. Also, why does "moving code" end up with the addition of one thousand new lines? > --- >

[FFmpeg-devel] [PATCH] Reorganized libpostproc code

2015-03-10 Thread Tucker DiNapoli
From: Tucker DiNapoli The only changes were formating and moving code. --- libpostproc/postprocess.c | 436 ++-- libpostproc/postprocess_c.c| 1328 libpostproc/postprocess_internal.h | 30 +- libpostproc/postprocess_template.c | 1