GDBusWorker: as a precaution, access 'stopped' boolean atomically

This member is written in _g_dbus_worker_stop from arbitrary threads, and
read by the worker thread, so it should be accessed atomically.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=651268
Bug-NB: NB#271520
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

Regression test for closing a stream, slowly, while a message is queued

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=651268
Bug-NB: NB#271520
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusWorker: annotate more functions with lock/pending state

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=651268
Bug-NB: NB#271520
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusConnection: delegate to the worker to close the stream

We can't safely close the output part of the I/O stream until any
pending write or flush has been completed. In the worst case, this could
lead to an assertion failure in the worker (when the close wins the
race) or not closing the stream at all (when the write wins the race).

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=651268
Bug-NB: NB#271520
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusWorker: combine num_writes_pending with flush_pending

num_writes_pending was a counter, but it only took values 0 or 1, so make
it a boolean: it would never make sense to be trying to write out two
messages at the same time (they'd get interleaved).

Similarly, we can never be writing and flushing at the same time (that'd
mean we were flushing halfway through a message, which would be pointless)
so combine it with flush_pending too, calling the result output_pending.

Also assert that it takes the expected value whenever we change it,
and document the locking discipline used for it, including a subtle
case in write_message_in_idle_cb where it's not obvious at first glance
why we don't need the lock.

(Having the combined boolean at the top of the block of write-related
struct members improves struct packing on 64-bit platforms, by packing
read_num_ancillary_messages and output_pending into one word.)

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=651268
Bug-NB: NB#271520
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

Disable two GDBus tests

These tests try to use GMainContext across fork() which now fails a lot
more violently than it used to.

See https://bugzilla.gnome.org/show_bug.cgi?id=658999 for information.

Revert "Add g_memory_input_stream_new_from_data_full/add_data_full() apis"

I'm not sure that this API is right now that I try to use it.

This reverts commit 42738469d15585791bd5613356c47bcc2c413c20.

Add g_memory_input_stream_new_from_data_full/add_data_full() apis

Add an extra state pointer and an extra GDestroyNotify function
to the 'Chunk' definition... allowing bindings to attach some extra
state to memory chunks (to get memory management correctly from
language bindings).
Bug #589887

Fix docs

GApplication: don't create a mainloop

Just iterate the main context directly.  Also: avoid calling the virtual
functions for mainloops.

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

GApplication test: fix testcase to avoid fork()

The GApplication test case tried to fork() while using GMainLoop,
causing problems.  Avoid doing that by splitting the child process into
a separate program and spawning it in the usual way.

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

mainloop: detect fork() and abort

Abort if the child process returns to the mainloop after a fork().

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

GApplication: add chain-up checks

Make sure ::startup and ::shutdown are being chained up.

GApplication: introduce ::shutdown

Introduce ::shutdown as a dual to ::startup.

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

Updated Russian translation

gdbus-codegen: Fix segfault when remote property is invalidated

The 'key' variable is no longer valid outside the cycle, owned and
probably already freed by GVariant. This causes apps to segfault
when proxy is constructed and a property on remote d-bus service
changes (actually is invalidated). Looks like a typo anyway.

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

Updated Brazilian Portuguese translation.

Fix the description for g_app_info_get_icon()

It can return NULL when there is no default icon.

Updated Tamil Translations

Updated Tamil Translations

Merge remote-tracking branch 'gvdb/master'

hashing: always use signed chars

Our hashing of non-ASCII strings was undefined due to the fact that
'char' is signed on some platforms, unsigned on others.  Always use a
signed char.

Discovered by Alexander Larsson.

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

GDesktopAppInfo: Improve docs

Fix formatting in a doc comment

Expand mimeapps test

This adds a test involving both defaults.list and mimeapps.list.

Also check api results

In addition to looking at mimeapps.list, check that the default
and recommended apps returned by g_app_info api are as expected.

Revert "Fix default setting generation in g_desktop_app_info_set_as_last_used"

This reverts commit a13f5ca4da05c3a7edaed9a70c508f63772897cb.

Adapt tests to string changes

GType tutorial: clarify class struct initialisation

Make the information on how the class structure is initialised less
confusing to first-time readers.

Cosmetics

Make configure --help output a little more regular, by using
AC_HELP_STRING() more consistently.

Updated Spanish translation

GKeyFile: Unify error messsages a bit

This marks a few forgotten error messages for translation, and
makes some other strings more uniform. String change!

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

glib-compile-schemas: Pedantic string fixes

Replace 'can not' by 'cannot' in several places. String change!

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

Updated Hungarian translation

Fix default setting generation in g_desktop_app_info_set_as_last_used

 * Do not ignore the system default
 * Do not exclude the last used being set from the default list

This fixes the default applications dialog in control-center.

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

Add some tests for mimeapps.list handling

These tests directly look at mimeapps.list to verify that
we make the expected changes to the database.

Avoid some compiler warnings

Nix inaccurately named g_main_context_init_pipe()

...and fold its contents into g_main_context_new()

gmain: get rid of poll_waiting

This variable, which is the cause of much grief, exists for two reasons:

  - ensuring the the wakeup pipe doesn't fill up

  - preventing the first poll() after adding a source from waking up
    immediately

The first point is no longer an issue with GWakeup.

The second point is addressed by using different logic: we only signal a
wakeup in the case that the context is currently acquired by a thread
that is not us.

As an added bonus, we can now implement g_main_context_wakeup() without
taking a lock.

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

Add macros for GSourceFunc return values

The boolean values to be returned by a GSourceFunc are always ambiguous,
and even in case of experienced developers then can lead to confusion.

The Perl bindings for GLib have two simple constants, mapping to TRUE
and FALSE, that make the return values less confusing: G_SOURCE_CONTINUE
and G_SOURCE_REMOVE respectively.

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

Keep up with the standards

More choices, always better.

GRegex: fix thread-unsafe initialiser code

Clean up l10n threading stuff

Remove the explicit thread initialisation functions for g_get_charset(),
g_get_filename_charsets() and g_get_language_names().

Add a lock around one remaining case of access to libcharset (the other
2 cases already have the lock).

Do a proper g_once_init_enter() style initialisation for the GLib
gettext functions.

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

glib-private: #include <glib.h>

This is really our only choice if we don't want to annoy the
single-include checking when we use this from inside of gio.

Drop long-removed atomic thread init from header

Remove now-useless glibprivate.h

The only symbol in glibprivate.h was moved to glib-private.h.

glib worker: move to glib-private framework

Remove the private glib_get_worker_context() symbol and move it over to
using the glib-private stuff like GWakeup is doing.

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

gwakeup.h: add missing header for GPollFD

Add glib__private__() API to share between glib,gio; port GWakeup to it

Historically we've added random symbols to the public API with warnings
that they're private; examples are:

glib_gettext(), glib_pgettext()
g_thread_functions_for_glib_use, g_thread_use_default_impl, etc.

And we almost added "GWakeup" to public API just to share between glib and
gio.

This new glib__private__() API exports a hidden vtable, and adds a macro
GLIB_PRIVATE_CALL() that makes it generally convenient to use.

This adds an extremely tiny cost for the double indirection; but it has
the benefit that we don't need to either:

1) compile the code into both glib and gio (like GWakeup), with the
   inefficiency that implies.
2) Export a "do not use this" symbol; the serious problem with this is
   that someone CAN use it pretty easily.  Particularly if we document
   it.  It's far, far harder to peek into a structure without a public
   header file.

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

GMainLoop: remove wall clock time cache

Since GMainLoop is now purely monotonic, it really doesn't make sense to
cache the wallclock time just for the sake of making a deprecated call
more efficient.

gtester: Remove SIGCHLD race workarounds

With the GMainLoop changes, gtester should now be race-free.

Modify child and signal sources to use worker

GMain: simplify logic for g_wakeup_acknowledge()

Instead of messing around with context->poll_waiting, just look at the
GPollFD to see if the GWakeup needs to be acknowledged.

GMainLoop: remove single-threaded case

Since we now always have thread support in libglib, we can remove the
buggy single-threaded codepath for GMainContext.

Add private glib_get_worker_context() API

The first time this is called, this creates a GMainContext * and a
thread to run it.  Future calls return the same.  There are a lot of
places that we could use this in GLib.

GRand: Make sure to g_thread_init_glib()

The last commit should g_thread_init_glib() before attempting to use
GOnce.

GRand: remove setup from g_thread_init_glib

Use g_once_init_enter instead.

emufutex: remove init from g_thread_init_glib

Use a GStaticMutex instead.

win32: remove version init from g_thread_init_glib

Instead, make this use g_once_init_enter() in the usual way.

make g_thread_init_glib() idempotent

Move the GThread implementations to glib/

We can now get threads initialised from inside of libglib by calling
g_thread_init_glib().

invert gthread-impl includes

configure.ac defined G_THREAD_SOURCE and gthread-impl would #include it.

Instead, since we only have two thread implementations now, and since we
always use the Windows one only on Windows, move the logic to the
Makefile, predicated on 'if OS_WIN32'.  Then have the chosen backend do
the #include "gthread-impl.c" from there.

Remove the G_THREAD_SOURCE define from configure.ac.

win32 threads: assume TryEnterCriticalSection

Remove the dynamic lookup for TryEnterCriticalSection, which has existed
since Windows 2000.

Stop using HAVE_G_THREAD_IMPL_INIT

Just always call the init function.

Move thread priority translation into the backends

The translation of GLib priorities into the thread priorities of
different operating systems belongs in the implementation -- not
half-way in the front end.

Remove useless macros from when we supported DCE

These are non-ops now, and easily replaced.

Remove support for DCE threads

Remove support for custom thread implementations

Remove the dead 'none' thread implementation

GThread: remove errorcheck mutex support

This can only possibly work if we call g_thread_init(), which we are
moving away from.

gthread: remove some unused forward declarations

Deprecated (undocumented) g_thread_gettime

g_thread_gettime() is an undocumented public function pointer that
points to a function that returns the monotonic time in nanoseconds.
g_get_monotonic_time() does the same in microseconds, so it can be used
instead.

GLib had one internal user in GFileMonitor that only cared about
millisecond accuracy; it has been ported to g_get_monotonic_time().

Update building docs to mention mandatory threads

The docs used to say thread support was optional -- it's mandatory now.

Remove !g_thread_supported() codepaths in gio

In particular, remove the libasyncns import, which was only used by
GUnixResolver, which is only used when threads are not available.
Likewise remove GWin32Resolver, and the hacky broken non-threaded
parts of GIOScheduler.

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

Revert "GIOSchedulerJob: Improve struct packing"

This reverts commit a2094d5e564433ed3a4398afeacce80519bbda44.

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

Make threads mandatory

G_THREADS_ENABLED still exists, but is always defined. It is still
possible to use libglib without threads, but gobject (and everything
above it) is now guaranteed to be using threads (as, in fact, it was
before, since it was accidentally impossible to compile with
--disable-threads).

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

Fix spelling of G_IO_FLAG_IS_WRITEABLE

Fixes #657045.

g_key_file_parse_value_as_integer: Integers can have trailing whitespaces or tabs

Now it can parse integer values with trailing whitespaces or tabs before CR.
Using g_ascii_isspace() as suggested.

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

Updated Polish translation

Updated Polish translation

simpleaction: Fix documentation of :enabled

An over-zealous sed, most likely, transformed g_action_.*() into
g_simple_action.*().

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

Update French translation

bgo#640212 - Replace "error stating file" with friendlier messages

Non-technical users won't know that "stating" refers to stat(2), so we
just use "error when getting information" now.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Updated British English translation

Updated Italian translation

Updated Belarusian translation.

Added Norwegian bokmål translation

GSettingsBackend: emit changes to correct thread

When g_settings_apply() is called on a delayed settings backend and
there is a D-Bus error when communicating with dconf-service, recent
versions of the dconf GSettingsBackend call a function in GLib that
improperly delivered the signal directly instead of using
g_main_context_invoke().

This patch fixes this function to route in the same way as the others so
that the signal is dispatched in the proper GMainContext.

'master' is now glib 2.31.0

glib 2.29.90

Updated Indonesian translation

gio docs: lots more fixes

gio docs: only include GTlsInteraction once

It was included twice in the gio-docs.xml file.

gio docs: drop GTimeZoneMonitor from docs

Remove enums from gio.types

Only objects and interfaces should go in here.  If enums are in here
then gtk-doc responds by outputting two anchors for the same name (which
results in many warnings being printed).

GDBus object manager example: update -sections.txt

gio docs: ignore private .h files

gio docs: don't scan subdirectories

This clears up >50% of the huge warning spew from building GIO docs.

docs fixups for glib/

GObject docs: resolve broken links

Some links were broken due to typos, because functionality was removed
in GLib 2.0 or for various other reasons.  Fix up as many of them as is
reasonable.