Re: Porblem with avifile.......

On Thu, Jul 24, 2003 at 06:00:52AM +0000, lrodrigo wrote:
> I have this problem in the instalation process:
> 
>  -fomit-frame-pointer -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64 
> -D_GNU_SOURCE -pipe -c flvenc.c  -fPIC -DPIC -o flvenc.lo
> flvenc.c: In function `flv_write_trailer':
> flvenc.c:232: parse error before `file_size'
> flvenc.c:234: `flags' undeclared (first use in this function)
> flvenc.c:234: (Each undeclared identifier is reported only once
> flvenc.c:234: for each function it appears in.)
> flvenc.c:238: `file_size' undeclared (first use in this function)
> make[2]: *** [flvenc.lo] Error 1
> make[2]: Saliendo directorio 
> `/stuff/files/store/Apps_TARBALL/transcode/avifile-0.7-0.7.38/ffmpeg/libavformat'
> make[1]: *** [all-recursive] Error 1
> make[1]: Saliendo directorio 
> `/stuff/files/store/Apps_TARBALL/transcode/avifile-0.7-0.7.38/ffmpeg'
> make: *** [all-recursive] Error 1
> 
> Do you know what's the matter?

Yes I know - this was the bug of the ffmpeg which has sliped into this release
unfortunately (these are the results of the 'last minute' update.

This code has been unfortunately using gcc C extension where
you could declare variable anywhere in the source - in plain C you
have to declare them in front of execution block. So this extension
is more like C++ behaviour.

Well there are few possible solution - either you will put these
I guess to lines which declares them in the function begin.

Another one is to use CVS

Also you may upgrade your C compiler (gcc-3.3)

Also you may wait for next release - or if everything else will fail
you may try this patch (I've reversed it manualy I hope it will work)


--- flvenc.c    10 Jul 2003 09:48:38 -0000      1.1
+++ flvenc.c    11 Jul 2003 07:04:37 -0000      1.2
@@ -224,13 +224,15 @@
 
 static int flv_write_trailer(AVFormatContext *s)
 {
-    int64_t file_size;
-    int flags = 0;
-
     ByteIOContext *pb = &s->pb;
     FLVContext *flv = s->priv_data;
 
     Dump(flv,pb,1);
 
+    int64_t file_size = url_ftell(pb);
+    int flags = 0;
-    file_size = url_ftell(pb);
     flags |= flv->hasAudio ? 4 : 0;
     flags |= flv->hasVideo ? 1 : 0;
     url_fseek(pb, 4, SEEK_SET);



-- 
  .''`.
 : :' :    Zdenek Kabelac  kabi@{debian.org, users.sf.net, fi.muni.cz}
 `. `'           Debian GNU/Linux maintainer - www.debian.{org,cz}
   `-