Revert "Check that collectpads exists before removing pad"

This reverts commit 8e6b876e76c94410db160afe5eb30f21452e419f.

Depends on a core commit that was reverted

icydemux: fix tag list handling issues that might have caused crashes

Fix slightly confused tag handling in some places: make it clear when
we're taking ownership of a tag list and when not. For example,
gst_icydemux_tag_found() was taking ownership when the source pad
existed, but otherwise not (leak). Also, gst_event_parse_tag() does
not return a newly-allocated taglist, but a tag list that belongs to
the tag event, so don't give ownership of it away.

While we're at it, some minor clean-ups: don't re-invent g_strndup()
and simplify gst_icydemux_parse_and_send_tags() a bit, and don't
leak the tag list in case no valid tags where found.

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

Check that collectpads exists before removing pad

The core now calls release pad from finalize, at which point
the collectpads might have already been freed.

pngdec: Handle 16-bit-per-channel images

avidemux: stream->current_total is accumulated byte size and not time

Use timestamp for the stream index as well.

udpsink: warn when packet is too large

matroskaparse: New element

Copied from demux.  Duplicates much code, also some dead code
remaining.

matroskademux: Earlier debug category initialization

flvmux: don't set duration for live stream

debugutils: remove bitrotten negotiation element

Wasn't enabled, didn't work, and planned features have been
superceded by capsfilter and capsdebug.

rtpvrawpay: Implement interlacing

avidemux: also add the frame-type for the stream index

avidemux: get the index writer id when the pad has a parent

Otherwise the index writer has a weired name, as the pad has no parent yet.

avidemux, flvdemux: formatting cleanup

Trim trailing whitespaces and fix the formatting of double negation.

avidemux, flvdemux: mark delta-units in the index

We need to use the 'delta' flag for delta units and not the 'none' flag.

.gitignore: ignore moved equalizer test binary

qtdemux: mark delta-unit in the index

We need to use the delta flag fro delta units and not none. Print more details
to the debug log.

qtdemux: formatting cleanup

Trim trailing whitespaces and fix the formatting of double negation.

matroskamux: rework _request_new_pad to handle explict req-pad-names

Don't ignore explicit pad-names.

avimux: rework _request_new_pad to handle explict req-pad-names

Don't ignore explicit pad-names. Rearrange the code and the error handling a
bit. Add a FIXME-0.11 for the bad pad-names.

icles: Add equalizer-test to the build system

[MOVED FROM BAD 5/5] equalizer-test: Initialize debug category after gst_init() to fix segfault

[MOVED FROM BAD 4/5] tests/icles/equalizer-test.c: Fix gain ranges for the latest equalizer changes.

Original commit message from CVS:
* tests/icles/equalizer-test.c: (do_slider_fiddling):
Fix gain ranges for the latest equalizer changes.

[MOVED FROM BAD 3/5] ChangeLog: ChangeLog surgery. gst/equalizer/gstiirequalizer.c (ARG_BAND_WIDTH, _do_init, ARG_GAIN, _GstIirEqualizerBa...

Original commit message from CVS:
* ChangeLog:
ChangeLog surgery.
* gst/equalizer/gstiirequalizer.c (ARG_BAND_WIDTH, _do_init, ARG_GAIN,
_GstIirEqualizerBand, object, _GstIirEqualizerBandClass,
parent_class, gst_iir_equalizer_band_set_property,
gst_iir_equalizer_band_class_init, gst_iir_equalizer_band_get_type,
gst_iir_equalizer_child_proxy_get_child_by_index,
gst_iir_equalizer_child_proxy_get_children_count,
gst_iir_equalizer_child_proxy_interface_init, setup_filter,
gst_iir_equalizer_compute_frequencies, plugin_init):
* tests/icles/equalizer-test.c:
Add fixme and comment for example.

[MOVED FROM BAD 2/5] tests/icles/equalizer-test.c: Port the example to new equalizer api.

Original commit message from CVS:
* tests/icles/equalizer-test.c: (equalizer_set_band_value),
(equalizer_set_all_band_values),
(equalizer_set_band_value_and_wait),
(equalizer_set_all_band_values_and_wait), (do_slider_fiddling),
(main):
Port the example to new equalizer api.

[MOVED FROM BAD 1/5] Fix up to use the newly ported (actually working) GstAudioFilter.

Original commit message from CVS:
* configure.ac:
* gst/equalizer/Makefile.am:
* gst/equalizer/gstiirequalizer.c: (gst_iir_equalizer_base_init),
(gst_iir_equalizer_class_init), (gst_iir_equalizer_init),
(setup_filter), (gst_iir_equalizer_compute_frequencies),
(gst_iir_equalizer_set_property), (gst_iir_equalizer_get_property),
(gst_iir_equalizer_transform_ip), (gst_iir_equalizer_setup),
(plugin_init):
* gst/equalizer/gstiirequalizer.h:
Fix up to use the newly ported (actually working) GstAudioFilter.
Bump core/base requirements to CVS for this.
* tests/icles/.cvsignore:
* tests/icles/Makefile.am:
* tests/icles/equalizer-test.c: (check_bus),
(equalizer_set_band_value), (equalizer_set_all_band_values),
(equalizer_set_band_value_and_wait),
(equalizer_set_all_band_values_and_wait), (do_slider_fiddling),
(main):
Add brain-dead interactive test for equalizer.

v4l2: Add PJPG mapping

Adds mapping of progressive jpeg format

plug qtdemux refcount leaks

* gst/qtdemux/qtdemux.c (gst_qtdemux_src_convert): Unref the qtdemux; we
  weren't doing so before.
  (gst_qtdemux_handle_src_event, gst_qtdemux_chain): Fix some error
  cases which would leak a ref to the qtdemux.

souphttpsrc: Add URI query handler

Fixes bug #642337.

matroskademux: avoid sorting NULL array of cluster positions

theorapay: handle 0 sized packets

Handle 0 sized packets (repeat frame) in the payloader and depayloader.

Fixes #641827

taginject: resend tags when they are changed

Allow setting new tags on the property while running and send them.

Fixes #640249

Automatic update of common submodule

From f94d739 to 1de7f6a

rtspsrc: fix minor leaks when handling server requests.

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

qtdemux: extract MusicBrainz tags

Extract MusicBrainz tags added by MusicBrainz's Picard
tagger application. These tags (esp. the album id) are
helpful for rhythmbox et.al. to automatically downloads
cover art.

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

qtdemux: refactor iTunes tag parsing a bit

doap: update mailing list location

qtdemux: propagate error during expose_streams

... as it may occur during initial parsing of fragmented file.

qtdemux: avoid skipping exposing a stream following a removed stream

matroskademux: store cluster positions provided by SeekHead

... and use those, if available, to locate a cluster rather than scanning.

matroskademux: properly resume cluster scanning

... rather than getting offset tracking messed up, and then likely
failing a subsequent assert.

id3demux: ensure a taglist before adding the container tag

In the case of id3v1 also don't return NULL on empty tags, but also create a new
taglist and add the container tag for consistency.

rtspsrc: strip trailing spaces

rtpsrc: set multiple properties in one go

There is no need for separate g_object_set() calls here.

deinterlace: Handle image caps without asserting

Images might have framerate=0/1 in the caps, which caused an
assertion on deinterlace. I don't know of interlaced image formats
but deinterlace might be hardcoded on some generic pipelines and
it shouldn't assert.

The fix was to set field_duration to 0 if the input has a framerate
with a 0 numerator.

This patch also adds checks for this situation on the unit tests.

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

docs: fix parameter name in udpsrc docs

It is "buffer-size" and not "buffer". Also trim trailing whitespace.

v4l2: fix interlaced set_format configuration

Commit 6c8268dbfd5c88fac28c882ef2e4598a6522e2d6 broke recording
from interlaced v4l2 source (e.g. typical tv capture card) since
V4L2_FIELD_SEQ_TB (with fields stored separately) does not map
to currently defined interlaced format (fields stored interleaved).

Besides this mismatch, hardware might quite likely not support or
appreciate this field value, since querying supported formats mapped
_INTERLACED field formats to interlaced=true caps (so the latter should
not be mapped to field value that is not known to be supported).

source: fix type of ntpnstime

rtpbin: Get and use the NTP time when receiving RTCP

When we receive an RTCP packet, get the current NTP time in nanseconds so that
we can correctly calculate the round-trip time.

directsound: arrange for definition of _swab on Cygwin

gstdirectsoundsink.c: In function 'gst_directsound_sink_write':
gstdirectsoundsink.c:557: error: implicit declaration of function '_swab'
gstdirectsoundsink.c:557: error: nested extern declaration of '_swab'

rtptheoradepay: Request new keyframe on lost packets

Theora can only use the last frame (or the keyframe) as a reference, so in
practice. If we receive a buffer that references an unknown codebook, request
new headers. It probably means that headers were lost.

rtpsession: Add action signal to request early RTCP

rtpsession: Add callback to get the current time

rtpsession: Don't relay more than one PLI request per RTT

Drop PLI requests if one was relay in the last RTT, the other side may
just not have received the keyframe yet.

rtpsession: Send GstForceKeyUnit event in response to received RTCP PLI

gstrtpsession: Fallback for FIR to PLI if PLI isn't available

rtpsession: Implement sending PLI packets in response to GstForceKeyUnit

rtpsource: Retain RTCP Feedback packets for a specified amount of time

rtpsession: Make rtcp buffer metadata writable after processing it

Functions that process the rtcp buffer could decide to keep a ref
on the buffer for further processing. So make the metadata writable
only after they are done.

rtpsession: Emit signal on incoming RTCP FB packet

rtpsession: fix compilation

rtpsession: Add method to request early RTCP packet

Implement the early mode defined in RFC 4585. In this mode, RTCP feedback
packets are sent early to notifier.

rtpsession: Add property for minimum interval between Regular RTCP messages

This can be changed according to RFC 4585

rtpsession: Emit signal when sending a compound RTCP packet

This allows users to add extra RTCP packets to the compound
RTCP packet.

rtpptdemux: Tag upstream custom events with payload type

rtpssrcdemux: Tag upstream custom events with SSRC

rtpsession: Emit "on-ssrc-validated" when validating by RTCP

Emit "on-ssrc-validated" if the SSRC is validated by receiving
a RTCP SDES packet.

j2kpay: skip EPH packets

Include EPH markers into the previous chunk of packets.

rtppcmapay: Rename the class to have the right name

It was name pmca instead of pcma and made debug logs hard to search.

Pulsesink: Allow chunks up to bufsize instead of segsize

By allowing larger chunks to be sent, PulseAudio will have a
lower CPU usage. This is especially important on low-end machines,
where PulseAudio can crash if packets are coming in at a higher
rate than PulseAudio can process them.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>

deinterlace: simplify template caps

We can merge all the YUV variants into one single structure.

win32: fix DEFAULT_AUDIOSINK, should be direct*sound*sink

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

avidemux: initialize local variable to please mingw32 compiler

udp: use socklen_t where appropriate rather than custom type

In particular, fixes Cygwin build where socklen_t is defined as int
in line with native win32 api definition.

qtdemux: mind rounding issues when converting from global time to mov time

In particular, this avoids missing the intended keyframe when first converting
from the frame's mov time to global segment time, and then back from global
time to mov time when activating the segment.

matroskamux: don't leak ebml writer caps when re-using matroskamux

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

rtpjitterbuffer: don't divide by 0

matroskademux: pull mode should always report seekable

... as it no longer requires an index, but can seek by scanning as well.

qtdemux: support some more mpeg-4 fourcc variants

qtdemux: simplify retrieving stsd child entry atom

avidemux: Don't consider 0 fcc_handler as uncompressed.

Just avoids a warning

qtdemux: take configured start time into account

when creating the newsegment event, take the configured start time
into account.

qtdemux: fix printf format warning on mingw32

Make win32 build bot happy again, and nicefy output while we're at it.

qtdemux.c: In function 'qtdemux_parse_trun':
qtdemux.c:2162:3: error: format '%lu' expects type 'long unsigned int', but argument 9 has type 'guint32'

examples: autoaudisink -> autoaudiosink in RTP examples

Back to development

Release 0.10.27

h264depay: don't leak codec data buffer in byte-stream=true mode

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

rtspsrc: don't leak url string

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

qtdemux: Gracefully handle mov files misusing the WAVE atoms

Check that the WAVEHEADER node is present instead of blindly using it.
If not present we won't be able to provide a more refined caps, but at
least we won't crash.

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

v4l2sink: fix accidental breakage of navigation interface support

0.10.26.4 pre-release

deinterlace: rewrite how neighboring scan lines are calculated

Old code was difficult to understand exactly how the neighboring
scan lines are calculated, and it appeared that some were off by
+2 or -2, depending on the field flag.  Fixes #639321.

avisubtitle: set caps on srcpad to fix issue with discoverer

Set caps from the start so discoverer doesn't blow up on
seeing no negotiated caps between elements on preroll,
which might happen if no subtitle buffers have been
pushed yet at the time. See file from bug #603308.

pulsesink: Uncork stream while flushing the ringbuffer

After starting the ringbuffer, we wait for enough data to arrive before
uncorking the stream. This will cause the pipeline to stall if we get an
EOS (or otherwise need to flush the stream) before sufficient data
becomes available. This patch makes sure that the stream is uncorked
while flushing to avoid this problem.

Fixes issue with a webkit unit test testing reverse playback of
an MP4 H.264/AAC file.

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

matroskamux: avoid creating caps from string when possible

Fixes #639516.

avimux: set src pad caps when starting file

Fixes #639516.

v4l2: define V4L2_FIELD_INTERLACED_{TB,BT} if not available in header

Older kernels don't have these, and there's no easy way to check for the
existance of enums that doesn't involve a configure check, so just define
these if the V4L2_CAP_VIDEO_OUTPUT_OVERLAY define is not there, which was
added in the same commit as the TB/BT enum. Fixes compilation on CentOS 5.

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

0.10.26.3 pre-release

docs: update docs

pulsesink: Make corking during pause synchronous

This makes the call to pa_stream_cork() during ringbuffer pause()
synchronous, which makes sure that the clock does not advance after we
take a snapshot for start_time.

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