Improve option context test coverage

Improve array test coverage

Move markup parse tests to the test framework

Improve coverage of buffered output stream tests

Improve coverage of credentials tests

Improve coverage of converter stream tests

Improve coverage of memory stream tests

Remove redundant include

Improve coverage of icon tests

Improve coverage of application tests

Improve coverage of stream tests

Reenable unix-streams test

Work around deadlock in unix-streams test

Add tests about file associations

Improve regex test coverage

Remove duplicate utf8 strreverse test

Also add some tests for find next/prev char.

Improve strfuncs test coverage a bit

Improve option context test coverage a bit

Move markup collect tests to the test framework

Add some more GFile tests

Improve compressor test coverage a bit

Test g_settings_list_schemas

Improve GNode test coverage a bit

Add some logging tests

Silence log handler tests

Improve tree test coverage

Move markup escape tests to test framework

More pattern tests

Add tests for g_utf8_strreverse

Fix a warning

Don't leak in converter test

Fix an icon test

Add tests for GNetworkAddress

Drop an unneeded if

Remove some unused pcre code

Add some volume monitor tests

Although not much of this is easily testable.

Move queue tests to the test framework

Bug 625628 – GDBusProxy: wrong property name sorting

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

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

Fix gschema-compile test

This was broken by commit 1b5b87bfa0f5435dd6e973a4e36076db083dc5c7

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

Bug 625584 – Crashes application on unref with signal subscription

Don't do too much work in the finalizer - in particular, there's no
need to send RemoveMatch() messages to the bus daemon since we're
going to sever the connection and the bus will garbage collect
anyway. In this case it crashed the process.

Also add a test case that checks that the appropriate GDestroyNotify
callbacks are called when unreffing a connection with either 1)
exported objects; 2) signal subscriptions or 3) filter functions
.. yes, ideally apps would unregister such callbacks before giving up
their ref but that's not how things work :-)

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

GSettings: Mark some strings for cat translation

<vuntz> *miaou*

GDBusConnection: Fix up debug output when completing calls

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

GCredentials: use effective uid/gid

Otherwise e.g. setuid root processes can't connect to the system
bus. This was discovered when porting PolicyKit's pkexec(1) command to
a PolicyKit library using GDBus.

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

*bump*

add one more NEWS item

GIO: Add G-I annotations for several methods

Specifically:
 - g_io_stream_get_input/output_stream()
 - g_socket_client_get_local_address()
 - g_socket_connection_get_socket()
 - g_socket_listener_add_address()
 - g_socket_listener_accept_socket()
 - g_socket_listener_accept()
 - g_io_extension_point_get_extensions()

get a start on the NEWS

[gschema-compile] fix segfault

If files are passed on the command line and we don't set override_files
at all, don't try to read from the array.

Use correct type for GDBusServer:active property

It's a boolean, not a string.

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

GDBusConnection: Pass the GObject to g_simple_async_result_new()

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

Add support for vendor override files

You can drop a key-file in the schema directory that looks like:

[org.gtk.Example]
key='value'

to override the default value of 'key' in schema 'org.gtk.Example'.

Updated Galician translations

Updated Spanish translation

Updated Hebrew translation.

[gio]: gioenums.h: Remove trailing comma again.

Revert "notify: Remove unused g_object_notify_queue_clear()"

This reverts commit 0201a81f04ae5fd46c6db25859e90dde1fc35d4b and adds
locks in the right place.

As this file is a public header (see last commit), we cannot just remove
functions from it.

notify: Readd g_object_notify_queue_from_object()

gobjectnotifyqueue.c is a public header, so we cannot remove files from
it. (Yes you read that right, this .c file is a public header. See your
installation directory.)

Complete Simplified Chinese translation.

Update Simplified Chinese translation.

Cleanup the GSettingsBackend API

This is a substantial ABI break.  Rebuild dconf.

GSettings: add g_settings_reset()

Resets a key to its default value.

notify: Add tests for threadsafe object notifies

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

notify: Make dedup code clearer

A for loops seems easier to understand to me than gotos.

notify: Make GObject property change notifications thread safe

Adds locking around object property change notification handling. The
notifications are only emitted after all threads have called
g_object_thaw_notify().

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

notify: Remove g_object_notify_queue_from_object()

The function will not be safe with object locking in place, so we remove
it. The workaround is somewhat ugly, but it works.

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

notify: Refactor g_object_notify_queue_thaw()

This adds better error reporting and simplifies the code for adding
thread safety.

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

notify: Remove unused g_object_notify_queue_clear()

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

notify: Emit a g_critical() instead g_return_if_fail()ing

This does not change the code semantically in any way but avoids a
return in the middle of the code.

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

binding: Cast to avoid compiler warning

tests: Use GPOINTER_TO_SIZE() instead of just casting to guint

a) gcc doesn't like this ("Cast to pointer of different size)
b) It compares only half of the pointer

inotify: Constify function argument

gcc complained...

Fix very small leak in the GSettings test

Bug 624991 - GSettings mapping for G_TYPE_STRV

Added default mapping for G_TYPE_STRV based on a patch from Garrett
Regier.

Add a test case.

Updated Galician translations

Bug 624968  - div by zero in g_malloc_n family

Updated Hebrew translation.

GDBusAuthObserver: Fix docs

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

Bug 617483 – Credentials passing

 - Make GCredentials instance and class structures private so it can't
   be subclassed and we don't have to worry about ABI compat
   issues. This also allows us to get rid of the GCredentialsPrivate
   struct.

 - Add a GCredentialsType enumeration that is used whenever exchanging
   pointers with the user. This allows us to support OSes with
   multiple native credential types. In particular, it allows
   supporting OSes where the native credential evolves or even changes
   over time.

 - Add g_socket_get_credentials() method.

 - Add tests for g_socket_get_credentials(). Right now this is in the
   GDBus peer-to-peer test case but we can change that later.

 - Move GTcpConnection into a separate gtk-doc page as was already
   half-done with GUnixConnection. Also finish the GUnixConnection
   move and ensure send_credentials() and receive_credentials()
   methods are in the docs. Also nuke comment about GTcpConnection
   being empty compared to its superclass.

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

GDBusServer: Avoid handling incoming connections after stop() has been called

Without this fix, we'd sometimes run code after stop() and finalize()
to handle incoming requests. This was observed in the gdbus-peer test
case occasionally crashing:

 $ ./gdbus-peer
 /gdbus/peer-to-peer: OK
 /gdbus/delayed-message-processing: OK
 /gdbus/nonce-tcp:
 GLib-GObject-WARNING **: invalid uninstantiatable type `(null)' in cast to `GDBusServer'
 aborting...

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

GDBus: Add support for D-Bus type 'h' (ie. G_VARIANT_TYPE_HANDLE)

This allows sending and receiving D-Bus messages with instances of the
'h' D-Bus type. Unlike libdbus-1's dbus_message_iter_get_basic()
method, g_variant_get_handle() does not return a duplicated unix file
descriptor (that must be closed with close(2)) - instead, it returns
an index that can be used to get/dup the file descriptor from a
GUnixFDList object that can be obtained from the GDBusMessage object.

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

Move #include <sys/wait.h> down

Until after we include the glib stuff, so that we have G_OS_UNIX
defined.

For some reason <stdlib.h> pulls in <sys/wait.h> on Fedora so this
wasn't a problem, but many others have reported the issue.

Updated Spanish translation

Corrected and updated the new version of the Armenian translation - po file.

GDBus: Introduce G_DBUS_SEND_MESSAGE_FLAGS_PRESERVE_SERIAL

This is preferable to the current magical solution whereby the serial
is only rewritten if non-zero. In particular, it makes it easier to
send the same message on multiple connections without having to reset
the serial number.

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

GDBus: Forgot to add g_dbus_send_message_flags_get_type() to gio.symbols

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

GDBus: Add GDBusSendMessageFlags

This is currently unused but might be useful in the future. For
example, it might be nice with a way to bypass the current queue of
outgoing messages - having a flag enumeration allows us to add a
G_DBUS_SEND_MESSAGE_FLAGS_BYPASS_QUEUE etc. etc.

This commit breaks ABI and API. Users of the (rarely used) API to send
messages will have to port to this new API.

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

GDBus: Remove constness from introspection data structures

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

GDBus: Add GDBusSignalFlags and use it in g_dbus_connection_signal_subscribe()

This is currently unused but will probably be useful in the
future. For example, we could have a _ARG0_IS_PATH to specify that
arg0 should be used for arg0path.

This commit breaks API and ABI. Users of
g_dbus_connection_signal_subscribe() will need to port to this new
version.

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

Update the GDBus subtree example for the new API

Bug 624754 – gdbusaddress.c missing sys/wait.h

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

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

Updated Spanish translation

Fixed headers in galician translation file

Updated Hebrew translation.

GDBus: Add nonce-tcp: test case

Also fix a couple of TODO items in gdbusaddress.c

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

Updated Hebrew translation.

g_warning on inconsistent subtree behaviour

If the subtree introspection function indicates that an interface exists
but then the dispatch function returns a NULL vtable for that interface,
issue a g_warning pointing programmers in the right direction.

gio: don't assume that SOCK_CLOEXEC is supported whenever it's defined

Just because SOCK_CLOEXEC was defined at build time doesn't mean the
kernel we're running on supports it.  So if socket() fails with EINVAL,
try again without the flag.

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

Bug 624484: GDBusSubtreeDispatchFunc clarification

Pass NULL rather than "/" for the toplevel of a subtree.

Add clarification to the docs about how trees are flat.

Bug 624483: GDBusSubtreeEnumerateFunc clarification

Clarify a couple of things in the docs:

  1) you must return flat names (no slashes)

  2) g_strfreev() will be called on the result

  3) a benefit of using the DISPATCH_TO_UNENUMERATED flag

Bug 624473: GDBusSubtreeIntrospectFunc return type

Return a NULL terminated C array instead of a GPtrArray

Also, document that %NULL is a permitted return value and clarify its
meaning.

Finally, avoid calling the enumeration function during dispatch when the
G_DBUS_SUBTREE_FLAGS_DISPATCH_TO_UNENUMERATED_NODES flag was given.