- 28 Jul, 2019 1 commit
-
-
Andreas Rheinhardt authored
If the relevant elements (the color description elements for AV1 and the VUI elements in general for H.264 (since 1156b507)) are absent, then their correct values (usually meaning unknown) have already been inferred by the reading process, so that it is unnecessary to initialize them again in the av1/h264_metadata filters even when they were initially absent. Signed-off-by:
Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
- 08 Jul, 2019 1 commit
-
-
Andreas Rheinhardt authored
ff_cbs_delete_unit never fails if the index of the unit to delete is valid, as it is with all current callers of the function. So just assert in ff_cbs_delete_unit that the index is valid and change the return value to void in order to remove the callers' checks for whether ff_cbs_delete_unit failed. Signed-off-by:
Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
- 07 Jul, 2019 3 commits
-
-
Andreas Rheinhardt authored
If the fragment is empty after parsing (i.e. it contains no OBUs), then the check for the type of the fragment's first OBU is nonsensical; so error out in this case just as h264_metadata and hevc_metadata do. Signed-off-by:
Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
Andreas Rheinhardt authored
When testing whether a particular unit should be kept or discarded, it is best to start at the very last unit of a fragment and count down, because that way a unit that will eventually be deleted won't be memmoved during earlier deletions; and frag/au->nb_units need only be evaluated once in this case and the counter is automatically correct when a unit got deleted. It also works for double loops, i.e. when looping over all SEI messages in all SEI units of an access unit. Signed-off-by:
Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
Andreas Rheinhardt authored
This commit changes av1_metadata to (a) use ff_bsf_get_packet_ref instead of ff_bsf_get_packet (thereby avoiding one malloc and free per filtered packet) and (b) to use only one packet structure at all, thereby avoiding a call to av_packet_copy_props. (b) has been made possible by the recent changes to ff_cbs_write_packet. Signed-off-by:
Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
-
- 03 Apr, 2019 1 commit
-
-
James Almer authored
Reviewed-by:
Mark Thompson <sw@jkqxz.net> Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 25 Feb, 2019 1 commit
-
-
Andreas Rheinhardt authored
Currently, a fragment's unit array is constantly reallocated during splitting of a packet. This commit changes this: One can keep the units array by distinguishing between the number of allocated and the number of valid units in the units array. The more units a packet is split into, the bigger the benefit. So MPEG-2 benefits the most; for a video coming from an NTSC-DVD (usually 32 units per frame) the average cost of cbs_insert_unit (for a single unit) went down from 6717 decicycles to 450 decicycles (based upon 10 runs with 4194304 runs each); if each packet consists of only one unit, it went down from 2425 to 448; for a H.264 video where most packets contain nine units, it went from 4431 to 450. Signed-off-by:
Andreas Rheinhardt <andreas.rheinhardt@googlemail.com>
-
- 06 Oct, 2018 1 commit
-
-
Mark Thompson authored
It can't be set for 4:2:2 streams.
-
- 03 Oct, 2018 1 commit
-
-
James Almer authored
Reviewed-by:
Mark Thompson <sw@jkqxz.net> Signed-off-by:
James Almer <jamrial@gmail.com>
-
- 26 Sep, 2018 1 commit
-
-
Mark Thompson authored
Can adjust colour and timing information.
-