This should avoid some tsan warnings
Found-by: NChris Cunningham <chcunningham@google.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Previously only the edit_rate of material_track was checked.
If it's negative, it causes assertion failures in av_rescale_rnd.
Reviewed-by: NTim Nicholson <nichot20-at-yahoo.com@ffmpeg.org>
Signed-off-by: NAndreas Cadhalpun <Andreas.Cadhalpun@googlemail.com>

In case of bitstream errors the deblock filter and slices can access uninitialized
top_borders from previous slices which did not fill them as they stoped halfway due
to error or where entirely missing.

This also makes code using these tables deterministic in case of missing or damaged
slices

Found-by: Tyson Smith
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

This is a no-op.

* commit '5f2c8315':
  thp: set duration for audio stream too
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit '1d62ee38':
  movenc: Add a unit test for signaling of the track start times
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit 'bef3b1f5':
  movenc: Allow setting start_dts/start_cts before writing actual packets
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This way, it never starts with 0xFFF0, and never trips the
ADTS "Detection" code in movenc.c.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

DASH manifest should have framerate specified as an attribute in the
AdaptationSet element and Representation elements. Though ISO/IEC
23009-1:2014 doesn't seem to define frameRate as a required attribute,
it is at least optional, and DASH-IF IOP 3.0 seems to require it. See
section 3.2.4 of http://dashif.org/w/2015/04/DASH-IF-IOP-v3.0.pdf

In the event that avg_frame_rate is not set in the muxer, we ignore the
frameRate tag altogther.
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

See e.g https://stackoverflow.com/questions/1666353/are-typedef-and-define-the-same-in-c
for rationale.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>

In the spirit of commit 8199908f, likely
typo originally.

Found by enabling -Wparentheses on clang 3.7 and running a manual audit.
To the best of my knowledge, no such instances remain.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>

Unrecognized streams are not allocated
Fixes: flicker-1.color1.vp91447030769.08.webm
Found-by: NChris Cunningham <chcunningham@google.com>
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

* commit '3eeb7edf':
  movenc: Add a unit test for frag_discont with edit lists
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit '09e431b9':
  movenc: Assume streams starting at pts=0 for discontinuous fragments with editlists
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NJames Almer <jamrial@gmail.com>

Signed-off-by: NJames Almer <jamrial@gmail.com>

Fixes assertion failure
Fixes: 7f646252a30ee28b583aac1f82e7985e/signal_sigabrt_7ffff6ae7cc9_7353_62fc077bf2f454d39e188c69807193a6.png

Found-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Sed \r is not portable, it does not work on freebsd, hopefully tr -d will.
Signed-off-by: NMarton Balint <cus@passwd.hu>

All diferences in unit tests have been acounted for.

* commit '59e8ec0a':
  movenc: Add an API unit test for fragmenting options/calls
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This is possibly undefined behavior based on sequence point rules, but I
have not studied the spec at that level of detail.

Fixes: CID 1338321.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>

Fixes: CID 1338328.
Reviewed-by: NPaul B Mahol <onemda@gmail.com>
Signed-off-by: NGanesh Ajjanagadde <gajjanagadde@gmail.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Line endings do matter to md5sum...
Signed-off-by: NMarton Balint <cus@passwd.hu>

Signed-off-by: NMarton Balint <cus@passwd.hu>

This uses the new output lib naming scheme. In order to work all
dependency projects must also be updated to the latest version (the
supplied bat file can be used for this)

This now places all .libs in the lib directory but differentiates
between static and shared libs by only using a 'lib' prefix for static
libraries.

Small refactor of fps code for improved readability. In particular
the "cor" variable was unnecessary and misleading because it would
always be set to -delta0.
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>