more NEWS

The usual docs unbreaking...

configure.ac: join the cargo-cult of _cv_

For some reason, the setting of g_atomic_lock_free wasn't making it down
to the lower part of the configure script where glibconfig.h was being
generated when building using mingw32-configure.

If we prefix glib_cv_ to the start of the variable name (like everyone
else is doing) then it magically starts working.

I love you, automake.

gdbus-non-socket test: avoid use of a GMainContext across a fork

See https://bugzilla.gnome.org/show_bug.cgi?id=658999 for why this would
be bad.

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

gdbus-exit-on-close test: cover more possibilities

We didn't previously test anything except the implicit default of TRUE.
Now we test implicit TRUE, explicit TRUE, explicit FALSE, and
disconnecting at the local end (which regressed while fixing Bug #651268).

Also avoid some questionable use of a main context, which fell foul of
Bug #658999 and caused this test to be disabled in master.

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

GDBusWorker: if a read was cancelled it means we closed the connection

This was a regression caused by my previous work on GDBusWorker thread-safety
(Bug #651268). The symptom is that if you disconnect a GDBusConnection
locally, the default implementation of GDBusConnection::closed
terminates your process, even though it shouldn't do that for
locally-closed connections; this is because GDBusWorker didn't think a
cancelled read was a local close.

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

Revert "Disable two GDBus tests"

This reverts commit 05ef173466e32d8b3d212803e4a72239913a362d.

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

Fix bug in g_static_rec_mutex_unlock_full()

pthreads doesn't implement the _lock_full() and _unlock_full() calls on
recursive mutexes so we don't have it on GRecMutex either.  Now that
we're using GRecMutex to implement GStaticRecMutex, we have to fake it
by keeping an internal counter of the number of locks and calling
g_rec_mutex_unlock() the appropriate number of times.

The code to do this looked like:

  depth = mutex->depth;
  while (mutex->depth--)
    g_rec_mutex_unlock (rm);
  return depth;

which unfortunately did one last decrement after mutex->depth was
already zero (leaving it equal to -1).

When locked the next time, the count would then increase from -1 to 0
and then the next _unlock_full() call would not do any calls to
g_rec_mutex_unlock(), leading to a deadlock.

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

gatomic: introduce G_ATOMIC_LOCK_FREE

We clean up the detection of if we should do 'real' atomic operations or
mutex-emulated ones with the introduction of a new (public) macro:
G_ATOMIC_LOCK_FREE.  If defined, our atomic operations are guaranteed to
be done in hardware.

We need to use __GCC_HAVE_SYNC_COMPARE_AND_SWAP_4 to determine if our
compiler supports GCC-style atomic operations from the gatomic.h header
because we might be building a program against GLib using a different
set of compiler options (or a different compiler) than was used to build
GLib itself.

Unfortunately, this macro is not available on clang, so it has currently
regressed to using the mutex emulation.  A bug about that has been
opened here:

  http://llvm.org/bugs/show_bug.cgi?id=11174

gatomic: cast to the correct pointer sign

In the fallback case of the g_atomic_int_ macros we had some (gint*)
casts that should have been (guint*) casts.

[l10n] Updated German translation

gio: Introduce get_sort_key() methods on GDrive, GVolume and GMount

This is needed to implement efficient and predictable proxy volume
monitors, see

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

for details.

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

tests/gobject: Use G_VALUE_INIT

gobject: Use G_VALUE_INIT

gio: Use G_VALUE_INIT

GDBusConnection: Fix race in /gdbus/connection/life-cycle

GDBusConnection sets the closed flag in the worker thread, then adds an
idle callback (which refs the Connection) to signal this in the main
thread. The tests session_bus_down doesn't spin the mainloop, so the
"closed" signal will always fire if iterating the mainloop later (and
drops the ref when doing so). But _is_closed can return TRUE even before
signalling this, in which case the "closed" signal isn't fired and the
ref isn't dropped, causing the test to fail.

Instead simply always wait for the closed signal, which is a good thing
to check anyway and ensures the ref is closed.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=661896
Reviewed-by: Matthias Clasen <mclasen@redhat.com>

Actually Don't define GLIB_COMPILATION in libgthread

Last commit was wrong, fixing it up

Don't define GLIB_COMPILATION in libgthread

This was turning all the GLIB_VARs in the glib headers into
dllexports on windows, causing all sort of nastiness. libgthread is
mostly empty now anyway, so we don't need any GLIB_COMPILATION like
flag.

Fix an include guard mismatch

Spotted by Colin Walters.

Fix up testcase broken due to g_thread_new API change

Move _glib_get_locale_dir to ggettext.c

-Move _glib_get_locale_dir to ggettext.c, as Matthias suggested
-Made up for the headers that needed to be included in ggettext.c to avoid
 C4013 (implicit declaration of ...) errors/warnings

Updated POTFILES.in

Fix gio.symbols

g_simple_action_get_parameter_type is now static, so this symbol needs to
be removed from here...

GLib: Add forgotten includes and build fixes

-gcharset.c, genviron.c, gunicollate.c: Some headers were missed in those
 files that triggered C4013 warnings/errors (aka. implicit declaration
 of ... in GCC).  Make up for them here.
-gwin32.h: Only define g_win32_get_package_installation_directory/
 g_win32_get_package_installation_subdirectory as macros
 (alias of g_win32_get_package_installation_directory_utf8/
 g_win32_get_package_installation_subdirectory_utf8) on Win64 (x64) as
 g_win32_get_package_installation_directory/
 g_win32_get_package_installation_subdirectory have seperate existing
 implmentations for Win32-this is a long-standing problem but was covered-
 up by G_DISABLE_DEPRECATED, which we are stopping to use as of 2.31.0.

Revert "Bug 652827 - glib-2.29.8 no longer builds with mingw.org's toolchain"

This reverts commit 349212211247781e7bec43b95cb57dddc1d0e5b1.

Update VS property sheets

Some new headers were added...

glib/tests/thread.c: Add config guards

Not all systems come with unistd.h and sys/time.h, so use config guards
on them so that they are only included when available.

Update VS property sheets

gcache.h is now a deprecated header, so up the "install" part for it.

More include cleanups

Update VS property sheets

Stop using G_DISABLE_DEPRECATED as there are now warnings for usage of
deprecated APIs/items in GLib

include cleanup

Make things compile again

Move gettext variations into their own files

Reshuffle some functions between gutils and gfileutils

Move filename-related functions to gfileutils, and move
size formatting functions to gutils.

include cleanup

Trivial

Reshuffle genviron.c

Split win32 functions off into their own section, instead
of having large and unwieldy ifdef sections inside each function.

Also move the compat versions of env functions over from gutils.c

push G_THREADS_MANDATORY over the cliff

This was used as an optimisation for the macro hackery that used to live
in gthread.h.  If a particular library or program knew that it could
rely on thread support being enabled, it would allow for static
evaluation of conditionals in some of those macros.

Since the macros are dead and thread support is now always-on, we can
get rid of this bit of legacy.

gobject/gio: some last missed statics/inclues

gthread/: fix up declarations

g_thread_init() is now a deprecated API, so drop G_DISABLE_DEPRECATED
from the CFLAGS for gthread/.  Add the missing declaration for
g_thread_init_with_errorcheck_mutexes() back to deprecated/gthread.h.

Add private prototype for g_thread_init_glib()

This function was never put in a header and was only used internally
from libgthread, but we should keep the symbol around to avoid
triggering any false-positives on ABI checkers.

For -Wmissing-prototypes compatibility with this unusual case, we should
add a private prototype in the .c file just before.

gio static fixups

static and #include fixups in glib/

genviron: #include fixups

more 'static' adding in testcases

No dead code found this time...

contexts test: don't use deprecated GCond wait API

Mark some functions 'static' in glib/tests

This has uncovered two unused testcases in option-context.c.  They are
currently broken and require more investigation (which is probably why
they are unused).

Updated Japanese translation

README: Add note about gobject->gthread situation

gobject.pc: remove dependency on gthread

GObject no longer uses gthread, so libgthread should no longer be
included in the linker flags of programs built against GObject.

drop -uninstalled.pc files

These were meant to provide the ability to use GLib directly out of its
build tree but have a couple of rather serious limitations that prevent
their serious use:

 - these files only worked with programs built using libtool

 - for non-trivial uses, other environment variables will need to be
   setup to allow GLib to function properly

 - the system doesn't stack well when attempting to use multiple
   libraries all uninstalled

The jhbuild --prefix= style system works much better in every way, so we
remove this old hack.

gitignore

gthread: move test cases to glib/

tests: merge gthread/ and glib/ atomic tests

glib/tests/atomic: convert to proper gtester test

Also clean up includes

Move charset and locale name functions to their own files

They did not really belong into either gutils or gutf8.

Add deprecation annotations to mem chunk apis

Move version APIs out of gutils

Just another cleanup.

Move GTrashStack out of gutils.[hc]

Reducing the mess in gutils, and moving docs inline
at the same time. Double win.

Pass correct data to fatal log handlers

This was pointed out by John Ralls in bug 653052.
He also provided the test case.

Clarify g_spawn_*() behaviour without full path

Document the previously uncovered case of calling g_spawn_async_with_pipes()
without a full path but no G_SPAWN_SEARCH_PATH. This runs programs from the
current directory, which might be unexpected and even dangerous in some corner
cases.

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

Add a tests of some GVfs functions

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

Updated Japanese translation

Updated Japanese translation

Add app lauch environment test

Add g_get_codeset test

Add GAsyncQueue unit test

Move g_get_codeset next to g_get_charset

g_get_codeset is a close relatove to g_get_charset, and up to now
it lived a shadowy existence without any header presence.

Include genviron.h where necessary

And fix the syntax, too

Forgotten file

Drop unused  copy of _g_getenv_nomalloc

GDBusConnection: warn that direct access to the stream is a bad idea

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=661679
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Add two annotations to gio

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

Document that types are limited to 64 KiB

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=659916
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

GIOModule: Use G_SEARCHPATH_SEPARATOR_S instead of ":"

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

GKeyFile: improve introspection annotations

Ensure that all methods that take or return arrays are annotated
(including length). Mark ref, unref and free methods as (skip).

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

Turn GKeyFile into a boxed for introspection

Using the new refcounting API, introduce a boxed type wrapping
GKeyFile and expose it introspection bindings in glib-types.h.

GKeyFile: Add refcounting API

Adds g_key_file_ref and g_key_file_unref, to be used by a future
GKeyFile boxed type for language bindings.

Based on the patch by Christian Persch and Emmanuele Bassi.

Author: Christian Persch
Signed-off-by: Johan Dahlin
Signed-off-by: Giovanni Campagna
https://bugzilla.gnome.org/show_bug.cgi?id=590808

Add 'Since 2.32' indexes to gio and gobject docs

GAppLaunchContext: add environment-manipulating functions

Add functions for manipulating the environment under which a
GAppLaunchContext will launch its children, to avoid thread-related
bugs with using setenv() directly.

FIXME: win32 side isn't implemented yet

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

Misc doc formatting fixes

Add tests for new g_environ_ functions

Move environment-related functions into their own files

gutils.[hc] is a bit of a grab bag, so lets start cleaning
things up by moving all the environment-related functions
into separate genviron.[hc] files.

The private _g_getenv_nomalloc has been moved to its sole caller.

gutils: Add functions for working with environment arrays

When spawning a child process, it is not safe to call setenv() before
the fork() (because setenv() isn't thread-safe), but it's also not
safe to call it after the fork() (because it's not async-signal-safe).
So the only safe way to alter the environment for a child process from
a threaded program is to pass a fully-formed envp array to
exec*/g_spawn*/etc.

So, add g_environ_getenv(), g_environ_setenv(), and
g_environ_unsetenv(), which act like their namesakes, but work on
arbitrary arrays rather than working directly on the environment.

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

Add to the pitfalls: fork() and daemon()

tests/Makefile.am: remove unused LDFLAGS variable

Expand thread docs a bit

Add some discussion of pitfalls of threaded programming.

Documentation tweaks

Updaged Galician translations

Check for "our" threads in some places

Don't allow g_thread_join() to be called on or g_thread_exit() to be
called from within threads that were not created by GLib.  Document
this.

gtk-doc g_thread_ref() and g_thread_unref()

Rename g_thread_try to g_thread_try_new

Add atomic test to TEST_PROGS

Documentation additions

Improve test coverage

Add a test involving g_thread_ref/unref

Also, fix the resource-limit test to properly unset RLIMIT_NPROC.

Add g_thread_try to the docs

Update doc lists