Re: [libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-23 Thread Diego Biurrun
On Sun, Jul 22, 2012 at 08:46:10PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com Write out the NAL decoding loops in full so that they are easier to parse for a preprocessor without it having to be aware of macros or other such things in C code. This also makes

Re: [libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-23 Thread Ronald S. Bultje
Hi, On Mon, Jul 23, 2012 at 2:05 AM, Diego Biurrun di...@biurrun.de wrote: On Sun, Jul 22, 2012 at 08:46:10PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com Write out the NAL decoding loops in full so that they are easier to parse for a preprocessor without it

Re: [libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-23 Thread Kostya Shishkov
On Mon, Jul 23, 2012 at 07:11:49AM -0700, Ronald S. Bultje wrote: Hi, On Mon, Jul 23, 2012 at 2:05 AM, Diego Biurrun di...@biurrun.de wrote: On Sun, Jul 22, 2012 at 08:46:10PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje rsbul...@gmail.com Write out the NAL decoding loops in

Re: [libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-23 Thread Ronald S. Bultje
Hi, On Mon, Jul 23, 2012 at 7:14 AM, Kostya Shishkov kostya.shish...@gmail.com wrote: On Mon, Jul 23, 2012 at 07:11:49AM -0700, Ronald S. Bultje wrote: Hi, On Mon, Jul 23, 2012 at 2:05 AM, Diego Biurrun di...@biurrun.de wrote: On Sun, Jul 22, 2012 at 08:46:10PM -0700, Ronald S. Bultje

Re: [libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-23 Thread Måns Rullgård
Kostya Shishkov kostya.shish...@gmail.com writes: On Mon, Jul 23, 2012 at 07:11:49AM -0700, Ronald S. Bultje wrote: Hi, On Mon, Jul 23, 2012 at 2:05 AM, Diego Biurrun di...@biurrun.de wrote: On Sun, Jul 22, 2012 at 08:46:10PM -0700, Ronald S. Bultje wrote: From: Ronald S. Bultje

Re: [libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-23 Thread Kostya Shishkov
On Mon, Jul 23, 2012 at 07:16:53AM -0700, Ronald S. Bultje wrote: Hi, On Mon, Jul 23, 2012 at 7:14 AM, Kostya Shishkov kostya.shish...@gmail.com wrote: On Mon, Jul 23, 2012 at 07:11:49AM -0700, Ronald S. Bultje wrote: Hi, On Mon, Jul 23, 2012 at 2:05 AM, Diego Biurrun di...@biurrun.de

[libav-devel] [PATCH] h264: refactor NAL decode loop

2012-07-23 Thread Luca Barbato
From: Ronald S. Bultje rsbul...@gmail.com Write out the NAL decoding loops in full so that they are easier to parse for a preprocessor without it having to be aware of macros or other such things in C code. This also makes the code more readable. Signed-off-by: Luca Barbato lu_z...@gentoo.org

Re: [libav-devel] [PATCH] h264: refactor NAL decode loop

2012-07-23 Thread Ronald S. Bultje
Hi, On Mon, Jul 23, 2012 at 7:29 AM, Luca Barbato lu_z...@gentoo.org wrote: From: Ronald S. Bultje rsbul...@gmail.com Write out the NAL decoding loops in full so that they are easier to parse for a preprocessor without it having to be aware of macros or other such things in C code. This

[libav-devel] [PATCH] h264: refactor NAL decode loop.

2012-07-22 Thread Ronald S. Bultje
From: Ronald S. Bultje rsbul...@gmail.com Write out the NAL decoding loops in full so that they are easier to parse for a preprocessor without it having to be aware of macros or other such things in C code. This also makes the code more readable. --- libavcodec/h264.c | 42