GDesktopAppInfo: fix default app logic

We use "tweaks" structures to track how a particular directory impacts
the list of added, removed and default applications.  We maintain this
set of tweaks for each directory, in a hash table, keyed by unaliased
mime type name, in order to facilitate fast lookups.

A typo in the logic for creating and maintaining the uniqueness of these
structures was causing the default app to be selected incorrectly from
time to time.  Fix that.

GCredentials: credentials messages are not correctly supported on NetBSD

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

Updated Indonesian translation

Updated Polish translation

2.41.4

Updated Korean translation

Updated French translation

Updated German translation

Don't mark GThread struct as deprecated

Having a definition of struct _GThread inside a deprecation
ifdef confuses gtk-doc into marking it as deprecated. Avoid this.

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

Updated Greek translation

Updated Hebrew translation

Updated Oriya translation

GUnixMounts: Improve efficiency in polling case

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

gmacros.h: add G_GNUC_*_IGNORE_DEPRECATIONS macros for clang

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

Signed-off-by: Thomas Haller <thaller@redhat.com>

gsocketclient: Handle cancellation between CONNECTING and CONNECTED

If a g_socket_client_connect_async() operation is cancelled between the
CONNECTING and CONNECTED events (i.e. while in the
g_socket_connection_connect_async() call), the code in
g_socket_client_connected_callback() would previously unconditionally
loop round and try the next socket address from the address enumerator
(by calling enumerator_next_async()). This would correctly handle the
cancellation and return from the overall task — but not before emitting
a spurious RESOLVING event.

Avoid emitting the spurious RESOLVING event by explicitly handling
cancellation at the beginning of g_socket_client_connected_callback().

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

Updated Czech translation

Updated Traditional Chinese translation(Hong Kong and Taiwan)

GApplication: Add g_application_add_main_option

This function adds a single main option entry to be handeled by
GApplication. The option entry has it arg_data field set to NULL
and will be added to the applications packed_options.

The rationale for this is that bindings will be able to add
command line options even when they can't use the un-boxed struct
GOptionEntry.

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

GUnixMounts: Fall back to polling on systems without mtab

This is necessary for many of the BSD family at least.

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

Updated Assamese translation

docs: Fix broken gtk-doc formatting

docs: Add index for 2.42 api

[l10n] Updated Catalan (Valencian) translation

[l10n] Updated Catalan translation

Updates for 2.41.3

Updated German translation

Visual C++ Builds: Update README.txt's

Update notes about usage of PCRE and for people attempting to build GLib
on a Chinese, Japanese or Korean locale.

docs: hyphenate "High-level" and "Low-level"

MSVC Builds: Generate glib-mkenums If Possible

As glib-mkenums would likely be used in the building of the other
components of the stack, such as Cogl, Clutter and
gsettings-desktop-schemas, generate that using a Python script (if Python
can be found) and "install" it.

gtype: Fast-path for g_type_is_a

Do I really need to explain why ?

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

Do not crash when checking whether an instance type is of a given fundamental.

Not all instances have a TypeNode associated (e.g. GstEvent), so lets check if node is available
before trying to use it.

This crash can be easily reproduced by creating an event with gst_event_new_eos and using
G_IS_OBJECT on the event instance.

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

Updated Slovenian translation

Make sure compiled resources only have / as dirsep

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

Updated Slovenian translation

Updated Basque language

Change W32 console detection to look at the window, not stdout

Check for console window being attached to the process, not for stdout
being redirected into a console window.

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

Fix tests to compile again - add missing headers to W32, call  correct process-id-getting function

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

Silence a controversial warning in win_iconv

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

Silence some uncontroversial warnings

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

Fix printf-tests on W32 by ifdefing the expected output

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

Save errno and pass it along to make sure gettext does not change it

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

win32: improve the package installation dir lookup

As an example, the core of gedit is in a private library
placed in %INSTALLDIR%/lib/gedit/libgedit.dll

Before this patch we would get %INSTALLDIR%/lib/gedit as the
installation package dir, while what we actually want is to get
%INSTALLDIR%

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

Ensure critial sections are released before returning

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

doap category core

Remove atomics from g_clear_object/g_clear_pointer

Practically no caller of these functions require atomic behaviour,
but the atomics are much slower than normal operations, which makes
it desirable to get rid of them. We have not done this before because
that would be a break of the ABI.

However, I recently looked into this and it seems that even if the
atomics *are* used for g_clear_* it is not ever safe to use this.  The
atomics protects two threads that are racing to free a global/shared
object from freeing the object twice. However, any *user* of the global
object have no protection from the object being freed while in use,
because there is no paired operation the reads and refs the object
as an atomic unit (nor can such an operation be implemented using
purely atomic ops).

So, since nothing could safely have used the atomic aspects of these
functions I consider it acceptable to just remove it.

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

tests/gobject/performance: Clean up and add refcount performance test

gfile: Fix memory leak in g_file_move()

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

configure.ac: fix typo in last patch

20e709d8b92e28d162c78b147afe4040e3a48b85 fixed some issues in
configure.ac but also introduced a typo.  Fix that.

Updated Lithuanian translation

configure: Remove unintentional nested functions

More of the same as 9362a1b2d987125a8aa0723b916d29aaaece329a.

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

gdbus: Properly fix encoding of double arrays

It turns out that this bug actually would (sometimes) impact any sort of
fixed-sized array with an alignment requirement of 8 due to incorrectly
counting the alignment inserted between the (aligned 4) array length and
the actual data.

Fix this properly and remove the exception for doubles.

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

gtlsinteraction: Hoist precondition before allocation

We're using a precondition in the middle of the function, and if we
hit it, we leak the closure.

Let's allocate the closure per path; this allows us to allocate it
before path-specific preconditions, and better avoids a pointless
malloc/free pair in the unhandled case.

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

gio: Fix regression encoding an array of doubles

Take the simple slow path in this case. Encoding a double
takes special precautions as you can see in append_value_to_blob()
and friends.

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

gio: Add regression test for double array encoding

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

Revert "gatomic: statically assert that our assumptions hold"

This reverts commit 7269d75321b9d2a967a59fb35f243397f577eb41.

Adding G_STATIC_ASSERT() into a header file caused compilation
problems with at least one app (Anjuta). Reverting to keep
GNOME continuous testing running.

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

Updated Norwegian bokmål translation.

gcredentials: Add assertion to pacify static analysis

This should always be true.

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

gkeyfilesettingsbackend: Add assertion for known-true condition

We know the group exists since we enumerated it.  Should silence a
static analysis warning.

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

glocalfileinfo: Suppress static analysis return value warning

Just ignore the return value, since we're checking contents != NULL.

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

gsettingsschema: Suppress return value check

We're intentionally ignoring the value here.  Pacifies static
analysis.

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

gapplicationimpl-dbus: Fix leak of fd list object

Spotted by static analysis.

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

gsettingsschema: Close directory

Spotted by static analysis.

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

gthreadedresolver: Unref unexpected address

I don't believe any real app would hit this, but we do leak
if it occurs.

Spotted by static analysis.

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

gfileutils: Add missing g_free() in error path

Discovered by static analysis.

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

gfile: Initialize variable to pacify static analysis

Not a real bug, but will quiet the analysis.

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

glib-init: statically assert that we have 8-bit bytes

configure.ac assumes this.

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

gatomic: statically assert that our assumptions hold

This code assumes that int is exactly 4 bytes, and that pointers
are either 4 or 8 bytes, on platforms with __ATOMIC_SEQ_CST.
In practice this is going to be true.

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

Updated Spanish translation

gio/tests/socket: add some missing g_assert_no_error()s

gsocket: Set SO_NOSIGPIPE on sockets on Darwin

This is a best-effort approach to preventing SIGPIPE emissions on Darwin
and iOS, where they continue to be intercepted by the Xcode debugger
even if SIG_IGN prevents them crashing the program.

This is similar to the existing code which sets MSG_NOSIGNAL on all
send() calls. MSG_NOSIGNAL doesn't exist on Darwin though.

Based on a patch from Philip Withnall.

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

gsocket: move set_fd_nonblocking() into g_socket_constructed()

Updated Brazilian Portuguese translation

ginetaddress: Add a precondition to g_inet_address_new_from_string()

Ensure that the input string is non-NULL, as inet_pton() does not like
NULL strings.

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

socketservice: minor cleanup

Bump version

2.41.2

Remove redundant checks before free

Remove checks for NULL before g_free().

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

gspawn-win32: do not rely on __argc and __argv global vars.

Since we are getting passed Unicode values these global vars
might not have the correct value. Instead always get the wide arguments
and convert them to utf8 to use them.

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

Fix typos in gbytes_hash and g_time_zone_find_interval docs

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

gio: avoid g_object_ref(NULL) in g_dbus_object_manager_client_finalize

When establishing a D-Bus connection failed, g_dbus_object_manager_client_finalize()
calls g_object_ref(manager->priv->connection) when that pointer is NULL,
which is considered and logged as error by glib.

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

Fix a failing testcase

This fix is a bit of a blind guess, I couldn't really discern
from recent commits whether this is an expected behavior change
or not.

docs: Use "Returns:" instead "Return:"

Fix typo

GDBusMessage: add some G_UNLIKELY on error paths

More for documentation than anything else...

GCond (linux): fix g_cond_wait_until() return value on timeout

It should return FALSE on timeout (and only on timeout), and
TRUE otherwise.

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

GMutex (linux): detect and report some errors

Detect the following two errors:

 - attempting to unlock a mutex that is not locked

 - attempting to clear a mutex that was not initialised or was
   initialised but is still locked

Both of these are fatal errors.  We avoid using g_error() here because
doing so would involve calls back into the GMutex code, and if things
are going off the rails then we want to avoid that.

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

GLib: implement GMutex natively on Linux

If we have futex(2) then we can implement GMutex natively and gain a
substantial performance increase (vs. using pthreads).

This also avoids the need to allocate an extra structure in memory when
using GMutex or GCond: we can use the structure directly.

The main reason for the increase in performance is that our
implementation can be made more simple: we don't need to support the
array of options on pthread_mutex_t (which includes the possibility, for
example, of being recursive).

The result is a ~30% improvement in uncontended cases and a much larger
increase (3 to 4 times) in contended cases for a simple testcase.

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

Fix up failure-to-see-expected-message logging

When GLib had been told to expect message X, but then actually saw
message Y, it would log the "did not see expected message" error with
message Y's log level and domain, which makes no sense. Change it to
log with domain "GLib" and G_LOG_LEVEL_CRITICAL instead.

Also, include the expected domain in the error message, so that if
that's the reason why the expectation didn't match, you can tell that
from the error message.

Update glib/tests/testing.c for these changes; for all other test
programs in GLib and elsewhere, this change should not break any
existing tests, it should only improve the output on failure.

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

GDBusMessage: fast-path decoding of fixed arrays

Instead of creating a separate GVariant for each of the 'y's in an 'ay',
use g_variant_new_fixed_array().

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

GDBusMessage: fast-path encoding of fixed arrays

Instead of handling each item separately, handle the array as a whole.

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

gtask: fix leak in example code

Added missing "Since:" annotations for g_input_stream_read_bytes() APIs.

GDBusMessage: simplify byteswapping

Remove some duplication by simplifying our byteswap checks.

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

gapplication tests: test resource base path

Run some cases to make sure resource base path is behaving as we expect
it to.

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

GApplication: add a "resource base path"

We don't use this for anything inside of GApplication yet, but Gtk is
about to start using it to find various bits of the application (such as
its menus, icons, etc.).

By default, we form the base path from the application ID to end up with
the familiar /org/example/app style.

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

ginetsocketaddress: Explicitly initialise flags for getaddrinfo()

This is not a functional change, as the hints field is static and hence
automatically initialised to zero — which happens to be what AF_UNSPEC
and the unspecified protocol are defined as. However, it’s best to be
explicit about this, in case AF_UNSPEC is _not_ defined as zero.

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

Fix typo

Updated Lithuanian translation

doc: small improvement and fixes

- Add an example to g_strsplit(), like it is done for g_strsplit_set().

- GTK-Doc generates a list if a "1." is at the beginning of a line.

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

doc: improve doc of g_input_stream_read()

I recently needed to nul-terminate the returned buffer, and I wasn't
sure if g_input_stream_read() does that or not. I've checked
glocalfileinputstream.c, which calls read(2) which doesn't nul-terminate
the buffer. So I assume it's the same behavior for all GInputStream
subclasses.

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