Add a few more tests

This brings test coverage for glist.c and glist.c to the
coveted 100% lines mark.

Remove bits of dead code identified by coverage tests

These lines could were not hit by our tests, and examination
of the code reveals that they can't ever be hit.

Add some more thread tests

GAsyncQueue: properly set free function

The copying of code from g_async_queue_new() to g_async_queue_new_full()
in ef08aa786bca87c520ef319b97df4b3ed0782233 copied the setting of the
free function to NULL (instead of the one passed in by the user).

Fix that up so that the test passes again.

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

GUnixMountPoint: expose options

Make the options from an /etc/fstab entry available as public API -
this can be used to support options such as

 comment=gvfs.name=Foo\040Bar

to e.g. set the name of an fstab mount in the UI to "Foo Bar".

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

Signed-off-by: David Zeuthen <davidz@redhat.com>

Revert "GSettings: don't abort on missing schemas"

This reverts commit c841c2ce3fda6f754c88ae2c9099f36dff2f0814.

This approach has been an unmitigated disaster.  We're getting all sorts
of crashes due to functions that are returning NULL because they can't
find the schema for the default value.  The people who get these crashes
are then confused about the root cause of the problem and waste a lot of
time trying to figure it out.

Until we find a better solution, we should go back to what we had
before.

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

Cosmetics

Documentation fixes

Fix doc build

The markup here was not only broken, it was also unnecessary,
since gtk-doc knows to apply <function></function> tags to things
that end with () already.

locks: Add initialisation notes

For each of the 4, mention that static storage or _init() is needed.

locks: change the ABI just a bit

Add a little bit more room in the ABI for our synchronisation primatives
since we're going to need it when we add native implementations on
Linux.

Also: rename the pointer field and add /*< private >*/ annotations.

locks: drop _INIT macros

All locks are now zero-initialised, so we can drop the G_*_INIT macros
for them.

Adjust various users around GLib accordingly and change the docs.

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

Re-enable 'include' testcase

The bug is fixed now.

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

GThread posix: switch to Windows ABI

Modify the POSIX implementation of the synchronisation primatives to use
the same ABI as Windows: one pointer for each type.

This frees us from having to #include <pthread.h> and avoids the problem
with pthread_rwlock_t not being defined under certain compiler defines.

A few more changes are expected to the ABI -- they will be committed
separately.

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

Don't use g_thread_foreach in tests

Deprecate GStaticPrivate and g_thread_foreach

This commit moves GStaticPrivate, g_thread_foreach and all
related functions and variables to gthread-deprecated.c. We
introduce some internal API to make this possible.

g_thread_foreach is not a very useful function, since there is
virtually nothing you can do with a GThread*, and implementing
it requires us to keep a list of threads around.

GStaticPrivate has been made redundant by adding comparable
capabilities to GPrivate.

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

Don't put threads created with g_thread_new() on the list

This lets us avoid the overhead of maintaining the global
list in the non-deprecated case.

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

Make thread names useful in a debugger

Associate the name with the system thread, so that debuggers
can see it. This is currently only implemented for Linux, using
prctl.

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

Add new thread creation API

Deprecate both g_thread_create functions and add
g_thread_new() and g_thread_new_full(). The new functions
expect a name for the thread.

Change GThreadPool, GMainContext and GDBus to create named threads.

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

Define GLIB_COMPILATION when building libgthread.so

This is necessary, because gthreadprivate.h can only be
included inside GLib, otherwise single-include guards trigger.

Replace static privates by privates

GStaticPrivate is heading for deprecation soon, and GPrivate
can replace these uses now.

win32: fix GPrivate fallout

Fix minor mistake in win32 GPrivate code.

locks: rename a bunch of parameters

Stop dithering over GPrivate

Take out the half-private g_private_init() stuff and replace it with a
G_PRIVATE_INIT macro that allows specifying a GDestroyNotify.

Expose the GPrivate structure in a public header.

Add a g_private_replace() to (sort of) match the functionality of
g_static_mutex_set().

Improve the documentation.

Deprecate g_private_new().

Add a few comments

Add another GCond test

This test shows how to implement a barrier using a GCond.

Rework the way GStaticPrivate data is freed

To avoid iterating threads in g_static_private_free(), defer freeing
the per-thread data to thread exit. The one complication here is
that it is possible for the static private index to be reused while
'old' data is still around. To deal with that case, store the 'owner'
with each per-thread data node, and free old data in
g_static_private_get() if the owner doesn't match. The remaining
possibility that a private index could be reused by a GStaticPrivate
with the same address is sufficiently unlikely that we can probably
ignore it.

With this change, per-thread data is now truly private again,
and we can drop the lock for it as well.

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

Whitespace fixes

Clean up includes

Add tests for GPrivate and GStaticPrivate

tests: Use G_VALUE_INIT

docs: Use G_VALUE_INIT

mutex testcase: add a performance test

Add a reasonable performance test for uncontended and contended cases.

Move all hash functions to ghash.c

This matches their location in the headers.

GHash: Cosmetic changes

One forgotten moved doc

Move GStringChunk into its own files

Move GString docs inline

GString: cosmetic cleanups

Move file utility docs inline

Move string utility docs inline

Move test docs inline

Move keyfile docs inline

GAsyncQueue: simplify an internal function

g_cond_timed_wait() behaves like g_cond_wait() when given
NULL, so no need have different branches for that in
g_async_queue_pop_intern_unlocked().

GAsyncQueue: internal cleanup

Turn the 'try' parameter of g_async_queue_pop_intern_unlocked
into a 'wait', for better alignment with the GCond api.

GAsyncQueue: embed the GCond

Use g_cond_init/clear, now that we have them.

GAsyncQueue: Cosmetic fixes

Mostly doc formatting and whitespace fixes.

GAsyncQueue: Move private API to a private header

GThreadPool: cosmetic cleanups

Mostly documentation and formatting trivial, but also add
boolean return types to GError taking functions.

GThread: cosmetic fix

Explicitly include gthread.h, don't rely on gthreadprivate.h
to pull it in.

Updated Vietnamese translation

po/vi: import from Damned Lies

Add an index for 2.32 api additions

Mark g_thread_create_with_stack_size as new API

GSettings schemas: allow for zero items in schema

GVDB deals with empty lists by returning NULL for the list instead of a
zero-length (non-NULL) strv.  We can work around that in GSettingsSchema
by checking for the NULL case and treating it like a zero-length list.

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

Fix URL to strlcpy(3) manpage in comment.

markup-subparser test: use a real GError domain

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Matthias Clasen <mclasen@redhat.com>
Bug: https://bugzilla.gnome.org/show_bug.cgi?id=660371

sleepy-stream test: use a real GError domain

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Matthias Clasen <mclasen@redhat.com>
Bug: https://bugzilla.gnome.org/show_bug.cgi?id=660371

g_dbus_error_encode_gerror: don't segfault on bad domains

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Matthias Clasen <mclasen@redhat.com>
Bug: https://bugzilla.gnome.org/show_bug.cgi?id=660371

Prevent data loss in gdesktopappinfo.c

Don't loose the content of mimeapps.list when it doesn't contain
a [Added Associations] group. Pointed out by Alexander Larsson.

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

Clean up atomic cruft

Nothing is using these defines anymore, and the messages
are misleading. Based on a patch by Kean Johnston.

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

Some more gthread reshuffling

GUnixFileMonitor: Clean up /proc/mounts monitoring

Most suggestions from Colin Walters <walters@verbum.org>.

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

Signed-off-by: David Zeuthen <davidz@redhat.com>

GUnixFileMonitor: Use /proc/mounts for monitoring mount changes

On recent Linux distros /etc/mtab is just a symlink to /proc/mounts
and GFileMonitor does not work there because of how the kernel conveys
that the file changes.

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

Signed-off-by: David Zeuthen <davidz@redhat.com>

gfile: Be clear about freeing GFile objects

gthread: remove a stray reference to GStaticMutex

Updated Esperanto translation

update .gitignores

tests: add signal marshalling return tests for int/uint

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

tests: fix generic closure marshalling tests

We're testing enums here, not flags, so we need to make sure
we're trying to send enums through the marshalling.

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

tests: Add some return value torture testcases for the generic marshaller

Ensure that ENUM types are properly marshalled back from libffi types
to glib types, which was failing on ppc64.

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

Add thread-safety warnings to the g_setenv() and g_unsetenv() docs

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

GDesktopAppInfo: simplify how defaults work

Previously, we took the default application for a particular mimetype
from the system and copied it into the user's configuration as the
default there.

Instead of doing that we leave the user's default unset, and at time of
use, if the user has no explicitly-set default value, we use the system
default.

This avoids complicated situations where inappropriate applications were
being set as the default in the user's configuration.

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

Add another mimeapps test

This tests the interaction between mimeinfo.cache, defaults.list and
mimeapps.list to ensure g_app_info_set_as_last_used_for_type doesn't
incorrectly change the default.

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

g_simple_async_result_complete_in_idle: add a minor doc clarification

glib tests: Compile new thread tests on win32

They accidentally got added inside of a 'if OS_UNIX'.  Move them out.

winxp threads: detect SRWLock emulation reentrancy

We lack SRWLock on Windows XP, so we use CRITICAL_SECTION to emulate it
there.  SRWLock is non-recursive, but CRITICAL_SECTION will happily
allow itself to be acquired multiple times by the same thread.

We need to detect if our second acquire succeeded because of the
recursive nature of CRITICAL_SECTION.  In the case of a _lock()
operation, we would normally have deadlocked, so abort.  In the case of
a _trylock() operation, we need to ensure that FALSE is properly
returned.

Problem caught by Matthias Clasen and Chun-wei Fan.

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

[l10n]Updated Catalan (Valencian) translation

Update Czech translation

[l10n]Updated Catalan translation

gvariant: Add g_variant_get_fixed_array()

Using g_variant_new_from_data() for creating new byte arrays is non-obvious.
This patch adds a g_variant_new_fixed_array() function.

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

More GThread docs tweaks

Move GThread docs around

Move includes to the top of the file

Move deprecated GThread functions to a separate file

GThread: more doc updates

Drop G_THREADS_IMPL_NONE from the docs

GLib can no longer be built without thread support.

Fix warning/error C4013 (implicit declarations)

-Include gthread.h in gregex.c as g_once_init_enter and g_once_init_leave
 are used.
-Define prototype for g_thread_DllMain in gthreadprivate.h for Windows

gthread.c: Declare variable at start of block

This is so that C89 compilers will not complain.

Document G_THREADS_IMPL_WIN32

GThread doc additions

Fix an uninialized variable warning

Updated Slovak translation

Updated Danish translation

Add a testcase for bug 659866

Mere inclusion of glib headers should not require you to define
any XYZ_SOURCE macros.

Add a test for GCond

Add a nontrivial read-write lock test

Add GOnce tests

Make the rec mutex test a little more verbose

giostream: "closed" property should not be marked writable

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