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>

qtmux: simplify and fix pts_offset storing

In particular, only write a ctts atom if and only if ever a non-zero offset.

qtmux: add some more documentation

qtmux: remove large-file property

Rather, auto-determine if 64-bits fields are needed for a valid result, and
stick to plain 32-bits if not needed.

API: GstQTMux:large-file (removed)

qtmux: Free AtomInfo structs

qtmux: Free tag string after use

tagschecking: Fix some more memory leaks

qtmux: allow zero duration tracks

qtmux: add documentation

qtmux: handle msvc ftruncate incompatibility

Fixes #636185.

qtmux: gst_qtmux_check_difference verify before subtract

Avoid negative overflow by checking the order of operands
on subtraction of unsigned integers.

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

qtmux: remove remnant of obsolete property

tests: qtmux: also unit test fragmented file cases

qtmux: allow specifying trak timescale

This is mainly because Smoothstreaming client are broken and don't
take the TimeScale property into account.

qtmux: include sdtp atoms for ismv fragmented files

Based on patch by Marc-André Lureau <mlureau@flumotion.com>

qtmux: enable default fragmented file for ismlmux

qtmux: add ismlmux, for fragmented isml major brand

qtmux: finalize sinkpads list

qtmux: add moov in streamheader

qtmux: add streamable property to avoid building fragmented mfra index

qtmux: add mfra to fragmented file

Based on patch by Marc-André Lureau <mlureau@flumotion.com>

qtmux: optionally create fragmented file

In this mode, an initial empty moov (containing only stream metadata) is written,
followed by fragments containing actual data (along with required metadata).
New fragments are started either at keyframe (if such are sparse) or when
property configured duration exceeded.

Based on patch by Marc-André Lureau <mlureau@flumotion.com>

Fixes #632911.

qtmux: use helper to set atom flags from given uint

qtmux: refactor configuring and sending of moov

Based on patch by Marc-André Lureau <mlureau@flumotion.com>

qtmux: refactor extra top-level atom handling

Also check a bit more for possible errors, and free proper items in such case.

qtmux: refactor slightly using buffer helper

qtmux: fix misinforming comment

qtmux: delegate mvex handling to atoms

... which keeps qtmux simpler.

qtmux: add mvex/trex in header if fragmented

One "trex" is added per "trak". We don't support default values,
but the "trex" box is mandatory.

qtmux: add a couple of fourcc for fragmented mp4

qtmux: avoid removing temp file when error occurred

qtmux: truncate buffer file after each send

qtmux: remove temp file when reset/finalize

various (gst): add missing G_PARAM_STATIC_STRINGS flags

Canonicalize property names as needed.

qtmux: prevent infinite loop when adjusting framerate

Fixes #632070.

qtmux: Add G_PARAM_STATIC_STRINGS

Add G_PARAM_STATIC_STRINGS to qtmux properties

qtmux: Follow xmp serialization guidelines closer

qt and isom variants have different ways of serializing
xmp, follow these guidelines.

Those can be found in Adobe's xmp docs.

qtmux: autodetect out-of-order input timestamps and determine DTS accordingly

Favour using input buffer timestamps for DTS, but fallback to using buffer
duration (accumulation) if input ts detected out-of-order.

Fixes #624212.

qtmux: use caps bitrate at last chance

If we didn't get the stream's bitrate from one of the atoms,
try getting it from the caps as a last resort.

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

qtmux: btrt - max bitrate before average

According to iso base media file format, the max bitrate
is before the avg

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

qtmux: Write 'btrt' atom for H.264 media if possible

This writes out the optional 'btrt' atom (MPEG4BitrateBox) for H.264
media if either or both of average and maximum bitrate are available for
the stream.

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

qtmux: Write avg/max bitrate to ESDS if available

This collects the 'bitrate' and 'maximum-bitrate' tags on the
corresponding pad and uses these to populate these fields in the ESDS
where applicable.

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

qtmux: Don't use bogus codec/format tags

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

qtmux: Write uint tags that don't have a complement

Write uint tags that have complements (e.g. track-number/
track-count) even when we only have one of them available
and set the other one to 0.

Fixes #622484

qtmux: Remove the pad from our internal list before calling collectpads

Previously we would end up with the collectpaddata structure already freed.
This would result in a bogus iteration of mux->sinkpads (all the
GstQTPad being freed) and it wouldn't be removed from that list.

Finally, due to it not being removed from that list, we would end up
calling a bogus gst_qt_mux_pad_reset on those structures => SEGFAULT