GDBusConnection: Avoid dereferencing vtable for unregistered objects

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

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

bump version

Fix distcheck

automake decided to complain about some more leftovers.

2.31.22

Updates

Don't return from a void function

Fix a C99ism

Silence a compiler warning

GDBus: set no-reply flag on calls with no callback

If g_dbus_connection_call() or g_dbus_proxy_call() are given a NULL
callback then set the no-reply flag on the outgoing D-Bus message.

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

GDBusProxy: report an error instead of leaking it

GDBusProxy sets an error on a GSimpleAsyncResult and then returns
without dispatching the result for completion (and leaks the result in
the process).  Fix that.

Also add a testcase.  Unfortunately, adding the testcase uncovered
bug #672248.  We can work around that by reordering the tests.

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

Update config.h.win32(.in)

Make it more like the one that is generated by autotools.

It is true that Visual C++ has sig_atomic_t, at least for Visual C++ 2008
and later, but this is currently only used for UNIX builds of GLib, as a
point of note here.

Update glibconfig.h.win32(.in)

Added the definitions below, as these #defines are needed for gqsort.c

#define ALIGNOF_GUINT32 4
#define ALIGNOF_GUINT64 8
#define ALIGNOF_UNSIGNED_LONG 4

Updated French translation

Make inf testcase more forgiving

printf may produce "infinity" instead of "inf".

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

Use __builtin_bswap* for GUINT*_SWAP_LE_BE if building with GCC

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

Fix out of tree build for MinGW/MSYS

When building with MinGW/MSYS with srcdir != builddir the build fails:
- to locate the generated .def files
- creating libglib-gdb.py
- creating libgobject-gdb.py

Solved this by explicitly instructing these files to be generated
in $(builddir)/...

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

Updated Portuguese translation

Updated Lithuanian translation

man pages: document the order of schema overrides

GSettings overrides are processed in such a way that
alphabetically-later files have precedence over earlier files (eg: 20_
will beat 10_).  Document that fact.

Updated Russian translation

gmain: Use sig_atomic_t for list of pending Unix signals

Pointed out by: Simon McVittie <simon.mcvittie@collabora.co.uk>

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

gio: Only show mounts in /run/media/$USER

And reflect the recent changes + sync with gvfs+udisks2 rules.

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

Continue searching in PATH on ENODEV and ETIMEDOUT

Synchronize the error-handling logic from glibc. Users with networked
filesystems in their PATH may hit ETIMEDOUT.

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

gtype: Note idempotency of g_type_init()

Remove now unused qsort_r checks

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

Make g_array_sort* methods use a stable sort

Also, remove previous comments about sort stability in g_array_sort docs,
as the method that was explained does not work. Adds a new comment
about this.

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

Make g_qsort_with_data stable, based on glibc msort

We need a stable sort, and we might as well always use it rather
than have multiple sort versions. This picks up the glibc
merge sort implementation which it uses by default for qsort,
except we don't fall back to non-stable quicksort in some cases
like glibc

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

Updated Hungarian translation

[l10n]Updated Catalan (Valencian) translation

[l10n]Updated Catalan translation

GDBusConnection: Don't store GCancellable in qdata

Now that we're using g_simple_async_result_set_check_cancellable() we
no longer need this terrible hack of carrying the GCancellable on the
GSimpleAsyncResult using qdata. See bug 672013 for more details.

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

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

GDBus: make use of reliable async cancellation

Call g_simple_async_result_set_check_cancellable() after all
GSimpleAsyncResult creation in order to take advantage of the new
reliable cancellation feature.

The guarantee of reliable cancellation fixes a bug in dbusmenu (which
was already assuming that cancellation was reliable).  See this bug:
https://bugs.launchpad.net/ubuntu/+source/libdbusmenu/+bug/953562

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

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

Fix "install" paths in VS property sheets

...and cleanup the VS2010 property sheets a little bit

[l10n] Updated German translation

GSimpleAsyncResult: support reliable cancellation

Add a function g_simple_async_result_set_check_cancellable() to provide
a GCancellable that is checked for being cancelled during the call to
g_simple_async_result_propagate_error().

This gives asynchronous operation implementations an easy way to
provide reliable cancellation of those operations -- even in the case
that a positive result has occured and is pending dispatch at the time
the operation is cancelled.

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

Invalidate single_va_closure when adding an emission hook

If the optimization is used for only having one closure handling a
signal emission, then hooks will not be run, so it should be disabled
when an emission hook is added.

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

g_log: mask log level before checking prefix flags

We check if the log level is in the "prefixed" list by checking it
against the g_log_msg_prefix bitfield.

Unfortunately we were failing to mask by G_LOG_LEVEL_MASK first, so if
the FATAL bit was set (for example) then it would never match.  This was
the case for g_error().

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

glib-init: restore default for G_MESSAGES_PREFIXED

In the glib-init shuffle this ended up being set to no messages prefixed
by default.  Fix that so that we have prefixes back.

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

glib-init: fix obvious buffer size mismatch

Updated British English translation

Updated Danish translation

GSocketMsgFlags: annotate as a flags

This was being incorrectly guessed as an enum.  Fix that.

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

Updated Norwegian bokmål translation

Finnish translation update from l10n.laxstrom.name/wiki/Gnome_3.4 translation sprint

Updated Esperanto translation

Updated Polish translation

logging test: unset G_MESSAGES_DEBUG

Having G_MESSAGES_DEBUG=all in the environment will fail this testcase.

deprecated threads: fix race in GStaticRecMutex

The very last access to the 'depth' field of GStaticRecMutex in
g_static_rec_mutex_unlock_full() was being performed after dropping the
implementation mutex for the last time.

This allowed the lock to be dropped an additional time if it was
acquired in another thread right at that instant (which is somewhat
likely, since another thread could have just been woken up by the lock
being released).

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

Fix crash in signal emission optimization

The trace of the signal emission of the was calculating the instance
type after the instance was unrefed. Fix this by keeping the instance type around.

signal emission: Allocate instance_and_param with alloca

This Gives a 5% performance increase in e.g. the emit-handled test.

Use native glib-genmarshal when cross-compiling

The ability to cross-compile glib got broken after the
merge of the 'signal-performance' branch as the assumption
was made that the generated glib-genmarshal can be executed
on the host (which isn't valid when cross-compiling).

Fixed this by using the just-built glib-genmarshal for normal
compilations and the native (host) glib-genmarshal when doing a
cross-compilation as was also done in several other areas of GLib

Tested for host=x86_64-unknown-linux-gnu, target=x86_64-unknown-linux-gnu
and host=x86_64-unknown-linux-gnu, target=i686-w64-mingw32

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

gio-querymodules: unlink instead of writing empty cache

If there are no modules installed then the most appropriate thing is to
have no cachefile instead of an empty one.  This unbreaks the "clean
directory after 'make uninstall'" check that automake does.

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

msvc_recommended_pragmas.h: Silence C4819 warnings

This warning appears when unicode chars that cannot be displayed in the
current Windows code page is used anywhere in the file, including comment
blocks.  We probably don't need to see these, especially as problems
caused by such characters are manifested as other warnings or errors,
for example, the need to add BOM to a file when compiling code with
complex script on Windows.

Assamese translation completed

Updated Traditional Chinese translation(Hong Kong and Taiwan)

Update config.h.win32(.in)

Remove the config for ENABLE_REGEX, as GRegex is now included in all builds.

glib-genmarshal.c: Use G_VA_COPY instead of va_copy()

In the code generation portion, va_copy() is not universally available,
so use the existing G_VA_COPY macro that in turn calls va_copy() if it
is available or call an appropriate emulation otherwise.

Use G_VA_COPY instead of va_copy()

va_vopy() is not universally available in all compilers, so make use of
the existing G_VA_COPY macro which either calls va_copy() if it is
available, or emulates it if otherwise.

gobject: Fix a compiler warning

Shuffle the parenthesis around.

tests: fix GAction test for headless run

Assamese translation completed

Updated Korean translation

Fix unused variable warning

GUnix{Input,Output}Stream: Only release cancelable poll-fd if we allocated it

Otherwise we get criticals a'la

 GLib-GIO-CRITICAL **: g_cancellable_release_fd: assertion `cancellable->priv->fd_refcount > 0' failed

when reading/writing to certain kinds of file descriptors.

Patch reviewed by Dan Winship on IRC.

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

bump version

Use G_DEFINE_CONSTRUCTOR for the glib_init constructor

No need to hand-roll this when we have a macro version.

2.31.20

Update glib-genmarshal docs

A bunch of new commandline options sneaked in without docs...

Fix a compiler warning

gobject: Fix symbol lists

More updates

Merge branch 'signal-performance'

Fix distcheck

update Simplified Chinese (zh_CN) translation

Fix distcheck

Updates for 2.31.20

Fix binding properties of the same object

This problem was pointed out in bug 639873.
The patch here is based on Matt Barnes patch,
I've added a testcase as well.

Run gdb under libtool so that local libraries are used

If this isn't done, the system-wide glib is used for this test,
and that can cause test failures.

https://bugzilla.gnome.org/621368

Updated Slovenian translation

Updated Slovenian translation

build: include glib-compile-resources.xml in the dist tarball

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

build: Fix failure on "make distclean"

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

fastpath: Do allow NO_RECURSE fastpath for NOP emissions

This fixes a performance regression wrt the old NOP emission
handler.

glib/gmacros.h: Fix up attribute to avoid namespace collisions

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

gspawn: rename G_SPAWN_ERROR_2BIG to be more bindings-friendly

Rename G_SPAWN_ERROR_2BIG to G_SPAWN_ERROR_TOO_BIG (while keeping the
old name for compatibility), to fix problems with language bindings
where the old name translates into something that would be
syntactically invalid due to starting with a digit.

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

GApplication: fix name unregister logic

We were attempting to unregister our ownership of our D-Bus name even in
the case that we were non-unique (ie: we didn't actually own the name).
Rework the logic a bit to prevent that: for non-unique, we leave
impl->bus_name as NULL and we only register/unregister if it is
non-NULL.

Updated Vietnamese translation

po/vi: import from Damned Lies

Updated Basque language

Bug 640202 Impossible to build multi-architecture libraries

Hides ELF-only linker flag -export-dynamic from non-ELF linkers.

Mask out G_SIGNAL_TYPE_STATIC_SCOPE when comparing types

build: Fix rules for marshalers.[ch]

Add more tests for signal emissions

Optimize single-handler va_marshaller case

When there is only one closure handling a signal emission and
it doesn't have a bunch of complicated features enabled we
can short circuit the va_args collection into GValues and call the
callback via the va_marshaller directly.

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

Use builtin marshallers and va_marshallers if possible

If the signal argumment types matches a built in standard
marshaller we use the va_marshaller for that, and also the
normal marshaller if NULL was specified (as its faster than
the generic one).

Add g_signal_set_va_marshaller

This lets you set a va_marshaller on your signal which will be
propagated to all closures for the signal. Also, automatically
uses the generica va_marshaller if you specify a NULL c_marshaller.

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

Add _g_closure_is_void to check for NULL vfuncs

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

Support generating va marshallers in glib-genmarshal

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

Add optional support for varargs marshallers to GClosure

These closures support being invoked on a va_args which can
be useful as you can then avoid boxing the va_args into
GValues in certain cases.

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

Add GRealClosure and move meta_marshallers there

This means we're not abusing the notifiers for meta_marshallres,
and we're able to later cleanly add other fields to GClosure.

We still have to leave the ABI intact for the GClosure->meta_marshal
bit, as old G_CLOSURE_N_NOTIFIERS macro instances still accesses it.
However, we always set it to zero to keep those macros working.

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