On Thu, Jun 09, 2016 at 04:35:02PM +0200, Clément Bœsch wrote: > On Thu, Jun 09, 2016 at 03:59:30PM +0200, Clément Bœsch wrote: > > On Thu, Jun 09, 2016 at 01:35:19PM +0000, Carl Eugen Hoyos wrote: > > > Clément Bœsch <u <at> pkh.me> writes: > > > > > > > + if (b[3] == '#') > > > > + return AVPROBE_SCORE_EXTENSION + 1; > > > > + if (b[3] >= '0' && b[3] <= '9') > > > > + return AVPROBE_SCORE_MAX - 1; > > > > > > Imo, this should be: > > > if (b[3] == '#' || (b[3] >= '0' && b[3] <= '9')) > > > return AVPROBE_SCORE_EXTENSION + 2; > > > or similar > > > > > > I count 37 and 34 bits which is only a little more than > > > the usual 32 bit for EXTENSION + 1. > > > > > > > Sure. Changed locally, will push soon, thanks. > > > > For some reasons it makes seeking with pgm somehow working. The reference > test doesn't look that great IIUC, but seeking in these file with ffplay > is fine AFAICT.
it seems it messes with pgmyuv detection (which i think was filename extension
based)
example:
./ffmpeg -i matrixbench_mpeg2.mpg -pix_fmt yuv420p16be -vframes 1 test.pgmyuv
./ffplay-ref test.pgmyuv
Input #0, image2, from 'test.pgmyuv':B vq= 0KB sq= 0B f=0/0
Duration: 00:00:00.04, start: 0.000000, bitrate: 248835 kb/s
Stream #0:0: Video: pgmyuv, yuv420p16le, 720x576, 25 tbr, 25 tbn
./ffplay test.pgmyuv
Input #0, ppm_pipe, from 'test.pgmyuv':vq= 0KB sq= 0B f=0/0
Duration: N/A, bitrate: N/A
Stream #0:0: Video: ppm, gray16le, 720x864, 25 tbr, 25 tbn, 25 tbc
>
> Is this change acceptable?
>
> --
> Clément B.
> _______________________________________________
> ffmpeg-devel mailing list
> [email protected]
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
