[FFmpeg-devel] [PATCH] libavcodec/sanm: implement codec37 subcodec1

[Manuel Lauss]

Interesting, which compiler do you use?  I see no such errors with gcc-15.
I'll redo the patch, thanks for testing!

Manuel

On Thu, Jan 9, 2025 at 4:02 AM Michael Niedermayer
<michael at niedermayer.cc> wrote:
>
> Hi
>
> On Tue, Jan 07, 2025 at 03:46:09PM +0100, Manuel Lauss wrote:
> > RLE-compressed stream of motion vector indices and a special opcode
> > to fill a block with data from the source stream.
> >
> > It is used in the LucasArts "Full Throttle" blink*.san animations.
> >
> > Signed-off-by: Manuel Lauss <manuel.lauss at gmail.com>
> > ---
> >  libavcodec/sanm.c | 64 ++++++++++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 63 insertions(+), 1 deletion(-)
>
> libavcodec/sanm.c: In function ‘old_codec37’:
> libavcodec/sanm.c:669:9: error: a label can only be part of a statement and a declaration is not a statement
>   669 |         int run = 0;
>       |         ^~~
> libavcodec/sanm.c:670:9: error: expected expression before ‘int’
>   670 |         int len = -1;
>       |         ^~~
> libavcodec/sanm.c:671:9: warning: ISO C90 forbids mixed declarations and code [-Wdeclaration-after-statement]
>   671 |         int code = 0;
>       |         ^~~
> libavcodec/sanm.c:676:21: error: ‘len’ undeclared (first use in this function)
>   676 |                 if (len < 0) {
>       |                     ^~~
> libavcodec/sanm.c:676:21: note: each undeclared identifier is reported only once for each function it appears in
> make: *** [ffbuild/common.mak:81: libavcodec/sanm.o] Error 1
>
>
> [...]
> --
> Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
>
> Nations do behave wisely once they have exhausted all other alternatives.
> -- Abba Eban
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".