stats: always free log

We always want to free the open file log if fopen() succeeded. Independently
of if fgets() succeeds or fails.

CID 1326055

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

segment: Correct stream_time calc for negative applied rate

Updated gst_segment_position_from_stream_time and gst_segment_to_stream_time to reflect correct calculations for the case when the applied rate is negative.

Pasting from design docs:

===============================
Stream time is calculated using the buffer times and the preceding SEGMENT
event as follows:

    stream_time = (B.timestamp - S.start) * ABS (S.applied_rate) + S.time

For negative rates, B.timestamp will go backwards from S.stop to S.start,
making the stream time go backwards.
===============================

Therefore, the calculation for applied_rate < 0 should be:

    stream_time = (S.stop - B.timestamp) * ABS (S.applied_rate) + S.time

and the reverse:

    B.timestamp = S.stop - (stream_time - S.time) / ABS (S.applied_rate)

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

tracer: rename the envvar to GST_TRACER_PLUGINS

The subsystem reused the GST_TRACE var that is allready in use by the alloc tracer.
Fixes #756760

multiqueue: Improve incoming SEGMENT handling

Previously this code was just blindly setting the cached flow return
of downstream to GST_FLOW_OK when we get a SEGMENT.

The problem is that this can not be done blindly. If downstream was
not linked, the corresponding sinqlequeue source pad thread might be
waiting for the next ID to be woken up upon.

By blindly setting the cached return value to GST_FLOW_OK, and if that
stream was the only one that was NOT_LINKED, then the next time we
check (from any other thread) to see if we need to wake up a source pad
thread ... we won't even try, because none of the cached flow return
are equal to GST_FLOW_NOT_LINKED.

This would result in that thread never being woken up

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

gstutils: Fix build with clang -Werror=cast-align

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

bin: Make sure to free all cached messages when going to NULL

An ASYNC READY->PAUSED might have failed without the bin code noticing during
the state change, in which case we will never get PAUSED->READY and would leak
messages.

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

docs: mention xdot utility to view .dot files directly

alloctrace: show details of events and messages leaked

So it's clearer what leaked.

docs: manual: improve advanced metadata example a bit

Accept both filename and a URI as argument, and print
the error from the error message if there's an error.

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

memory: fix typo in documentation

It should be 1.2 unless this is a flag from the future

multiqueue: Accept STREAM_START after EOS

In the same way core now allows STREAM_START to remove the flushing
state from pads, we need to do the same thing in multiqueue

segment: Convert function to macro in unit test to get proper line numbers on failures

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

baseparse: Update internal position even if not linked

Our current position has nothing to do with being linked or not.

Avoids having stray segment updates fired every 2s

pad: fix memory leak when sending events to an EOS pad

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

tracers: Only link against libgstprintf.la if the debugging system is enabled

It does not exist otherwise and linking will fail.

gst-ptp-helper: #include <sys/socket.h> to fix net/if.h include on OSX 10.6

In file included from gst-ptp-helper.c:40:0:
   /usr/include/net/if.h:265:19: error: field 'ifru_addr' has incomplete type
      struct sockaddr ifru_addr;

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

Revert "tracers: Only build getrusage() tracer if RUSAGE_THREAD is available"

This reverts commit 8ddbf76626a48420306869db1d171f854cc25310.

tracers/rusage: ifdef the RUSAGE_THREAD usage

Some versions of andoid don't seem to have it.

tracers: Only build getrusage() tracer if RUSAGE_THREAD is available

win32: remove gst_tracer_quark_id_get_type

Revert addition from 777bbeea605051ae3d2fa7e02ad8589001e78ce0.

tracer: move prototype to the right header

Fixes the build when the tracing subsystem is disabled.

tracer: mark GstTracerQuarkId as non GEnum

This reverts 72ca02b1de4066eeae35c891e275386770117778 and marks the enum
accordingly.

tracers: disable the log tracer if debug logging is disabled

makefile.am: Remove obsolete Android build cruft

This is not needed any longer.

tracer: fix the build with debug (tracer) disabled

Sync the macro definitions. The dummy defines has argument mismatches.

tracer: fix the build with debug (tracer) disabled

Remove commas at the end of the macros.

tracers: stats: add message structure to output

The name of the message is not enough. For example, state-change
is not enough to know the transition.

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

gst: adding tracer quark id to gst init and deinit

Fixes issues at make check

win32: libgstreamer: add tracer functions

tests: fix the tag test

The previous change (see bgo #756069) was causing us to free the same pointer
multiple times. If we actually get a sample back, the test fails, no need to
free anything in that case.

tracer: add missing hooks

Add following hooks: element-new, element-add-pad, element-remove-pad,
bin-add-pre, bin-add-post, bin-remove-pre, bin-remove-post, pad-link-pre,
pad-link-post, pad-unlink-pre, pad-unlink-post.

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

tracer: gststats: add thread-id to log line

tracer: add pad query hooks

tracer: strdup the passed parameters.

tracer: Use GST_TIME_ARGS when printing with GST_TIME_FORMAT.

tracing: update docs

stats: TIMESTAMP -> PTS

tracing: rename the global api to gst_tracing

This makes it more obvious what is the api for tracer elements and what is api
for the global state.

stats: fix cpu stats printing

Only print them if we have them. Also scale them by 10.0 as the are in
per-mille now.

tracers: eliminate var_args

Register tracer hooks as GCallback. Use macros for hook dispatch and cast the
hook functions back to the appropriate type.

stats: don't warn on ".class" log lines

tracers: code cleanups

Move static variables to instance variables. Add finalize methods. Remove code
that is commented out. Cleanup locking code.

tracer: use GQuark or strings for the hook id

This way one can define new tracing probes without changing the core. We are
using our own quark table, as 1) we only want to initialize them if we're
tracing, 2) we want to share them with the tracers.

tracer: simplify hook api

Instead of a single invoke() function and a 'mask', register to individual
hooks. This avoids one level of indirection and allows us to remove the
hook enums. The message enms are now renamed to hook enums.

tracer: drop the HookId hid from the invoke method

The MessageId is more detailed and anyway needed to interpret the varargs.

stats: fixup doc name and remove commented code

tracers: add metadata for the logged values

rusage: improve cpu load meassurements

Get the number of cpus and scale process cpu-load accordingly. Switch the
cpuload to be per-mille to get smoother graphs. Add a bit more logging and use
the _OBJECT variant.

tracer: remove commented code

design: update tracer design

Update the tracer event classes section. Add a performance section.

tracer: use GST_PTR_FORMAT to log the structure

This way we only expand the structure when we're logging. This allows us to
meassure the pure tracing seperately from the logging.
Also add some comments on further improvements.

rusage: implement windowing of cpuload

Add a local help to the rusage plugin that supports windowing of values. We want
to generalize this for use in other plugins.

rusage: announce the data format

Rusage will now announce what is meassures and how it is logged. Use the new format in stats.
Cleanup the the code and naming.

stats: improve cpu load meassurements

Rename variables for clarity. Handle the initial disparity between debug time
and the time already spent in the proc and main thread.

design: update tracer design

quarks: revert the quark changes, we not using them anymore

tracer: split into tracer and tracerutils

Keep tracer base class in tracer and move core support into the utils module.
Add a unstable-api guard to the tracer.h so that external modules would need to
acknowledge the status by setting GST_USE_UNSTABLE_API.

docs: add gtk-doc blobs

latency: take stop time when buffer is handled

Now we meassure time from 'before buffer transfer on src' to when the 'buffer is processed on sink'.

tracers: add a logging helper to remove identical copies from the tracers

tracers: tweak the get_real_pad_parent()

By using the we ended up on the actual element, not the parent.

tracers: add a new latency tracer

Add a new tracer with pushes extra events to meassure src-to-sink processing latency.

design: update design docs

Add new tracer idea.

gst-stats: use the rusage stats

Add cpuload info from rusage traces.

rusage: add a new rusage tracer

The tracer hooks up to all probes and logs resource usage figures.

gst-stats: filter complete thread section if we have no pads

stats: improve the handling of parentage

Log new object after we did the check for parents.

stats: print thread key for stats and filter empty pad-sections

stats: update buffer flags

Remove some buffer flags that were leftovers from 0.10 and handle new 1.0 buffer
flags.

stats: add a stats frontend

Parse the log and collect data from tracer messages.

tracer: use the same timebase as the logging

design: update design

log: add query log category

tracer: parse parameters

tracer: add more hooks and handle it in the plugins

stats: handle buffer lists

log: make the log tracer more verbose again

Define log formats per message type and print details.

tracer: use a macros for the enabled check

Avoid a function call and check the variables from the macro.

tracer: use macros for hooks

Wrap the hook with a pre and post macro. This looks less intrusive than the
previous version, although it is a little less optimized.

tracer: pass the timestamp directly

Avoid the structure mashalling (and weird field naming).

stats: add a tracer that collects pipeline statistics

This is more or less equiv to the the statistics in gst-tracelib.

log: add cast macro

tracer: pass the instance to the vmethod

design: more planning

tracer: switch to quarks and add another hook for buffer flow

Use pre-defines quarks as this will be called quite often.

tracer: add the hook-id to the invoke signature

Tracers that subscribe to multiple hooks can know what hook was used.

inspect: add support for the new factory

Handle tracer modules.

tracer: initial prototype for the tracing subsystem

tests/gst-launch: Fix sample memory leak

When sample is got using gst_tag_list_get_sample_index, it should
be free'd.

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

Update GLib dependency to 2.40.0

gstreamer: Fix memory leaks when context parse fails

When g_option_context_parse fails, context and error variables are not getting free'd
which results in memory leaks. Free'ing the same.

And replacing g_error_free with g_clear_error, which checks if the error being passed
is not NULL and sets the variable to NULL on free'ing.

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

timedvaluecontrolsource: Use g_sequence_lookup where possible

When looking for exact matches in the sequence, this results
in much simpler code than when using g_sequence_search.

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

interpolationcontrolsource: fix write over the array size

The '++' got incidentially added during the refactoring in
2fe3939ce7ea84c45dd922e7f1097dd07f11fc5d.

controlpoint: change the padding to be of arch-independent size

The default padding I introduced in d4f81fb4e62d34a4c1dabc65b23ede7ce7694c63 is
actually only 4 pointers and on 32bit platforms already smaller than the union.
Replace it with a fixed 64byte padding. Don't add the normal padding for now.

Fixes #755822

gstreamer-uninstalled: add path for OpenCV haar cascade files

Some OpenCV plugins use haar cascade files that are included in the
GStreamer sources. To be able to use these from uninstalled, they need
to be found through an environment variable.

Adding this environment variable pointing to haar cascade files to
gst-uninstalled.

check: fix 'format string is not a string literal' warning with clang

Broke this when I removed the G_GNUC_PRINTF in a previous
commit to fix indentation, since it was not really needed.
Turns out unlike gcc clang warns though if a non-literal
format string is passed then. Fix indentation differently.

http://clang.llvm.org/docs/AttributeReference.html#format-gnu-format

tests: fix the manual tests by setting the right env-vars

directcontrolbinding: fix formatting

index: mark two structs that don't have abi padding

controller: add the missing abi padding

While this technically is an abi break, we decided to do this:
1) the struct is documented to be internal
2) the struct is alloced and freed inside the library
3) there are no public methods that receive or return instances
4) the only code known to use this struct are classes containd here