flacparse: tell baseparse the duration in samples for better accuracy

Tell GstBaseParse the duration in samples instead of time, so that
a duration query in DEFAULT format will return the correct number
of samples without rounding errors. Baseparse will convert this
into time itself when needed.

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

flacdec: also try upstream first for duration query in DEFAULT format

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

flacparse: make conversion from TIME to DEFAULT format (samples) work

Fix copy'n'paste error in the previous commit.

flacparse: Implement conversions between TIME and DEFAULT format

Fixes bug #650785.

flacparse: don't error out on invalid minimum_blocksize value in streaminfo header

We don't use it, so may just as well accept an invalid value
of 0 here, which is likely inconsequential anyway.

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

rtp: fix static array overruns in a nicer way

Use G_N_ELEMENTS instead of hard-coding the array size.

rtp: fix static array overruns

Yes array[10] has elements from 0...9.

docs: update plugin introspection data

Now more files are merged and produced in a canonical fashion, which hopefully
creates less or no delta in the future.

Automatic update of common submodule

From 9e5bbd5 to 69b981f

qtdemux: add missing break

deinterlace: Add support for deinterlacing using buffer caps/flags

When not using the fieldanalysis element immediately upstream of deinterlace,
behaviour should remain unchanged. fieldanalysis will set the caps and flags on
the buffers such that they can be interpreted and acted upon to produce
progressive output.

There are two main modes of operation:

- Passive pattern locking
  Passive pattern locking is a non-blocking, low-latency mode of operation that
  is suitable for close-to-live usage. Initially a telecine stream will be
  output as variable framerate with naïve timestamp adjustment. With each
  incoming buffer, an attempt is made to lock onto a pattern. When a lock is
  obtained, the src pad and output buffer caps will reflect the pattern and
  timestamps will be accurately interpolated between pattern repeats. This
  means that initially and at pattern transitions there will be short periods
  of inaccurate timestamping.

- Active pattern locking
  Active pattern locking is a blocking, high-latency mode of operation that is
  targeted at use-cases where timestamp accuracy is paramount. Buffers will be
  queued until enough are present to make a lock. When locked, timestamps will
  be accurately interpolated between pattern repeats. Orphan fields can be
  dropped or deinterlaced. If no lock can be obtained, a single field might be
  pushed through to be deinterlaced.

Locking can also be disabled or 'auto' chooses between passive and active
locking modes depending on whether upstream is live.

configure: Remove config script check for caca

rtp: Fix segmentation fault processing payload buffers

This commit checks if the value returned by
gst_rtp_buffer_get_payload_buffer and
gst_rtp_buffer_get_payload_subbuffer is NULL before using it.

Automatic update of common submodule

From fd35073 to 9e5bbd5

avidemux: ensure 0-padding when correcting dubious list size

Automatic update of common submodule

From 46dfcea to fd35073

rtspsrc: use EINVAL for missing url parameter

Fixes gcc warning about using uninitialized variable 'res'.

various: fix author tag in element details

auparse: implement seeking

Implement seeking and seeking query. Fixes #644512

rtspsrc: also allow PAUSE to be interrupted

... as it is on the way out to NULL.

See #632504.

rtspsrc: ensure proper closing and cleanup

... since the TEARDOWN sequence might not have had a chance to even start,
but at least connections should be closed (synchronously) and state cleaned up.

See #632504.

rtspsrc: fix and improve async handling

Simplify the command handling; passing a command to thread means we really
want it to get the message, which means to always flush provided the command
can handle being interrupted.  Command thread indicates whether command
allows interruption and ensure non-flushing connection as it subsequently
needs it.

In particular, this also makes the TEARDOWN sequence interruptable
and also prevents races where _loop_ could miss a command and would
continue receiving (or at least trying to).

See #632504.

rtspsrc: tweak post-seek loop handling

rtspsrc: open on play and pause when not done yet

With the async state changes, it is possible that we need to open the stream
before play and pause.
Also make sure we remember a previous open failure so that we don't keep trying
again.

rtspsrc: improve async handling

Simplify the command handling, only continue looping when we have not received
another command or when the previous loop was successfull.
Avoid looping on a disconnected socket.

rtspsrc: rework reconnect code

Use the same async code path to implement reconnects.
Make sure we only post progress messages when doing async things.

rtspsrc: small cleanups

Make sure we cancel the previous task when queuing a new one.
Move the messages to a central place so we can more easily post them.

rtspsrc: don't post errors when interrupting

rtspsrc: implement more async handling

Remove some old locks.
Make sure we never go into the loop function when flushing.

rtspsrc: first attempt at async implementation

rtspsrc: small header cleanups

ssrcdemux: Fix uninitialized variable compiler warning for (pre-) releases too

v4l2objects: Only allow mpeg-ts on source objects

Ugly fix for #648312

rtpssrcdemux: Fix uninitialized variable compiler warning

ssrcdemux: Implement iterate internal links for sink pads

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

rtpssrcdemux: iterate pad function is only valid for src pads

The iterate function is only used for src pads, so mark it as such and remove
dead code.

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

rtpssrcdemux: Release lock before emitting signal

If the lock is not released before emitting a signal, it may cause a deadlock
if any other function in the element is called.

Also removed an unused timestamp parameter

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

matroskaparse: calculate segment duration after parsing all the IDs

Since the segment duration is given in terms of the
GST_MATROSKA_ID_TIMECODESCALE we should only convert it into
nanoseconds when we are sure that any scale specified in the file has
been read.

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

matroskademux: additional lock safety

Fixes #619590.

qtdemux: also check for bitrate info in caps

qtdemux: guess bitrate if only one stream's bitrate is unknown

If the bitrates for all but one audio/video streams are known, and the
total stream size and duration can be determined, this calculates the
unkown bitrate as (stream size / duration) - (sum of known bitrates).
While this is not guaranteed to be very accurate, it should be good
enough for most purposes.

For example, this is useful for H.263 + AAC streams where no 'btrt' atom
is available for the video portion.

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

qtdemux: Export max bitrate for AMR-NB/-WB streams

This parses the 'damr' atom if present, and exports the maximum bitrate
of the stream using the mode set field to determine the highest bitrate
frame type that might be present.

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

pulse: Define PATH_MAX if it isn't defined

GNU Hurd for example doesn't define it.

matroskademux: calculate segment duration after parsing all the IDs

Since the segment duration is given in terms of the
GST_MATROSKA_ID_TIMECODESCALE we should only convert it into
nanoseconds when we are sure that any scale specified in the file has
been read.

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

flvmux: Add support for mpegversion 2, which is also AAC

flacdec: Send EOS when seeking after the end of file instead of failing

Fixes bug #649780.

wavenc: Set fixedcaps getcaps function on the sinkpad

wavenc does not allow to change the caps during playback
and always returning the template caps is just wrong.

wavenc: Allow setcaps to be called after a format was negotiated if it's compatible

Otherwise wavenc will fail if upstream decides to set equivalent caps or caps
with additional information later.

Thanks to Alexander Schremmer for finding this bug.

Back to development

Release 0.10.29

Highlights:

  - amrparse, aacparse, ac3parse, flacparse, mpegaudioparse, dcaparse audio parsers (moved from -bad)
  - muxers now mux based on running time
  - ISO MP4 muxers: mp4mux/3gppmux/qtmux/mj2mux (moved from -bad)
  - new matroskaparse element
  - new v4l2radio element
  - rtpsession: support RTCP Early Feedback (the AVPF profile)
  - orc 0.4.14 or newer recommended
  - many other fixes and improvements

qtmux: Fix signed floating point values writing

You would end up on some architectures with 0 being written out
instead of the proper value.

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

matroskamux: avoid building index when streamable

... as it will not be written anyway.

Fixes #648937 (?).

build: add old qtdemux/quicktime directories to CRUFT_DIRS and CRUFT_FILES

flvmux: don't overwrite metadata tag with duration in streaming mode

A duration tag gets inserted only for streamable=false, so only
update/write the duration later if we actually inserted that tag,
otherwise we write garbage into other tags.

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

0.10.28.4 pre-release

quicktime: rename plugin to isomp4

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

0.10.28.3 pre-release

rtpgstpay: fix buffer leak

jpegdec: documentation typo "jpegddec"

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

avimux,matroskamux: Add stream-format to h264 caps

Fixes #606662.

pngdec: Remove temporary code

Now that we depend on (what will be) -base-0.10.33.

configure: don't pass -Waddress to ObjC compiler on OSX when compiling osxvideosink

Temporary workaround until we fix this properly and check for
the ObjC warning/error flags instead of just passing CFLAGS to the
ObjC compiler.

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

quicktime: rename plugin filename from *qtdemux* to *quicktime*

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

Automatic update of common submodule

From c3cafe1 to 46dfcea

docs: add various qtmux variants to documentation

quicktime: register 3gppmux element in addition to the misnamed gppmux

rtpsession: Remove incomplete support for RTCP FIR

Remove bits that were meant to suppport RTCP FIR

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

tests: add generic set_index test

flvdemux: fix deadlock on setting index on flvdemux

tests: add index-setting test for baseparse/flacparse

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

wavpack: Remove bus GSource to prevent a valgrind warning

wavenc: Remove bus GSource to prevent a valgrind warning

tagschecking: Remove bus GSource to prevent a valgrind warning

imagefreeze: Remove bus GSource to prevent a valgrind warning

avidemux: fix 'variable may be used uninitialized' warnings caused by -DG_DISABLE_ASSERT

0.10.28.2 pre-release

gst: update disted orc backup code

docs: update for pre-release

po: update translations

qtmux: refuse incomplete legacy h264 caps

Refuse h264 caps without stream-format and codec_data fields for
now, to avoid creating broken files. This might cause some pipelines
that worked previously to fail. However, the move from -bad to -good
is our only chance to fix this up, so make it strict for now. We can
always change it back to be less strict in future.

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

v4l2sink: fix another unused-but-set-variable warning

pulse, speexenc, rtpgsmpay: don't use g_assert() for error handling

Don't use g_assert() for error handling, even if they're highly unlikely.
Either we *know* that something can't happen, in which case we
should just not handle it, or we think something can happen, but it is
very very unlikely that it will ever happen, in which case we should
handle it like any other error instead of asserting.

g_assert() is best left for conditions we have control of, like checking
internal consistency of our code, not checking return values of external
code.

Fixes a bunch of warnings when compiling with -DG_DISABLE_ASSERT:
gstrtpgsmpay.c: In function 'gst_rtp_gsm_pay_handle_buffer':
gstrtpgsmpay.c:130:17: warning: variable 'rtpgsmpay' set but not used
gstspeexenc.c: In function 'gst_speex_enc_encode':
gstspeexenc.c:904:19: warning: variable 'written' set but not used
pulsesink.c: In function 'gst_pulsesink_change_state':
pulsesink.c:2725:9: warning: variable 'res' set but not used
pulsesrc.c: In function 'gst_pulsesrc_change_state':
pulsesrc.c:1253:7: warning: variable 'e' set but not used

examples: fix some warnings in rtp example

Caused by -DG_DISABLE_ASSERT

examples: don't put code with side-effects into g_assert()

Otherwise things won't work too well when compiling with
-DG_DISABLE_ASSERT (as we do for pre-releases and releases).

deinterlace, matroska: fix two variable-may-be-used-uninitialized compiler warnings

We use -DG_DISABLE_ASSERT for the pre-releases, which makes these
warnings pop up in cases that were previously covered by g_assert_not_reached()
and the like:
tvtime/greedyh.c:801:14: warning: 'scanline' may be used uninitialized in this function
matroska-mux.c:501:19: warning: 'context' may be used uninitialized in this function

jack: fix unused-but-set-variable warnings with gcc-4.6

examples: fix 'control reaches end of non-void function' warning in cairo example

v4l2src: Address unused but set variable

The v4l2object formats list was being obtained into a local variable and
then still used from the context. Make use of the local variable.

oss4: Address unused but set variables

GCC 4.6.x complains about such variable usage. Unused but set variables
were removed except that gst_oss4_mixer_slider_set_mute () now returns
the value from the call to gst_oss4_mixer_set_control_val ().

jpegenc: pulsesink: raw1394: Address unused but set variables

GCC 4.6.x spits warnings about such usage of variables. The variables in
raw1394 were marked with G_GNUC_UNUSED as this seemed omre appropriate.
The others were removed.

y4mencode: shapewipe: Address unused but set variables

GCC 4.6.x complains about such usage.

tests: Address unused but set variables

GCC 4.6.x spits warnings about such usage of variables.

videomixer: Fix argb/rgba overlay orc code

Remove some redundant operations (convubw) and use the correct variable,
t2, in the orc_overlay_bgra function.

videomixer: address unused but set variables

GCC 4.6.x spits warnings about variables that are set but unused. Such
variables have been removed in blend, collectpads2 and videomixer2.

rtp, rtpmanager: Address unused but set variables

GCC 4.6.x spits warnings about variables that are unused but set. Such
variables have been removed where trivial but with comments left behind
for informational purposes in some cases.

gst_rtp_session_chain_recv_rtcp () was changed in commit 490113d4
to always return GST_FLOW_OK instead of the return value of
rtp_session_process_rtcp (), so we'll keep it that way.

quicktime: Remove unused but set variables

GCC 4.6.x spits warnings about such variable usage. Note that some
calculations are left as comments for informative purposes.

matroska: Remove unused but set variables

GCC 4.6.x spits warnings about such variable usage.

imagefreeze: Remove unused but set duration variable

GCC 4.6.x spits warnings about such variable usage.

flxdemux: Remove unused but set keyframe variables

The FIXMEs about the keyframe flag never being used are left for later
fixing, at which point the keyframe variables could be added back.

edgetv: Remove unused but set height variable

GCC 4.6.x spits warnings about such variables.