matroskademux: Unlock stream lock after use

Stream lock of sink pad was not unlocked after non-updating seek.

multipartmux: Re-set need_segment flag after FLUSH_STOP

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

v4l2: bufferpool: don't forget to release buffer on error

If the pool is stopped while gst_v4l2_buffer_pool_dqbuf() waits for a
buffer then the return value is GST_FLOW_FLUSHING. In this case the buffer
to queue must also be released. Otherwise is will never be deleted or
returned to its pool.

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

rtp: Fail payloading unit test if an error message is received

rtph263ppay: Don't pass upstream filter caps to downstream

Downstream usually can't accept video/x-h263 but only application/x-rtp,
so we would always get an empty intersection here.

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

po: update POTFILES.in

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

rtspsrc: avoid some strdup

rtspsrc: add select-stream signal

Add a signal to let the app select what streams will be selected.

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

rtspsrc: avoid strdup

rtp: Fix documentation and comments to use rtpbin instead of old gstrtpbin

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

v4l2: don't extract data from caps twice

gst_video_info_from_caps() always extract width, height, interlace mode and
framerate now. It is no longer necessary to do it again for encoded
formats.

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

souphttpsrc: also consider stop positions in seeks

Use seek stop position as range end for requests

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

souphttpsrc: allow seeks in ready

On is_seekable, check if the server's headers have already been
received. If not, do a HEAD request to get them before responding
to basesrc.

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

rtspsrc: add signal to notify of the SDP

This way, the app can look and modify the SDP.

gdkpixbufoverlay: Allow negative offsets to specify offset from bottom/right

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

qtdemux: compute framerate from average sample duration

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

flvdemux: Add flvversion 1 to the flash-video caps

This allows using avdec_flv which requires this field to be
present in the caps. FLV only supports flash-video version 1
right now.

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

deinterleave: Don't hold object lock while sending events downstream

Based on a patch by Kishore Arepalli <kishore.arepalli@gmail.com>

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

matroskademux: Add MPEG4 video profile/level to the caps

matroskademux: Add AAC profile/level to the caps

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

vorbispay: add support for config-interval

Align code with the theora payloader and add support for the config-interval to
periodically send out the config headers.

theorapay: small cleanups

theorapay: handle streamheaders as well

vorbispay: always collect headers on data

When we see a data packet, always check if we need to collect any previous
headers.

vorbispay: handle streamheader as well

Take config strings from the streamheader when we can

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

rtph264pay: avoid double buffer unmap on error

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

rtspsrc: reset-sync before play

Call reset-sync on the rtpbin before we go to playing. This makes us require SR
packets for all streams again before we attempt to sync them. If we don't reset,
it might be that we combine SR packets from before and after the PAUSE/PLAYING
state change and end up with huge bogus offsets.

jitterbuffer: improve sync on first packets

Don't throw away the first RTCP packet if it arrives before the first
RTP packet but remember and use it to signal sync once we get the
RTP packet.

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

jitterbuffer: only signal loop when active

Only signal the loop function when it is active.

jitterbuffer: signal timestamp discont

We can now use the RESYNC buffer flag to mark a timestamp discont when we update
the ts-offset property.

jpegpay: turn some errors into warnings

Turn some errors into warnings, we can continue processing so this should
not be fatal.

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

rtspsrc: avoid some flushes

rtspsrc: handle data message when waiting for reply

When we are waiting for a server reply, handle data messages instead of
ignoring them.

rtspsrc: handle data messages in separate method

Refactor and make a method to handle a data message.

rtspsrc: add some more docs to handle-request signal

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

Send a clock_provide message on the bus when we get a netclock

rtspsrc: Expose use-pipeline-clock property

udpsink: bind to the given interface

Actually call BINDTODEVICE to bind to the interface as given by the
property.

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

vp8dec: Error out gracefully if we get an unsupported color format

In theory we can only get I420 though, just to be on the safe side.

vp9: Add support for YV12, Y42B and Y444 color formats

The encoder does not work with Y42B and Y444 yet it seems.

vp9dec: Update default postproc settings from vp9_dx_iface.c

matroska: Add initial VP9 support

vpx: Add initial, experimental VP9 support

rtsp: go back into the loop after doing pause

After we do a pause request, go back to loop mode so that we can listen
for server messages again.

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

rtpptdemux: Wait after the caps to forward the other events

First forward the stream-start, then the caps, then the rest

ximagesrc: clear dts on buffer acquired from pool

When setting timestamps on outgoing buffers, clear the
dts explicitly, otherwise it may end up being set to a
bogus value from last time it was used. Avoids every
second or so buffer's dts being set to 0. Not that it
should matter for raw video.

v4l2: don't redefine the PERFORMANCE debug variable

It is already defined in core.

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

rtspsrc: fix race in state change to paused

When we go to paused, we first flush the connection and then send the pause
command. As a result of the flushing, the scheduled paused command can get
lost. Wait until the connection is completely flushed and the rtsp task is
waiting before issuing the paused or playing request.

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

qtdemux: handle SEGMENT query

v4l2: Optimize negotiation by removing the query filter

As cameras tend to have a quite specific set of capabilities (specific
framerates for each resolution), getting the peer caps filtered by our
probed caps can cause a big increase in the caps size which slows down
things quire a bit.

As for negotiation v4l2 iterates through the caps of the peer to find the
first intersection with the probed caps, getting the fully expanded
intersection of capabilities is not useful.

Using the same testcase as for bug #702632, adding this patch on top of
the patches suggested there speeds up getting the inital frame from
around ~14-15 seconds to around ~3-4 seconds.

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

avidemux: duration query returns zero for DV video in avi

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

qtdemux: Disable usage of allocation queries

This can only reliably work if demuxers have a
separate streaming thread per srcpad. This should be
done in a demuxer base class, which integrates parts
of multiqueue

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

Avoid skipping moov atoms for fragmented MP4 files.

bug #700505

Following a representation change that causes a resolution change,
the video decoder fails to decode correctly. Dashdemux detects the
representation change and pushes a new caps event and an
initialization segment (a new moov atom) to the downstream qtdemux,
but it doesn't handle this new moov yet, it will only parse the
first one it receives.

This commit changes qtdemux to accept a new moov in a dash bitstream
switching scenario.

qtdemux: send stream-start only once for each stream

Do not send stream start again when reconfiguring a pad for new caps.
That is common for adaptive streams

osxvideosink: fix support in VM's without hardware acceleration

rtpmp2tdepay: accept mislabelled streams from GStreamer 0.10 as well

The mp2t payloader in 0.10 mislabelled the streams as MP2T-ES
instead of MP2T, so accept that as well for compatibility reasons.

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

rtspsrc: manage element state ourselves

Lock the state of the all our elements and manage their states
outselves. Because we are working async, we can't rely on the state
change function to set the state at the right time or to return the
right return value from the state change function.

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

matroskademux: Don't unlock stream lock without locking it first

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

rtpsession: Use the right hashtable to calculate bandwidth

Don't use an unused hashtable to iterate source to calculate bandwidth.
Remove unused code.

pulsesink: Require PulseAudio >= 2.0

This is needed for pa_format_info_get_prop_* functions.

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

Revert "pulsesink: Make 2.0 dependency optional"

This reverts commit 01457027e0d384aca3e551ae684e0aa074ee5498.

We'll just depend on PulseAudio 2.0 or above instead of having the bug
partially fixed based on the installed libpulse version.

pulsesink: Make 2.0 dependency optional

The getcaps function we added uses some pa_format_info_get_prop...
accessor functions that were only added in 2.0, so we only have our
getcaps implementation exist if we're compiling against libpulse 2.0 or
above.

Eventually, we could bump the minimum requirement to 2.0 or above.

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

Revert "videomixer: When all sinkpads are eos, update output segment stop and forward it"

This reverts commit 2d3910fc7901b5f29e16c0fdd4e9067a6d7f66fe.

It's not solving any problem and instead causes code to fall apart.

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

matroskademux: mark subtitle streams as sparse in stream-start event

And also mark the streams that should be selected by default if
marked so in the headers.

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

audiopanorama: add prebuilt files

audiopanorama: cleanup and expand the tests

Split out two more tests. Extract more common code into helpers. Add coverage for float.

audiopanorama: cleanup of transform()

Only map input if we are reading it. Cleanup the logging and the comments a bit.

audiopanorama: use orc to speedup processing

Use special variants for the case when we don't change the panorama (pan=0.0).
Simplify the processing functions by passing the panorama value directy instead
of the instance. Use orc for clearing buffers too.

videomixer: check last end_time after conversion to running segment

The last end_time was saved after conversion, so the comparison
had to be made after conversion for it to make sense.

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

videomixer: add mix->segment.start to output_end_time

When the segment start is not 0, this created a situation where
the output_end_time is inferior to output_start_time, and the duration
of the next buffer ended up underflowing.

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

matroskademux: Send stream headers after the segment event

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

qtdemux: Do allocation query after exposing all pads and no-more-pads

Also configure video streams as early as possible.

Related https://bugzilla.gnome.org/show_bug.cgi?id=701856
but not fixing that.

flvdemux: Don't forward CAPS events from upstream

Just use the default pad event handler.

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

pulsesink: Cache the getcaps/acceptcaps probe stream

getcaps is called frequently during stream setup, and creating a new
stream each time is very inefficient. There's some more room for
optimisation by caching the queried sink formats as well, but this needs
some more changes to listen for format changes on the sink (for when
supported formats change between probe stream creation and sink
querying).

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

pulsesink: Add a getcaps function

This allows us to have more fine-tuned caps in READY or above. However,
this is _really_ inefficient since we create a new stream and query sink
for every getcaps in READY, which on a simple gst-launch line happens
about 35 times. The next step is to cache getcaps results.

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

pulsesink: Take a lock on the ringbuffer in acceptcaps

This is needed as a concurrent state change could pull the context or
stream out from under our feet.

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

audiopanorama: move the enum to the header and use instead of gint

Move the enum for the processing method to the header so that we can use the
type for the instance struct.

level: rework the tests to cover other formats too

v4l2: make sure the element is not deleted before the pool

The pool accesses data from the v4l2object so it must exist at least
as long as the pool. Refcount the element which controls the object
live-time.

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

png: Link with libgstbase for GstByteReader and GstAdapter

wavenc: Link with libgstbase for GstByteWriter

wavparse: Push stream-start event in pull mode before anything else

pulsesink: Get rid of acceptcaps side-effects

The sink info callback should not have side-effects on the GstPulseSink
object since we are sometimes using with a dummy stream in acceptcaps.

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

Back to development

Release 1.1.1

Update .po files

wavenc: Fix taglist ref handling that made the unit test fail

Automatic update of common submodule

From 098c0d7 to 01a7a46

v4l2: iterate controls with V4L2_CTRL_FLAG_NEXT_CTRL if possible

In v2.6.18 control classes where added to the v4l2 API.
Iterating over CIDs starting with V4L2_CID_BASE will only find controls for
the first control class.
By iterating with V4L2_CTRL_FLAG_NEXT_CTRL all controls are found.

This is necessary to make controls from other control classes available in
the extra-controls property.

If V4L2_CTRL_FLAG_NEXT_CTRL is not defined at compile time or not supported
at runtime then the old mechanism for iterating is used.

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

udpsink: avoid leaking the host

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

v4l2: improve pixel aspect ratio handling

Instead of just assuming a aspect ratio of 1/1 use VIDIOC_CROPCAP to ask
the device.
This also add a pixel-aspect-ratio property to overwrite the value from the
driver and a force-aspect-ratio property to ignore it.

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

v4l2: Fix compilation with older kernels

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

v4l2: call VIDIOC_REQBUFS with count = 0 in pool_finalize

Without this the following sequence fails:

- set_caps()
  - object_stop() (does nothing)
  - set_format() -> VIDIOC_S_FMT
- set_config() -> VIDIOC_REQBUFS with count = N
- set_caps()
  - object_stop()
    - pool_finalize()
  - set_format() -> VIDIOC_S_FMT => EBUSY

Usually the pool is started after set_config(), in which case object_stop()
will result in a pool_stop and therefore VIDIOC_REQBUFS with count = 0 but
that is not guaranteed.
Also calling VIDIOC_REQBUFS with count = 0 in pool_finalize() if necessary
fixes this problem.

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

v4l2: rework sink buffer refcounting

This is a followup patch for #700781, which is not quite correct.
The buffer handling is quite complicated here.
The original code intended to the the following:

- gst_v4l2_buffer_pool_process() calls QBUF and adds the buffer to the
  local list.
- The sink calls gst_buffer_unref() which returns the buffer to the pool
  but not the 'free list'.
- Some time later DQBUF returns the buffer and
  gst_v4l2_buffer_pool_release_buffer() puts in on the 'free list'.

If the buffer must be copied then (parent_class)->acquire_buffer() is
called directly to keep the buffer in the pool.

This has two problems:
1. If gst_v4l2_buffer_pool_release_buffer() is called before the buffer is
   returned to the pool, then the buffer is put on the 'free list' twice.
   This can happen if a reference to the buffer is kept outside the sink,
   of if DQBUF returns the buffer, that was just queued with QBUF.
2. If buffers are copied, then all buffers are in the pool at all times. As
   a result gst_v4l2_buffer_pool_stop() and gst_v4l2_buffer_pool_dqbuf()
   can access pool->buffers at the same time, which can lead to memory
   corruption.

The patch for #700781 fixes those problems, but with the side effect that
there are always buffers outside the pool (because they are queued) and
the pool is never stopped.
This patch fixes this by releasing the reference to the buffer after
handling it (to avoid problem 2.) so it can be returned to the pool.
gst_v4l2_buffer_pool_release_buffer() is only called if the buffer is
already in the pool (to avoid problem 1.).

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

qtdemux: make sure taglist is writable before adding tags

Avoids assertions

qtdemux: effectively skip tracks that weren't listed on the 1st moov

Without this, stream is NULL and the code will try to access it, leading
to segfaults.

qtdemux: skip redundant check

!got_moov is already checked the line above

tests: cleanup level tests

Split out a few more tests to avoid checking the same stuff over and over again.

level: remove unused variables in instance struct

level: add a test for continous timestamps

A test that checks that msg[n].ts + msg[n].dur == msg[n+1].ts.