2
86%
master: 86%

Ran 03 Sep 2019 11:43PM UTC

Files 51

Run time 6s

Badge

Embed ▾

Committed 03 Sep 2019 10:55PM UTC coverage: 70.583% (+1.0%) from 69.579%

Job # CACHE_NAME=COVERALLS

Build Type

push

travis-ci

Committed by Ivan Molodetskikh

Commit Message

Only cap overflow if it's actually a danger.

In the normal case (when we're not in danger of wasting bits
 because we have not used enough), soft_limit is negative.
As a result, log_soft_limit was being set to -1.
Normally this is fine, since we then compare it against
 (log_scale_pixels - log_q_exp), the log of the estimated number of
 bits that will be used, which is usually positive.
However, when the number of pixels is very small (e.g., for a 1x1
 video), it is possible for (log_scale_pixels - log_q_exp) to be
 negative, and we will try to enforce the limit anyway.
This results in very bad things (integer overflows) when we try.

This commit also corrects a parentheses error that was introduced
 when porting the original implementation to rav1e (the division by
 margin should happen before the multiply, which can also cause
 overflows).

Run Details

13007 of 18428 relevant lines covered (70.58%)

0.71 hits per line

xiph / rav1e / 4421 / 2
86%
master: 86%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4421.2 (CACHE_NAME=COVERALLS)

xiph / rav1e / 4421 / 2 86% master: 86%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4421.2 (CACHE_NAME=COVERALLS)

xiph / rav1e / 4421 / 2
86%
master: 86%

README BADGES
x