[FFmpeg-devel] [PATCHv2] add signature filter for MPEG7 video signature

[Gerion Entrup]

Attached improved version of patch.

Differences to last time:
- reduce amount of errors in the signature (the last patch included some int
foo = a/b). This version replaces this with a rational.
- implement binary output.
- fixes in configure, some typos

I have found the conformance testfiles [1]. Both the binary and the xml output
passes the conformance test but are not bitexact. I wrote some python script
to prove this (see attachment). I don't see why this happens. If someone want
to help, the correspondent reference code is in the file
"ExtractionUtilities/VideoSignatureExtraction.cpp" beginning with line 1615,
that could be found here [2].

Then a few questions:
- The timebase of the testfiles is 90000. In the binary output unfortunately there
is only place for a 16 bit number, so this don't fit. Currently the code simply crop
remaining bits. Is there a better solution (devide with some number etc)?

- I try to use put_bits32 where it is possible, because I thought is is faster. Then
I saw it internally uses put_bits as well. Does it have a performance impact to
replace it with put_bits(..., 8, ...) (would simplify the code a lot)?

Gerion

[1] http://standards.iso.org/ittf/PubliclyAvailableStandards/c057047_ISO_IEC_15938-7_2003_Amd_6_2011_Conformance_Testing.zip
[2] http://standards.iso.org/ittf/PubliclyAvailableStandards/c056735_ISO_IEC_15938-6_2003_Amd_4_2011_Electronic_inserts.zip
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-add-signature-filter-for-MPEG7-video-signature.patch
Type: text/x-patch
Size: 78507 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160330/527ad235/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: check.py
Type: text/x-python
Size: 10554 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160330/527ad235/attachment.py>