Release 1.13.91

validate: GST_EXPORT -> GST_VALIDATE_API

We need different export decorators for the different libs.
For now no actual change though, just rename before the release,
and add prelude headers to define the new decorator to GST_EXPORT.

debug-viewer: solved crash when maximum freq sentinel is 0

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

debug-viewer: Fix copying current line

debug-viewer: Add an appdata file

Release 1.13.90

meson: enable more warnings

debug-viewer: fix names of actions/functions

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

debug-viewer: Added filter for threads

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

debug-viewer: Added filter for function

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

meson: simplify GST_DISABLE_GST_DEBUG check and don't use add_global_*

add_global_arguments() can't be used in subprojects. It's
entirely possible that devtools is a subproject but gstreamer
is picked up from an installed location, so we should
really use add_project_arguments() in both cases.

validate: tools: fix build dependencies for validate-rtsp-server

In file included from ../subprojects/gst-devtools/validate/tools/gst-validate-rtsp-server.c:21:0:
.../gst/gst.h:31:10: fatal error: gst/gstenumtypes.h: No such file or directory

validategtk: Stop using deprecated keymap API

gstvalidategtk.c:184:7: error: ‘gdk_keymap_get_default’ is deprecated: Use 'gdk_keymap_get_for_display' instead [-Werror=deprecated-declarations]
       gdk_keymap_get_entries_for_keyval (gdk_keymap_get_default (),
       ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

debug-viewer: Fixed C++ destructors detection

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

Fix versions

Back to development

Release 1.13.1

validate: dist enum types templates

debug-viewer: Fix C++ detection of lambdas as function

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

validate-pad-monitor: Use GST_SEQNUM_INVALID

Instead of 0 (which is valid)

debug-viewer; Store thread as long instead of int

On 64-bit platforms, the thread id can be over 2^32 so use a long
to handle it.

tools: gst-validate-images-check: Fix typo

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

meson: use -fno-strict-aliasing where supported

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

validate/baseclasses: Release un-needed data when test ends

This was keeping around 500-700kB of data for each test, which was
gradually raising memory usage of a full run by 100MB+

The reports are definitely not needed, and we only need to keep
information from the subprocess env variable that we might need
later on for final reporting

validate/baseclasses: Don't leak several hundred MB of XML

The xml-based MediaDescriptor were keeping open the XML file and the
associated ElementTree structures, resulting in memory usage of several
hundred megabytes.

Instead cache the information we need immediately and release the
XML structure

meson: Add missing force_rtsp2 scenario

Without this file, gst-validate installed using meson will fail all
RTSP2 tests

validate: Remove protocol-specific timeouts

Since we now check position/status of pipeline at regular intevals,
we no longer need to impose a different timeout based on the
protocol used.

Avoids having 4min long timeouts for no reason (30s is enough)

validate: Use a single TCPServer for subprocess communication

Instead of creating a separate TCPServer for each test, just create
one which handles all connections in a threaded fashion.

Shaves off ~500ms per test

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

validate-launcher: Allow running tests out-of-order

When the --shuffle option is used, the tests will be run out of order.
This optimizes CPU utilization since it allows running synchronized
and unsynchronized tests at the same.

validate: Reduce time waiting for subprocess to stop

stopping the subprocess is done from the main thread, this would
throttle starting/stopping any tests by one second.
Start with 50ms, and gradually increase the wait between iterations

gstvalidate: Lower timeout to check for rtsp-server to be up

Check every 100ms, avoids throttling all rtsp tests by 500ms

validate:launcher: Launch tests in `_TestsLauncher` not in TestsManagaer

So that Test from several TestManager can run in parallel and thus avoid
waiting for tests from one TestManager to run the following one.,

Also by design TestsLauncher should always have been the responsible for
... launching tests.

validate: Don't leak strings

We only use them in the error/debug case anyway

validate-report: Plug leaks

The trace was never freed, nor were the output of g_str_split

validate-scenario: Don't assume element have factories

Some elements might not originate from factories (like custom/internal
elements).

Avoids dereferencing a NULL pointer

validate-scenario: Handle non-relative switch

Make sure we stay within the number of present streams (and avoid
out-of-bound read).

CID #1415470

validate-scenario: Handle switching stream of type not present

Unlikely to happen, but at least don't end up doing unsafe calculation
with n == 0 afterwards

CID #1415453

meson: Always require the latest gst-rtsp-server

In the worst case, when building with gst-uninstalled, we will try to
link against an older gst-rtsp-server provided by the system. Found by
philn.

validate: Call g-ir-scanner with the same toolchain as the rest

validate-analyze: Update for xml format changes

validate: launcher: Run rtsp tests against both V1 and V2

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

validate:launcher: Allow disabling using the number of failed tests as exitcode

This is usefull on CI servers where the test results will be inspected
and the status of the build built from it.

validate: launcher: Modify the order of the parser argument

Modify the order of the parser argument before setting dir_group

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

launcher: Automatically disable output coloration if not supported

meson: Fix the way we set the testsuite version

validate:launcher: Use the number of failed test as exit code

We used to always return 0, which was not right!

launcher: Avoid exceptions when inspecting renders files

We were a bit to strict on the Exception types which lead to
the launcher failling itself when it shouldn't

meson: hide symbols by default unless explicitly exported

validate: hide some private symbols

validate: Plug a potential leak when retrieving peer pad

validate: Fix going over ghostpads/proxypads

meson: fix a few meson warnings

WARNING: The variable(s) 'DATADIR', 'LIBDIR' in the input file
  'subprojects/gst-devtools/validate/launcher/config.py.in' are not
  present in the given configuration data
WARNING: Passed invalid keyword argument "scanobj_args". This will
  become a hard error in the future.
WARNING: Keyword argument "install" defined multiple times. This
  will be a an error in future Meson releases.

validate:win32: Update .def file.

validate: Mark symbols explicitly for export with GST_EXPORT

With an exception:
  * gst_validate_monitor_setup

which was never declared in headers and should always have been static.

validate: Fix building the tracer

It fails on some platforms, I guess this is the reason

validate: Also mkenums with autotools

And fix the build with stricter gcc arguments.

validate: Add a way to print information about pipeline status

Similare to what is done with gst-launch.

And finally generate GTypes for our flags and enums.

validate-scenario: Fix NULL pointer usage

for good this time ...

CID #1415570

validate-report: Fix a leak in error cases

CID #1415494

validate: Re-enable mxf op2b tests

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

wind32: Update .def file.

validate-scenario: Protect against priv NULL usage

CID #1415570

validate: Do not install now removed setup_sink_props_max_lateness.scenario file

validate:launcher:check: Make sure to register tests from the testsuite

Instead of having them listed from the app manager. This is needed
to avoid backtrace as tests now have to be register when setting up
the testsuite.

validate: Factor out a method to set properties on elements in utils

Make sure to use it where appropriate and add some logging when
setting an object property from an action.

And use the valgrind.conf to set all the properties instead of having
a mixture of a config scenario and the config file (making sure the
max-lateness is set on any sink)

validate:scenario: Allow not config action to be executed from config files

When those are special cased to support that, such as the `set-property`
action.

This special handling was added in

  4927c657107dd23405456a703bb23173ab60f27d
  validate: disable QOS features when running with valgrind

before we started to support executing arbitrary config action from
configuration files.

validate: scenario: Fix running config action from the config file

validate: Blacklist op2b mxf files

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

validate: Cast GList data content before usage

Apart from code readability, it allows compilers to detect wrong usages,
such as the call to gst_validate_action_new() which was using the wrong
argument

validate:launcher: Error out if no testsuite could be loaded

validate: launcher: Namespace test name with the testsuite name

Also allowing users to pass test names directly

validate: launcher: Properly use TestsLauncher.list_test to load tests

Otherwise we might skip check_defined_tests.

validate: Remove some dead code

CID 1415457

validate: Plug a minor string leak

CID 1415459

validate: Don't create scenario on a monitor which has no target

Not very probable but avoids a potential NULL pointer dereferencing.

CID 1415460

validate: Always only use the first description in scenarios

Also pluging a leak of the descrption copied structure

CID 1415463

validate: ssim: Do not compare unsigned to < 0

CID 1415473

validate: Plug a string leak

CID 1415475

validate:ssim: Let user know when no file have been compared

Fixing a possible division by zero issue.

CID 1415482

validate: Add missing break statement

CID 1415485

Check g_file_set_contents() return value

CID 1415486

validate: Plug leak of copy of a va_list

CID 1415490

validate: Do not check NULL pointer uselessly

CID 141593

validate: Plug minor leak in issue creation error path

CID 1415494

validate: Remove some dead code

Next will never be NULL as `done` is always set to TRUE when next is
set.

CID 1415503

validate: monitor: Add missing break; statement

Fixes CID 1415500

validate-scenario: Fix wrong return value

We were always returning ok ...

CID #1415484

validate-scenario: Check g_file_set_contents() return value

CID #1415487

validate-scenario: Fix copy/paste error

CID #1415502

validate-scenario: Fix string usage

Use the string representation of the index if it *IS* present (and
not the opposite).

CID #1415506

validate-utils: Add NULL check

Unlikely to be triggered. And fix typo at the same time

CID #1415464

validate: Un-blacklist scrub_forward_seeking.op2b-mpeg2-wave_hd_mxf

It works now after various mxfdemux changes.

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

gstvalidate: Re-allow tests that should be fixed

check: Remove dead assignments

validate: launcher: Add some missing env variables in command to launch test

meson: find python3 via python3 module

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

validate: Recalculate latency on LATENCY messages

validate:scenario: Allow setting properties by element factory name

validate:scenario: Enhance the set_property action to handle enum props

User needs to specify the enum value as a string, to be used
as with gst_util_set_object_arg.

Also enhance reporting and verify that the set value has actually
been taken into account.

validate: Use Gst printing utils in our reporting system

Allowing us to use GST_PTR_FORMAT and friends!