[Ffmpeg-devel] [PATCH] cosmetic
Wed Nov 1 22:03:24 CET 2006
On Wed, 01 Nov 2006 21:37:08 +0100
Steve Lhomme <slhomme at divxcorp.com> wrote:
> Michael Niedermayer wrote:
> > Hi
> > On Tue, Oct 24, 2006 at 04:24:45PM +0200, Steve Lhomme wrote:
> >> Hi,
> >> This patch just fix the indentation in some parts of the code.
> >> Steve
> >> Index: libavcodec/g726.c
> >> ===================================================================
> >> --- libavcodec/g726.c (revision 6775)
> >> +++ libavcodec/g726.c (working copy)
> > [...]
> >> + int* F; /**< special table #2 */
> > [...]
> >> - int pk; /**< signs of prev. 2 sez + dq */
> >> + Float11 sr; /**< prev. reconstructed samples */
> > this doesnt look correct
> If you cut the patch, of course it doesn't look right. The last line
> doesn't replace the line just above but many lines above...
The cut was intentionnal and pretty clear. Notice the "int* F;".
> If you look at the patch in its entirety, it's fine.
No, it's wrong.
The point is that the following two lines don't have the same indentation:
+ int* F; /**< special table #2 */
+ Float11 sr; /**< prev. reconstructed samples */
More information about the ffmpeg-devel