GitLab

so gnome-shell's application search does not
show zero result for an application that is
installed both by distro package and by flatpak.

Fixes issue gnome-shell#1013

tests: Fix closure-refcount to preserve old semantics

See merge request !699

The threads used to iterate at least 10000 times before setting the
"seen thread" flag to true. After porting they inadvertently did that
in the first iteration.

Move closures refcount test to gobject/tests/

See merge request !696

gthreadedresolver: Remove unused thread pool

See merge request !698

gsocketclient: Fix critical on cancellation

Closes #1693 and #1653

See merge request !688

Closes #1653

We need to be more explicit in handling cancellation to avoid
multiple task returns.

Fixes #1693

Previously, the test assumed that thread1 and thread2 would be scheduled
enough to set seen_thread{1,2} by the fact that the test runs for a high
number of iterations. On some platforms/schedulers, that’s not true,
which causes the test to spuriously fail.

Fix that by forcing the test to continue iterating until both threads
are seen. If this takes too long, the Meson test runner timeout will be
hit and the test will be terminated.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Previously, all three threads would access several global variables
without locking.

Fix that by using atomic accesses to data stored within the
test_closure_refcount() function, which also eliminates the global state
(which would confuse further tests if they were added to this file).
Signed-off-by: Philip Withnall <withnall@endlessm.com>

g_thread_create() has been deprecated for a long time.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

g_assert() can be compiled out if G_DISABLE_ASSERT is defined; and
g_assert_*() provide more specific error messages on failure.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Signed-off-by: Philip Withnall <withnall@endlessm.com>

These functions are not run more than once, so the variables don’t need
to be static to save state between runs.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Signed-off-by: Philip Withnall <withnall@endlessm.com>

This allows more structured test output.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

This allows more structured test running and output.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

One step towards removing the top-level tests/ directory.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Helps: #1434

Resolve "g_base64_encode(NULL, 0) causes critical warnings"

Closes #1698

See merge request !695

At that point in the code, len can only be 0, 1 or 2. The code below is
a no-op if (len == 0), so the condition is pointless.

Remove it, and we should be able to achieve full branch coverage of
gbase64.c.

This should introduce no functional changes.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

While I’m here, we might as well check that we output what the RFC says
we should output.

https://tools.ietf.org/html/rfc4648#section-10

(We do.)
Signed-off-by: Philip Withnall <withnall@endlessm.com>

len is unsigned, so it’s not possible for it to be less than zero.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Relax a precondition in g_base64_encode_step() to allow this. It’s valid
to base64 encode an empty string, as per RFC 4648.

Similarly for g_base64_decode(), although calling it with a NULL string
has never been allowed. Instead, clarify the case of calling it with an
empty string.

This includes a unit test.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Fixes: #1698

The caller needs to check this themselves in any case, so we might as
well at least follow convention in defining the precondition.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

gvariant-parser: Fix error handling when type coalescing fails

See merge request !639

Update gvdb submodule

See merge request !674

Fix size of preallocated array

See merge request gvdb!3

Commit 084e1d86 added a preallocation to an array to avoid
reallocations later on, but neglected the fact that after N insertions
into the array, there’s always a NULL terminator added to the end. Fix
the preallocation to include that NULL terminator.

This doesn’t change the correctness of the code, but should eliminate
one reallocation.

Spotted by Sebastian Dröge. See
!674.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

gthreadedresolver: Fix initialisation on FreeBSD

Closes #1697

See merge request !693

gtask: Separate GTask fields memory locations to avoid data races

See merge request !691

res_ninit() requires the __res_state struct passed to it to be
zero-filled on FreeBSD.

Spotted and analysed by Ashish SHUKLA.
Signed-off-by: Philip Withnall <withnall@endlessm.com>

Fixes #1697