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.

flacparse: update for gst_base_parse_frame_init() API change

rtpsession: Use existing functions to parse RTCP FB packets

Use existing functions to get the FCI from FB packets.

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

rtpsession: marshal GstBuffer as a MiniObject instead of a pointer

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

matroskademux: Better calculation of framerate

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

qtmux: default to dts-method=reorder and presentation-time=true

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

tests: qtmux: test various dts-methods

qtmux: fix corner case buffer handling for reorder method

flvdemux: Don't leak the SEEKING query

qtmoovrecover: Don't leak the static recursive mutex

v4l2radio: Free videodev string before replacing it

matroskaparse: Allow webm and matroska caps and don't leak caps

Add parser plugin

directsoundsink: Add conditionals on WAVE_FORMAT_DOLBY_AC3_SPDIF

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

avidemux: fix unused-but-set-variable warning with gcc 4.6

Most likely a leftover from when the index parsing code was rewritten.

ac3parse: fix unused-but-set-variable warning with gcc 4.6

videobalance: fix handling of YUV images with 'odd' widths

Fixes unused-but-set-variable warnings with gcc 4.6.

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

audiowsinc{band,limit}: Fix check for divison by zero

audiowsincband: Fix range of kernel elements (lim -> lim-1)

audiowsinclimit: Add some more braces to make the code more readable

audiowsinclimit: Fix range of kernel elements (lim -> lim-1) in high/low-pass filters

audiowsincband: Add new windowing functions: gaussian, cos and hann

audiowsinclimimt: Add new windows to high/low-pass filters: gaussian, cosine, hann

matroskademux: set stream-format=byte-stream on h264 caps if there's no codec data

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

qtmux: restrict h264 some more to only accept AU-aligned AVC

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

mpegaudioparse: The VBRI header is always at offset 0x20, independent of MPEG version

Also clean up advancing of the data pointer a bit.

Fixes bug #647659.

qtmux: add variant-less video/quicktime to source pad template caps

This is needed for automatic transcoding using encodebin. Our typefinder
does not always add a variant to the found caps, and encodebin needs
an *exact* match to the caps on the source pad template, so we need
to add the variant-less video/quicktime caps to the template as well
for encodebin to be able to find it. Add unit test for this as well.

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

flacenc: Properly interprete the result of strcmp()

flacenc: Don't store image tags inside the vorbiscomments and the flac metadata

Instead only store them inside the flac metadata. There's
no point in storing them twice and the flac metadata is
still the official way to store image tags inside flac.

tests: ignore new qtmux-related test binaries

quicktime: move qtmux plugin from -bad to -good

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

qtmux: more helpful debug error message when no needed duration on input buffers

Fixes #646256.

qtmux: Adding GstTagXmpWriter interface

Adds GstTagXmpWriter interface support to qtmux

qtmux: use running time for synchronization

See also #432612.

qtmux: provide for PTS metadata when so configured

... and not only when sort-of feeling like it.
In any case, if it turns out all really is in order,
and presumably DTS == PTS, then no ctts will be produced anyway.

qtmux: also track original PTS buffer timestamp in reorder dts-method

Revert "Check that collectpads exists before removing pad"

This reverts commit 6d8740476ccd3a3498dc4f18c19733643825c7b8.

Depends on a core commit that was reverted

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.

test: qtmux: Tests qtmux reuse

Forces the use of qtmux after it has been put to PLAYING and back
to NULL once

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

qtmux: set src pads when starting file

... rather than at _init time, so they are also available following a
pad (de)activation cycle.

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

qtmux: adjust nasty case timestamp tracking

That is, all sorts of problems arise with re-ordered input timestamps that
tend to defy automagic handling for every case, so allow for a few variations
that can be tried depending on circumstances.

Also try to document accordingly.

Also fixes #638288.

qtmux: get rid of timestamp overprotectiveness

Signed-off-by: Felipe Contreras <felipe.contreras@nokia.com>