| Message ID | 20200128071711.23009-1-onemda@gmail.com |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [FFmpeg-devel] avformat/tty: add probe function | expand |
| Context | Check | Description |
|---|---|---|
| andriy/ffmpeg-patchwork | fail | Make failed |
On Tue, Jan 28, 2020 at 08:17:11AM +0100, Paul B Mahol wrote: > Signed-off-by: Paul B Mahol <onemda@gmail.com> > --- > libavformat/tty.c | 18 +++++++++++++++++- > 1 file changed, 17 insertions(+), 1 deletion(-) > > diff --git a/libavformat/tty.c b/libavformat/tty.c > index 8d48f2c45c..c127500fa1 100644 > --- a/libavformat/tty.c > +++ b/libavformat/tty.c > @@ -24,6 +24,8 @@ > * Tele-typewriter demuxer > */ > > +#include <ctype.h> > + > #include "libavutil/intreadwrite.h" > #include "libavutil/avstring.h" > #include "libavutil/log.h" > @@ -34,6 +36,8 @@ > #include "internal.h" > #include "sauce.h" > > +static const char *const tty_extensions = "ans,art,asc,diz,ice,nfo,txt,vt"; > + > typedef struct TtyDemuxContext { > AVClass *class; > int chars_per_frame; > @@ -42,6 +46,17 @@ typedef struct TtyDemuxContext { > AVRational framerate; /**< Set by a private option. */ > } TtyDemuxContext; > > +static int read_probe(const AVProbeData *p) > +{ > + int cnt = 0; > + > + for (int i = 0; i < p->buf_size; i++) > + cnt += !!isprint(p->buf[i]); > + > + return (cnt * 100LL / p->buf_size) * (cnt > 400) * > + !!av_match_ext(p->filename, tty_extensions); > +} > + > /** > * Parse EFI header > */ > @@ -153,8 +168,9 @@ AVInputFormat ff_tty_demuxer = { > .name = "tty", > .long_name = NULL_IF_CONFIG_SMALL("Tele-typewriter"), > .priv_data_size = sizeof(TtyDemuxContext), > + .read_probe = read_probe, > .read_header = read_header, > .read_packet = read_packet, > - .extensions = "ans,art,asc,diz,ice,nfo,txt,vt", > + .extensions = tty_extensions, src/libavformat/tty.c:174:23: error: initializer element is not constant .extensions = tty_extensions, ^~~~~~~~~~~~~~ src/libavformat/tty.c:174:23: note: (near initialization for ‘ff_tty_demuxer.extensions’) src/ffbuild/common.mak:59: recipe for target 'libavformat/tty.o' failed make: *** [libavformat/tty.o] Error 1 [...]
diff --git a/libavformat/tty.c b/libavformat/tty.c index 8d48f2c45c..c127500fa1 100644 --- a/libavformat/tty.c +++ b/libavformat/tty.c @@ -24,6 +24,8 @@ * Tele-typewriter demuxer */ +#include <ctype.h> + #include "libavutil/intreadwrite.h" #include "libavutil/avstring.h" #include "libavutil/log.h" @@ -34,6 +36,8 @@ #include "internal.h" #include "sauce.h" +static const char *const tty_extensions = "ans,art,asc,diz,ice,nfo,txt,vt"; + typedef struct TtyDemuxContext { AVClass *class; int chars_per_frame; @@ -42,6 +46,17 @@ typedef struct TtyDemuxContext { AVRational framerate; /**< Set by a private option. */ } TtyDemuxContext; +static int read_probe(const AVProbeData *p) +{ + int cnt = 0; + + for (int i = 0; i < p->buf_size; i++) + cnt += !!isprint(p->buf[i]); + + return (cnt * 100LL / p->buf_size) * (cnt > 400) * + !!av_match_ext(p->filename, tty_extensions); +} + /** * Parse EFI header */ @@ -153,8 +168,9 @@ AVInputFormat ff_tty_demuxer = { .name = "tty", .long_name = NULL_IF_CONFIG_SMALL("Tele-typewriter"), .priv_data_size = sizeof(TtyDemuxContext), + .read_probe = read_probe, .read_header = read_header, .read_packet = read_packet, - .extensions = "ans,art,asc,diz,ice,nfo,txt,vt", + .extensions = tty_extensions, .priv_class = &tty_demuxer_class, };
Signed-off-by: Paul B Mahol <onemda@gmail.com> --- libavformat/tty.c | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-)