mpegtspacketizer: Fix PCR LUT table

We were only resetting the first 512 values of the lookup table instead
of the whole 8192.
This resulted in any PCR PID over 0x0200 ... ending up taking the first PCR
table around :(

dashdemux: properly error out when manifest has no streams

Post an error message when the manifest is parsed and no valid streams
are found be activated and used.

Fixes #704308

mpegts: private/teletext -> application/x-teletext

win32: update

tests: ignore new example binaries

po: update translations

Back to development

Release 1.1.3

Update .po files

midi: Fix handling of the SEGMENT query

gst: Don't swap start/stop for negative rates in the SEGMENT query

mpegdemux: Implement SEGMENT query

tsdemux: Implement SEGMENT query

mxfdemux: Implement SEGMENT query

all: Fix for GST_DISABLE_GST_DEBUG

Where applicable, remove methods that don't do anything different than
the default implementation.

docs/spec: Really remove all mentions of CELT plugin

h264parse: Show NALU string type in debug messages

If you know the NALU type by heart I tip my hat to you, for the rest of
us mere mortals this is a bit more helpful

tsdemux: Remove unused macros

tsdemux: Add more AC3 detection cases

ATSC ac3 streams are always guaranteed to be AC3 if EAC3 descriptor
is not present
If stream registration id is 'AC-3' then it's also guaranteed to be AC3.
Finally if AC3 descriptor is present it's guaranteed to be AC3.

Only silences a warning, but still.

tsdemux: Flush out pending data on EOS

Ensures downstream elements (like decoders) really get all the data

mpegtspacketizer: More AFC flags

I should have put those in a previous commit. sorry :(

mpegdefs: Simplify PCR/PTS/DTS => GST conversion macros

We know we will not overflow 64 bits, therefore just use direct
multiplication/division instead of the scale method (trims usage from
50 instruction calls to 2/3).

dvdspu: artifacts when rendering clipped spu

https://bugzilla.gnome.org/show_bug.cgi?id=705039

mpegvideoparse: do not update caps when bitrate changes

https://bugzilla.gnome.org/show_bug.cgi?id=705035

Fix URIs in REQUIREMENTS

celt: remove, replaced by opus.

uvch264_mjpgdemux: Remove duplicated variables

uvch264_mjpgdemux: Don't unmap/unref buffer if it's pushed out as-is

uvcv4l2_mjpegdemux: Unmap buffer

https://bugzilla.gnome.org/show_bug.cgi?id=699517

Update spec file to be current

tsdemux: HDV only has mpeg 1 audio layer 2

Set that accordingly on the caps

asfmux: Various debug argument fixes

mainly G_GUINT16_FORMAT to ... the right type (guint32 or guint8)

mpg123: Remove dead assignment

harder ? :)

ladspa: initialize debug category earlier

You know, before it's used.

mpegtsdemux: Add more flag debugging for PES and TS packets

Helps with debugging issues. And also remove unused variable (opcr)

This will also allow us in the future to properly detect:
* random-access location (to enable keyframe observation and
  potentially seeking
* discont location (to properly handle resets)
* splice location (to properly handle new stream changes)

codecparser: Make warnings more meaningful

By stating whether we don't have enough data or it's a corrupted
packet

pesparse: Demote a Warning

We actually now know what to do ... have a log message :)

mpegtsmux: avoid using clipped out buffers

If a buffer was entirely clipped out (ie, it's out of the segment
entirely), we'll end up with a NULL buffer, which we don't want
to process/dereference.

tsdemux: Remember requested seek rate and use it

The new seek handling re-creates the segment time information once it
has enough information after a seek.

The problem was that we'd completely ignore the requested rate. So store
that and use it in the newly created segment.

https://bugzilla.gnome.org/show_bug.cgi?id=694369

applemedia: atdec: destroy the queue if set_format fails

applemedia: atdec: remove c++ style comments

applemedia: atdec: 1st round of fixes

mpegvideoparse: Fix codec_data handling

Set to sequence_offset so that _process_config accepts it.

(btw: seq_offset usage throughout this plugin is really dodgy)

https://bugzilla.gnome.org/show_bug.cgi?id=702806

tsdemux: Create and send codec tag

Helps with applications that rely on presence of that tag for user-facing
information.

https://bugzilla.gnome.org/show_bug.cgi?id=702216

hlsdemux: Implement pkcs7 unpadding

Every encrypted fragment will be a multiple of 128 bits, the last byte
contains the number of bytes that were added as padding in the end
and should be removed.

https://bugzilla.gnome.org/show_bug.cgi?id=701673

applemedia: add AudioToolbox based audio decoder

decklink: Add support for group-id in the stream-start event

mxfdemux: Add support for group-id in the stream-start event

tsdemux: Add support for group-id in the stream-start event

tsparse: Add support for group-id in the stream-start event

mpegdemux: Add support for group-id in the stream-start event

midiparse: Add support for group-id in the stream-start event

mssdemux: Add support for group-id in the stream-start event

resindvd: Add support for group-id in the stream-start event

hlsdemux: Add support for group-id in the stream-start event

dash: Add support for group-id in the stream-start event

interlace: fix negotiation if filter caps are passed to query_caps

Make videotestsrc ! interlace ! $anything work again. Problem
was that upstream filter caps were passed which contained
interlace-mode=progressive, which doesn't intersect too well
with interlace's source pad template caps, leading to
not-negotiated errors.

h264parse: Don't abort on invalid streams

Just return an error

hls: fix for assert failure when using encrypted HLS streams

When using an HLS encrypted stream, an assertion failure is thrown:
(gst-launch-1.0:31028): GLib-GObject-WARNING **: cannot register
existing type `GstFragment'

(gst-launch-1.0:31028): GLib-CRITICAL **: g_once_init_leave: assertion
`result != 0' failed

Eventually tracked this down to the call gst_fragment_new()
in function gst_hls_demux_decrypt_fragment.

The GstFragment class is defined in ext/hls/gstfragment.c and in
gst-libs/gst/uridownloader/gstfragment.c. Having two class definitions
with the same name causes the assert failure when trying to allocate
GstFragment. Deleting the version from hls and editing the
Makefile.am solves this assert failure.

https://bugzilla.gnome.org/show_bug.cgi?id=704555

decklink: actually enable decklinksink

https://bugzilla.gnome.org/show_bug.cgi?id=704505

decklink: add "device-number" to select from multiple devices

https://bugzilla.gnome.org/show_bug.cgi?id=704505

mpegvideoparse: Only map input buffer once

Instead of constantly map/unmapping it a bit everywhere, we pass along
to all functions the GstMapInfo.

Makes mpeg video frame parsing 6% faster

https://bugzilla.gnome.org/show_bug.cgi?id=702330

tsdemux: disentangle requested program number from active program number

The program_number attribute was overloaded, trying to indicate both
the currently playing program, and the program requested via the
"program-number" property.  The end result was that setting the
property didn't work (see #690934).

I added a new requested_program_number field rather than reviving the
current_program_number field because it seemed this would result in
fewer changes overall and be less confusing.  It breaks symmetry with
the "program-number" property, but it retains parallels with the likes
of program->program_number.

Because gst_ts_demux_reset is called after the properties have been
parsed, requested_program_number is initialised in gst_ts_demux_init.
Whether this is exactly the right place, I don't know.

tsdemux: fix program-number functionality

Setting the program-number property does not affect which program
is actually being demuxed.

Moving the initialization of the program_number from
gst_ts_demux_reset to gst_ts_demux_init seems to fix this issue.

https://bugzilla.gnome.org/show_bug.cgi?id=690934

resindvd: Send stream-start event before anything else on the mpeg demuxer srcpads

mpegdemux: do not push discont buffers if they aren't discont

Explicitly unset discont flag when it shouldn't be set.

https://bugzilla.gnome.org/show_bug.cgi?id=682110

grabcut: Add GrabCut segmentation element

https://bugzilla.gnome.org/show_bug.cgi?id=702722

h263parse: fix height extraction for H263p

h263parse: add pixel-aspect-ratio if upstream doesn't provide one

dashdemux: fix typo in check_queue_full function

It should return True when the queue IS full

Fixes #704226

mpegtsdemux: Remember seek sequence number

* Avoids handling twice the same seek (can happen with playbin and files
  with subtitles)
* Set the sequence number of the segment event to the sequence number of
  the seek event that generated it (-1 for the initial one).

tsdemux: implement seeking in push mode

tsdemux: recalculate new segment after byte seek

The seeking start time is approximated from the seek offset in bytes
using the accumulated PCR observations, so on a VBR stream there might
be a big difference between the actual PCR and the estimated one after
the seek. This might result in a long wait to skip all out of segments
packets.

Instead we just recalculate the new segment to start at the first PTS
after the seek, so that playback starts immediatly.

Update docs and win32 config.h to development version

docs: add newly-ported elements to docs

mpegvideoparse: Use sequence_display_extension values to update the src caps

The caps should always represent what the user is supposed to see.
So if there is a sequence_display_extension associated with the
stream then use the display_horizontal_size/display_vertical_size
to update the src caps (if they are less than the values provided
by sequence header).

https://bugzilla.gnome.org/show_bug.cgi?id=704009

Release 1.1.2

Oops... should've been part of 1.1.2 tag already :(

Back to development

Update .po files

mpegpsmux: Send stream-start event before any data

mpegtsdemux: Avoid parsing section without enough data

This is actually a workaround (we'll be skipping the upcoming section)

This will only happen for sections where the beginning is located within
the last 8 bytes of a packet (which is the minimum we need to properly
identify any section beginning).

Later we should figure out a way to store those bytes and mark that
some analysis needs to happen. The probability of this happening is
too low for me to care right now and do that fix. There is a good chance
that section will eventually be repeated and won't end up on such border.

examples: Add BAT support to mpegts example

mpegts: Unify section checks and add BAT parsing

* Avoid repeating code everywhere, and instead provide all parsing
  information in one go.
* Add BAT support
* Refine BAT/CAT identification (by adding PID checks)

mpegtsdemux: Avoid vmethod calls

Depending on usage, tsparse and tsdemux might not need to be called
when new section/pes-data is available.

mpegtsdemux: Remove unneeded variables

* packet.origts is no longer used since the PCR refactoring done ages ago
* known_packet_size is a duplicate of packet_size != 0
* caps was never used outside of the packetizer

h264parser: fix size caps to report cropped dimensions.

Restore the original h264parser behaviour to report cropped dimensions
in size caps.

https://bugzilla.gnome.org/show_bug.cgi?id=694068

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

codecparsers: h264: fix calculation of the frame cropping rectangle.

Fix calculation of the frame cropping rectangle, and more precisely
the actual cropped height. The frame_crop_top_offset subtraction
was not scaled up with SubHeightC.

Also clean-up variables to align more with (7-18) to (7-21).

Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne@intel.com>

h264parser: Parse the cropping-rectangle separately.

Assign the un-cropped width/height to sps->width/sps->height
during sps header parsing. Added new fields to SPS header structure
to provide the crop-rectangle dimensions.

https://bugzilla.gnome.org/show_bug.cgi?id=694068

dashdemux: implement queue full check function

Checks if the queue is full according to max buffering time
set by the user

https://bugzilla.gnome.org/show_bug.cgi?id=701404

dashdemux: minor refactor

Split one very large function into 2 smaller but still large functions.

Also change the if conditions to positive checks to improve readability.

dashdemux: handle live playback resync

During a live stream it is possible for dashdemux to lag behind on a
slow connection or to rush ahead of the connection os too fast.

For the first case it is necessary to jump some segments ahead to be able to
continue playback as old segments are usually deleted from the server.

For the later, dashdemux should wait a little before attempting another
download do give time to the server to produce a new segment

dashdemux: fix template based segment generation

When using a template based segment list, do not try to
contruct a finite segment list for the limits of the available periods.

We might not know when the period ends (for live streams) and we can
always create the segment on demand when requested by dashdemux,
avoiding use of some memory and cpu when re-creating this list.

dashdemux: use more appropriate data structures for parsing

Replaces the 2 likely larger lists with more appropriate structures
to improve performance.

Replaces S nodes GList for a GQueue, this reduces latency to startup
because of traversing the list just append an element.

Replaces the processed media segments GList for a GPtrArray as it is
constantly acessed by index during playback.

dashdemux: answer to latency queries

Set live if stream is live and also add to the max latency the
max internal buffering

uridownloader: fix crash when download is null

Do no try to unref a NULL download attribute

uridownloader: do not set cancelled unless explicitly called by user

Cancelled is a 'permanent' state of the uridownloader and is only
removed by a call to _reset. When a download fails we just want to
return NULL on the fetch function and leave the downloader ready
for another fetch, otherwise the user has to call _reset after
failed downloader, even when it didn't call _cancel.

mpegts: Avoid false-positives in section type detection

Due to the variety of section types out there, we need to add
some checks when identifying section types.

We check here that the PID is also consistent with the table_id.

mpegtsdemux: Fix multiple-SI detection/parsing

We had two issues with the previous code:
1) We were badly handling PUSI-flagged packets. We were discarding the
   initial data (if pointer != 0) whereas we should have been accumulating
   it with the previous data (if there was a continuity of course).
   => First series of information loss
2) We were not checking whether there were more sections after the end
   of one (i.e. when the following byte was not a stuff byte).

This fixes those two issues.

Fixes #677443

https://bugzilla.gnome.org/show_bug.cgi?id=677443

mpegtbase: Fix switch/case typo

Ended up trying to analyze PMT ... with a PAT :(

mpegtsdemux: Handle private section streams

Until now we simply ignored those streams (since we couldn't do anything
with it anyway). Now that we have the mpegts library and we offload the
section handling to the application side we can properly identify and
extract them.

By default it is disabled for tsparse and enabled for tsdemux, but there is
a property to change that.

This should open the way to properly handle all private section streams,
including:
* DSM-CC
* MHEG
* Carousel data
* Metadata streams (though I haven't seen any of those in the wild)
* ... And all other specs/protocols making use of those

Partially fixes #560631