[FFmpeg-devel] [RFC] Split libavformat

[Aurelien Jacobs]

Luca Abeni wrote:

> Luca Abeni wrote:
> > Diego Biurrun wrote:
> > [...]
> >>> +    REGISTER_INDEVOUTDEV (AUDIO_BEOS, audio_beos);
> >> Maybe INOUTDEV would sound better.
> > 
> > Ok. configure does not seem to like it; I am investigating the
> > reason and I hope to fix this soon.
> 
> ...And this is the new patch. I hope I addressed all the comments.
> I even went one step further and changed all the *_muxer and *_demuxer
> structure names to *_outdev and *_indev (so the patch is bigger, now).
> If this is wrong, I will remove this last change.
> 
> [...]
> 
> +#ifndef FFMPEG_AVDEVICE_H
> +#define FFMPEG_AVDEVICE_H
> +
> +#define LIBAVDEVICE_VERSION_INT ((51<<16)+(19<<8)+0)
> +#define LIBAVDEVICE_VERSION     51.19.0
> +#define LIBAVDEVICE_BUILD       LIBAVDEVICE_VERSION_INT

Why do you set version to 51.19.0 ???
It looks like an old avformat version.

> [...]
> 
> Index: ffmpeg/ffmpeg.c
> ===================================================================
> --- ffmpeg.orig/ffmpeg.c	2007-11-22 08:45:38.000000000 +0100
> +++ ffmpeg/ffmpeg.c	2007-11-22 08:46:04.000000000 +0100
> @@ -28,6 +28,7 @@
>  #include <signal.h>
>  #include <limits.h>
>  #include "avformat.h"
> +#include "avdevice.h"
>  #include "swscale.h"
>  #include "framehook.h"
>  #include "opt.h"
> @@ -3884,7 +3885,7 @@
>      int i;
>      int64_t ti;
>  
> -    av_register_all();
> +    avdevice_register_all();

It should be possible to build ffmpeg/ffplay without linking
them to libavdevice (needs the call to avdevice_register_all
to be conditionnal, but can be done later).
This means that you shouldn't rely on avdevice_register_all()
to register formats and codecs.

Aurel