summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* lavf/srtdec: rewrite parsing logicClément Bœsch2016-01-014-0/+2182
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes Ticket #5032 The samples in Ticket #5032 is using \r\r\n as line breaks. Since we already are handling \r, or \n, or \r\n as line breaks, \r\n\n will be considered as a double line breaks. This is an issue because ff_subtitles_read_text_chunk() will as a result stop extracting a chunk after just one line. So instead of parsing the SRT by "chunks" (which means splitting every double LB), this new parser is detecting timing lines, and split the events on this basis. While this sounds safe and simple, it needs to take into account the event number preceding the timing line while handling situations such as: - event number starting at 0 or actually any number instead of 1 - event numbers not being ordered at all - event number being followed by text garbage (this really happened, see Ticket #4898) - event payload containing one or multiple number (a protagonist saying a count-down, a date or whatever) which could be confused with a chapter number - event number being empty (see Ticket #2167) - all kind of weird line breaks can appear randomly like wild pokémons - untrustable line breaks (Ticket #5032) The sample madness.srt tries to sum up most of this into one sample, ticket5032-rrn.srt is the file containing \r\r\n line breaks. and empty-events-2167.srt contains empty events.
* lavc: add text encoderClément Bœsch2015-12-212-0/+216
|
* fate: increase FUZZ by 1 for aac-tns-encodeAndreas Cadhalpun2015-12-131-1/+1
| | | | | | | | This should fix this test failing on kfreebsd, a regression since 6e5dbe7, which decreased the CMP_TARGET by 1. Reviewed-by: Rostislav Pehlivanov <atomnuker@gmail.com> Signed-off-by: Andreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>
* avfilter/vf_delogo: round to the closest valueJean Delvare2015-12-091-109/+109
| | | | | | | | | | | | | When the interpolated value is divided by the sum of weights, no rounding is done, which means the value is truncated. This results in a slight bias towards dark green in the interpolated area. Rounding properly removes the bias. I measured this change to reduce the interpolation error by 1 to 2 % on average on a number of sample input and logo area combinations. Signed-off-by: Jean Delvare <jdelvare@suse.de> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* fate/api-jpeg-codec-param: rename to api-mjpeg-codec-paramMatthieu Bouron2015-12-082-3/+3
|
* aacenc_ltp: disable LTP with high lambda valuesRostislav Pehlivanov2015-12-081-1/+1
| | | | | | Makes no sense to enable for high bitrates, the coder does well enough. Signed-off-by: Rostislav Pehlivanov <atomnuker@gmail.com>
* aacenc_tns: use 4 bits for short windowsRostislav Pehlivanov2015-12-081-1/+1
| | | | | | | | With only 7 coefficients per short window at most the extra precision makes a difference and seems to reduce crackling and stddev even further. Signed-off-by: Rostislav Pehlivanov <atomnuker@gmail.com>
* fate/api-{jpeg,png}-codec-param: add missing codec dependenciesMatthieu Bouron2015-12-081-2/+2
|
* fate/api-codec-param: fix codec context leakMatthieu Bouron2015-12-081-0/+6
|
* fate/api: add w32+os2 support for fate-api-threadmessageClément Bœsch2015-12-073-4/+3
|
* fate/api: fix fate-api-threadmessage dependencyClément Bœsch2015-12-071-1/+1
|
* fate/api: test threadmessageClément Bœsch2015-12-073-0/+268
|
* fate: add limited_input_seek testsSimon Thelen2015-12-073-0/+8
| | | | | Signed-off-by: Simon Thelen <ffmpeg-dev@c-14.de> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* fate: change the CMP_TARGETs for the recent AAC encoder changesRostislav Pehlivanov2015-12-061-7/+7
| | | | | | | The case of PNS was outdated and resulted in failures on some kdfreebds systems. Signed-off-by: Rostislav Pehlivanov <atomnuker@gmail.com>
* fate/concatdec: Use -bitexactTimothy Gu2015-12-066-10/+10
| | | | | | Fixes FATE failures on --enable-small builds. Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* aacenc_ltp: fix out of bounds memory accessRostislav Pehlivanov2015-12-031-1/+1
| | | | Discovered by Coverity.
* tests/api: Fix API test build on windows with --enable-sharedHendrik Leppkes2015-12-031-1/+1
|
* AAC encoder: improve SF range utilizationClaudio Freire2015-12-021-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch does 4 things, all of which interact and thus it woudln't be possible to commit them separately without causing either quality regressions or assertion failures. Fate comparison targets don't all reflect improvements in quality, yet listening tests show substantially improved quality and stability. 1. Increase SF range utilization. The spec requires SF delta values to be constrained within the range -60..60. The previous code was applying that range to the whole SF array and not only the deltas of consecutive values, because doing so requires smarter code: zeroing or otherwise skipping a band may invalidate lots of SF choices. This patch implements that logic to allow the coders to utilize the full dynamic range of scalefactors, increasing quality quite considerably, and fixing delta-SF-related assertion failures, since now the limitation is enforced rather than asserted. 2. PNS tweaks The previous modification makes big improvements in twoloop's efficiency, and every time that happens PNS logic needs to be tweaked accordingly to avoid it from stepping all over twoloop's decisions. This patch includes modifications of the sort. 3. Account for lowpass cutoff during PSY analysis The closer PSY's allocation is to final allocation the better the quality is, and given these modifications, twoloop is now very efficient at avoiding holes. Thus, to compute accurate thresholds, PSY needs to account for the lowpass applied implicitly during twoloop (by zeroing high bands). This patch makes twoloop set the cutoff in psymodel's context the first time it runs, and makes PSY account for it during threshold computation, making PE and threshold computations closer to the final allocation and thus achieving better subjective quality. 4. Tweaks to RC lambda tracking loop in relation to PNS Without this tweak some corner cases cause quality regressions. Basically, lambda needs to react faster to overall bitrate efficiency changes since now PNS can be quite successful in enforcing maximum bitrates, when PSY allocates too many bits to the lower bands, suppressing the signals RC logic uses to lower lambda in those cases and causing aggressive PNS. This tweak makes PNS much less aggressive, though it can still use some further tweaks. Also update MIPS specializations and adjust fuzz Also in lavc/mips/aacpsy_mips.h: remove trailing whitespace
* avformat/mp3dec: prefer "fast_seek" to TOC seek for CBR files.Chris Cunningham2015-12-021-1/+1
| | | | | | | | | | | | | | | "Fast seek" uses linear interpolation to find the position of the requested seek time. For CBR this is more direct than using the mp3 TOC and bypassing the TOC avoids problems with TOC precision. (see https://crbug.com/545914#c13) For VBR, fast seek is not precise, so continue to prefer the TOC when available (the lesser of two evils). Also, some re-ordering of the logic in mp3_seek to simplify and give usetoc=1 precedence over fastseek flag. Signed-off-by: wm4 <nfxjfg@googlemail.com>
* AAC encoder: make pe.min a local minimumClaudio Freire2015-11-291-2/+2
| | | | | | | | | | | | | | As noted in a comment, pe.min in the reference encoder is centered around current pe. The bit reservoir algo needs pe.min to be a local minimum, because it can only account for local PE variations. If it's set to a global minimum as was being done, bit reservoir logic doesn't work as efficiently. This patch tries to forget old minimums and converge to a local minimum without losing the stability of the previous solution. Listening tests until now suggest this solves numerous RC issues.
* Merge commit '823fa7004571cb8404ca5785f9fa6e85f0f9f3d3'Hendrik Leppkes2015-11-2912-34/+21
|\ | | | | | | | | | | | | * commit '823fa7004571cb8404ca5785f9fa6e85f0f9f3d3': fate: Rework sgi tests into a suite and add the missing ones Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * fate: Rework sgi tests into a suite and add the missing onesVittorio Giovara2015-11-247-28/+15
| | | | | | | | Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
* | Merge commit '99f40fd02f1870c2dee70977b672da9cb198f0be'Hendrik Leppkes2015-11-292-6/+6
|\ \ | |/ | | | | | | | | | | * commit '99f40fd02f1870c2dee70977b672da9cb198f0be': sgienc: Do not end RLE lines with 0s Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
| * sgienc: Do not end RLE lines with 0sVittorio Giovara2015-11-242-3/+3
| | | | | | | | | | | | | | | | This is never mentioned in the specifications, and decoders work just as fine without it. Update the fate references since the compressed file is smaller. Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
* | aac_ltp: split, reorder and improve prediction algorithmRostislav Pehlivanov2015-11-261-1/+1
| | | | | | | | | | This commit attempts to mirror what the decoder does more closely in addition to fixing some shortcomings.
* | fate: add FF_CODEC_CAP_SKIP_FRAME_FILL_PARAM testsMatthieu Bouron2015-11-265-0/+881
| |
* | fate-run: Fix indentationTimothy Gu2015-11-251-3/+3
| |
* | avformat/movenc-test: Fix integer overflowsMichael Niedermayer2015-11-231-1/+1
| | | | | | | | Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | avutil/motion_vector: export subpel motion informationClément Bœsch2015-11-231-18/+18
| | | | | | | | FATE test changes because of the switch from shift to division.
* | avcodec/atrac1: fix decoder: QMF delay compensation should be 39 samplesDaniil Cherednik2015-11-211-3/+10
| | | | | | | | | | | | This also adds a new fate test Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | tests/tiny_ssim: replace #define by typedefGanesh Ajjanagadde2015-11-201-3/+2
| | | | | | | | | | | | | | | | | | See e.g https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c for rationale. Reviewed-by: Ronald S. Bultje <rsbultje@gmail.com> Reviewed-by: Hendrik Leppkes <h.leppkes@gmail.com> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
* | Merge commit '1d62ee38894afb696674db78cee8f8d89204a8fe'Derek Buitenhuis2015-11-181-0/+4
|\ \ | |/ | | | | | | | | | | * commit '1d62ee38894afb696674db78cee8f8d89204a8fe': movenc: Add a unit test for signaling of the track start times Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * movenc: Add a unit test for signaling of the track start timesMartin Storsjö2015-11-111-0/+4
| | | | | | | | Signed-off-by: Martin Storsjö <martin@martin.st>
* | movenc-test: Pad the packet data start with 0sDerek Buitenhuis2015-11-181-20/+20
| | | | | | | | | | | | | | This way, it never starts with 0xFFF0, and never trips the ADTS "Detection" code in movenc.c. Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
* | Merge commit '3eeb7edfc2a1157b7b0e0ce21ac2cd44d55d405b'Derek Buitenhuis2015-11-171-0/+4
|\ \ | |/ | | | | | | | | | | * commit '3eeb7edfc2a1157b7b0e0ce21ac2cd44d55d405b': movenc: Add a unit test for frag_discont with edit lists Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * movenc: Add a unit test for frag_discont with edit listsMartin Storsjö2015-11-111-0/+4
| | | | | | | | | | | | | | This tests the case where the muxer correctly guesses that a stream starts at pts=0. Signed-off-by: Martin Storsjö <martin@martin.st>
* | fate: update fate-source reference fileJames Almer2015-11-161-1/+0
| | | | | | | | Signed-off-by: James Almer <jamrial@gmail.com>
* | fate: fix concat demuxer extended test portabilityMarton Balint2015-11-161-1/+1
| | | | | | | | | | | | Sed \r is not portable, it does not work on freebsd, hopefully tr -d will. Signed-off-by: Marton Balint <cus@passwd.hu>
* | Merge commit '59e8ec0aa8ab174701d01a3bfe96fedd0b7fcead'Derek Buitenhuis2015-11-163-0/+25
|\ \ | |/ | | | | | | | | | | | | | | All diferences in unit tests have been acounted for. * commit '59e8ec0aa8ab174701d01a3bfe96fedd0b7fcead': movenc: Add an API unit test for fragmenting options/calls Merged-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
| * movenc: Add an API unit test for fragmenting options/callsMartin Storsjö2015-11-102-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Contrary to the normal fate tests that run via avconv, this tests nontrivial call sequences that are only doable via the API (mainly for different corner cases when using the muxer for segmenting). The test muxes fake packet data (with extradata that looks enough like proper data to make the file be viewable with e.g. boxdumper) and checks the hash of the produced files. The test also verifies that fragments produced via different call sequences remain identical (to avoid e.g. updating the output hashes and suddenly having fragments that used to be identical suddenly diverging), for fragments written with frag_discont and/or delay_moov. Signed-off-by: Martin Storsjö <martin@martin.st>
* | fate: fix concat demuxer extended tests on windowsMarton Balint2015-11-161-1/+1
| | | | | | | | | | | | Line endings do matter to md5sum... Signed-off-by: Marton Balint <cus@passwd.hu>
* | fate: fix concat demuxer tests on msys/cygwin by using relative pathsMarton Balint2015-11-162-4/+4
| | | | | | | | Signed-off-by: Marton Balint <cus@passwd.hu>
* | fate: add concat demuxer testsMarton Balint2015-11-1511-0/+546
| | | | | | | | | | Reviewed-by: Nicolas George <george@nsup.org> Signed-off-by: Marton Balint <cus@passwd.hu>
* | swresample/resample: increase precision for compensationMichael Niedermayer2015-11-111-4/+4
| | | | | | | | Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | ffmpeg: set muxer packet duration based on framerate only for CFRMichael Niedermayer2015-11-119-248/+248
| | | | | | | | | | | | | | | | | | | | | | a set ost->frame_rate does not imply CFR in ffmpeg The changed fate tests had all wrong packet durations (like 1/1000 or 1/90000) There might be more cases in which is_cfr could be set Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | avformat/mxfenc: Only store user comment related tags when neededMichael Niedermayer2015-11-106-59/+59
| | | | | | | | | | | | | | | | | | | | Also support disabling them as they seem to cause problems to some Users. They are also not allowed in IRT D-10 thus the default for mxf_d10 is not to write them This also decreases the filesize when no user comment are stored Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | Merge commit '3efd71b4d0b4a73ccbbbdc092e6bbd54d92633f4'Hendrik Leppkes2015-11-109-248/+248
|\ \ | |/ | | | | | | | | | | * commit '3efd71b4d0b4a73ccbbbdc092e6bbd54d92633f4': avconv: set packet duration for CFR video streams Merged-by: Hendrik Leppkes <h.leppkes@gmail.com>
* | tests/fate/avformat: Fix fate-lavfMichael Niedermayer2015-11-103-2/+3
| | | | | | | | | | | | | | | | The CMP variable seems to have been inherited from fate-api-seek which set it to null the mxf reference needed a change due to c7e14a279fa7348db10ec824bb2d67858cb1c1ca Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
* | Add pixblockdsp checkasm testsTimothy Gu2015-11-074-0/+112
| |
* | lavu/opt: enhance printing durations.Nicolas George2015-11-071-4/+4
| | | | | | | | | | | | | | Trim unneeded leading components and trailing zeros. Move the formating code in a separate function. Use the function also to format the default value, it was currently printed as plain integer, inconsistent to the way it is parsed.
OpenPOWER on IntegriCloud