g_menu_model_get_item_link: be careful with refs

Don't unref the hashtable until after we already take the ref on the
value that was contained in the hashtable, otherwise we may kill the
value.

Add debug code

This adds a dump method to show the entire tree of proxies.

Avoid cross-talk between tests

Each test needs to remove the sources that it attaches
to the default main context, or else things will work
fine in isolation, but go bad in a full test run.

Typo fix

Remove unused variable

Add a test for menu subscriptions

Take out excessive width

Reduce excessive width

whitespace fix

Add some tests for links

Add parser roundtrip tests

GMenuMarkup: fixes uncovered by roundtrip testing

The print function was forgetting to emit type information for
attributes, and the parser was not handling types properly either.

Improve docs

Specify that g_menu_item_set_attribute_value consumes floating
@values.

GApplication: Add a menu example to the docs

GMenu: Enforce attribute name restrictions

The code assumes in various places that ':' does not occur
in attribute names. We are a little more strict than that,
and only allow lowercase ASCII, digits and '-'.

Reduce excessive width of gmenu.h

Match up a parameter name

Describe the org.gtk.Menus interface

Even though we consider the interface to be an implementation
detail, we should have internal documentation of the interface.

Describe the org.gtk.Actions interface

Even though we consider the interface to be an implementation
detail, we should have internal documentation for the interface.

Rename exporter APIs

There are no public 'exporter' objects, so don't allude to them
in the function names. At the same time, we want to make it clear
that these functions are D-Bus specific.

The new APIs are
g_action_group_dbus_export_start
g_action_group_dbus_export_query
g_action_group_dbus_export_stop
g_menu_model_dbus_export_start
g_menu_model_dbus_export_query
g_menu_model_dbus_export_stop

GActionGroup: Mention GMenuModel in the docs

Add g_application_set_menu

This adds ane asy way to export menu information alongside
the actions that are already exported by GApplication.

Add GMenuModel testcases

Add GMenuProxy

GMenuProxy can be used as a client-side wrapper
for the a menu model that has been exported on D-Bus.

Add GMenuModel D-Bus exporter

Add GMenu markup

These functions serialize and deserialize a GMenuModel
to and from XML.

Add GMenu

Add GMenuModel

GApplication: use GDBusActionGroup

instead of the internal version of the same

Add GDBusActionGroup

GDBusActionGroup can be used as a client-side wrapper
for the an action group that has been exported on D-Bus.

GApplication: port action client to new D-Bus API

for compatibility with GActionGroup exporter

GApplication: use GActionGroup exporter

instead of home-grown internal implementation of the same

Add GActionGroup D-Bus exporter

Revert "Use an XML blob for gapplication interface info"

This reverts commit 8013401f6e4edb01dd31b716c179806b55d4064b.

Revert "Forgot one interface..."

This reverts commit 30580bdca7764525a49899dd56dcdde25a300a21.

Updates

Clarify docs around main loop vfuncs

Mention that the vfuncs are no longer used, and tweak the
wording around what g_application_run() does.

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

Bug 665733 – GDBusConnection holds lock while calling destroynotify

Fix this problem by always running the destroynotify from an idle.

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

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

unicode: Fix a few issues with G_UNICHAR_MAX_DECOMPOSITION_LENGTH

Raised by Matthias in bgo#665685 but which I didn't spot until after pushing
commit 3ac7c35656649b1d1fcf2ccaa670b854809d4cd8.

Renames G_UNICHAR_MAX_DECOMPOSITION_LEN to G_UNICHAR_MAX_DECOMPOSITION_LENGTH
and fixes a few documentation issues.

See: bgo#665685

Bug 665685 — Add a #define for the max length of a Unicode decomposition

Add G_UNICHAR_MAX_DECOMPOSITION_LEN for the maximum length of the
decomposition of a single Unicode character.

Closes: bgo#665685

GDBus: Note in docs that g_dbus_node_info_new_for_xml() is using GMarkup

This was discussed in bug 665634.

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

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

GDBus: fix link to D-Bus spec

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

Minor doc fix.

Minor

Make the qsort_r check cross-compile friendly

Pointed out by Daniel Mack.

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

outputstream: Clarify docs of flush()

After questioning the semantics of flush on IRC, it seemed necessary to
clarify what flushing is supposed to do. The Linux man page for fflush()
seemed to cover it perfectly, so I just copied it.

I did not add the "via the underlying write mechanism" part as that in
my opinion is not something subclasses should need to guarantee.

Send CHANGES_DONE_HINT on file moves if no IN_CLOSE_WRITE is emitted

This patch makes GFileMonitor to emit EVENT_CHANGES_DONE_HINT when
EVENT_CREATED is emitted but the file is not opened for writing.

On file moves across different mounted volumes, inotify will always emit
IN_CREATE and IN_CLOSE_WRITE (plus other events).
This translates into GIO's _EVENT_CREATED and _EVENT_CHANGES_DONE_HINT.

On file moves across the same mounted volumes, inotify will emit
IN_MOVED_FROM/IN_MOVED_TO which will be translated into
_EVENT_DELETED/_EVENT_CREATED GIO's side. No _EVENT_CHANGES_DONE_HINT is
emited afterwards.
Under such circumstances a file indexer does not know when actually the
file is ready to be indexed, either waiting too much or triggering the
indexing twice. On small devices it's not advisable.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=640077
Bug-NB: NB#219982
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Tomas Bzatek <tbzatek@redhat.com>

GDBusConnection: propagate error correctly if flags are inappropriate

Previously, this would fail the assertion
"connection->initialization_error != NULL" after the label "out".

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

Link to libpcre correctly when static linking

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

Updated Spanish translation

GDBusInterfaceSkeleton: make it possible to export on multiple connections

This is useful in peer-to-peer connections.

With minor changes by David Zeuthen <davidz@redhat.com>.

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

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

GVariant: add preconditions to ref. counting functions

These prevent GVariants from accidentally being brought back to life after
being freed, and should make it easier to track down ref. counting issues.

Closes: bgo#665184

Updated Norwegian bokmål translation

GMemoryInputStream/GMemoryOutputStream: fix bug in previous commit

A g_input_stream_read_async() implementation can't call
g_input_stream_read() on itself directly because it will fail because
the pending flag is already set. So fix that by invoking the vmethod
directly rather than calling the wrapper. Likewise with
GMemoryOutputStream.

Add a test to gio/tests/memory-input-stream.c to catch read_async
failures in the future.

GMemoryInputStream/GMemoryOutputStream: make these properly subclassable

The async methods were directly invoking their own implementations of
the sync methods, making it impossible for a subclass to reimplement
them.

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

Don't abort if g_ascii_strtod tests fail

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

Remove two useless lines of code

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

Fix a little comment error in the _GTreeNode struct

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

Clarify local_command_line docs

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

Do not use static GTypeInfo and GInterfaceInfo

Either g_type_register_static_simple (used by G_DEFINE_TYPE_EXTENDED)
and G_IMPLEMENT_INTERFACE use automatic variables for GTypeInfo and
GInterfaceInfo structs, while tutorials and source code often use
static variables. This commit consistently adopts the former method.

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

gnetworkmonitornetlink.c: Make the error messages less geeky

This makes several different errors have identical messages, but
they're pretty much "can't happen" situations anyway.

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

gzlibdecompressor.c: fix a comment

Add missing annotation to g_file_set_attribute()

g_file_set_attribute() also permits a NULL value for value_p, and requires it
to be NULL to unset it. Also fix the wrong variable name in the documentation.

Signed-off-by: Martin Pitt <martin.pitt@ubuntu.com>

Forgot one interface...

Use an XML blob for gapplication interface info

This is both more readable and more efficient. Double win !

gtester: Add command line option to skip tests

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

Rearrange GApplication docs

Move the GApplication docs to the front of the section,
where they belong.

gio.symbols: add g_application_[gs]et_default()

Cosmetics

Reword the title of GSimpleAction in the docs.

GApplication: add default application

GApplication: unregister action interface export

We missed a few cases of unexporting the action group interface (in
error cases, and on GApplication teardown).

GApplication: Implement query_action

GSimpleActionGroup: Implement query_action

Add g_action_group_query_action()

This new API allows requesting multiple pieces of information about a
particular action in one go and also simplifies the burden for
GActionGroup implementations -- they need not implement all the separate
APIs now.

GBytes: A new type for an immutable set of bytes.

 * Represents an immutable reference counted block of memory.
 * This is basically the internal glib GBuffer structure exposed,
   renamed, and with some additional capabilities.
 * The GBytes name comes from python3's immutable 'bytes' type
 * GBytes can be safely used as keys in hash tables, and have
   functions for doing so: g_bytes_hash, g_bytes_equal
 * GByteArray is a mutable form of GBytes, and vice versa. There
   are functions for converting from one to the other efficiently:
   g_bytes_unref_to_array() and g_byte_array_free_to_bytes()
 * Adds g_byte_array_new_take() to support above functions

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

Fix distcheck

Now that we have switched to template-free docs, these
hand-rolled cleanup rules get in the way.

GInitable, GAsyncInitable: not initializing gives undefined behaviour

This is the ISO C sense of undefined behaviour, in which
works-by-coincidence, critical warning, abort, demons-fly-out-of-your-nose
are all valid implementations.

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

GDBusWorker: continue_writing: initialize flush_async_data if closing

This was a regression in commit f41178c6c: flush_async_data wasn't
necessarily NULL in the "don't flush" case.

Also move initialization of these variables up so that it's
unconditional, since that's easier to verify than checking
that each branch gets it right.

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

Add some more mainloop tests

Remove unnecessary includes

These were meant for some other test that ended up not getting
added. This fixes bug 664559.

Fix !debug builds

As pointed out by Chun-wei Fan in bug 664455, parts of the merged
gobjectnotifyqueue.c code ended up inside an #ifdef G_ENABLE_DEBUG
section.

Add back required _XOPEN_SOURCE define

This got lost somewhere between 2.30 and now, but it is needed
for isnan(). See bug 664354, reported by Tim Mooney.

GApplication: don't fail if can't get session bus

If we can't get on the session bus, just behave like a normal non-unique
application.

This turns out to be remarkably easy to implement and lets us avoid
adding a 'dummy' backend.

Add a test for this case as well.

Idea from Zachary Dovel.

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

gio: Move gmodule to Requires.private

Users of GIO are not generally expected to use gmodule API,
so don't force them all to link against it.

GDBusConnection: don't pass boolean > 1 to _g_dbus_worker_new

This happens to work at the moment (because GDBusWorker.frozen is a
gboolean and not just a 1-bit bitfield), but isn't right: the gboolean
ends up with values 0 or G_DBUS_CONNECTION_FLAGS_DELAY_MESSAGE_PROCESSING
(which is more than 1).

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

GSocketInputStream/GSocketOutputStream: fix win32 build

based on a patch from Chun-wei Fan

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

gio/tests: Fix for -Werror=format-security

post-release bump

release glib 2.31.2

dist gio/tests/key8.pem

Add test case for #662395

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

GDBus tests: factor out TestIOStream, test_pipe and test_bidi_pipe

These might even make useful public API if they grew a Windows
implementation, but for now they can be Unix-only test API.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

_g_dbus_worker_flush_sync: always flush if we need to

We didn't previously flush in a couple of cases where we should have
done:

* a write is running when flush is called: we should flush after it
  finishes

* writes have been made since the last flush, but none are pending or
  running right now: we should flush the underlying transport straight
  away

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

GDBusWorker: move flush async op into continue_writing()

This makes it easier to schedule a flush, by putting it on the same code
path as writing and closing.

Also change message_written to expect the lock to be held, since all
that's left in that function either wants to hold the lock or doesn't
care, and it's silly to release the lock immediately before calling
message_written, which just takes it again.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

schedule_write_in_worker_thread: require caller to lock; rename accordingly

When we use this function to schedule a flush, it'll be called
with the lock held. Releasing and immediately re-taking the lock would
be pointless.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

GDBusWorker: rename some functions

maybe_write_next_message now also closes, and I'm about to make it
consider whether to flush as well, so its name is increasingly
inappropriate. Similarly, write_message_in_idle_cb is a wrapper around
it which could do any of those things.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

GDBusWorker: distinguish between 3 sorts of output that might be pending

If the user calls flush_sync() with no messages in the queue, but an
async write call pending, then we ought to flush after that async write
returns (although we don't currently do that). If it was an async close
or flush that was pending, there's no need to flush (again) afterwards.
So, we need to distinguish.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=662395
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

Drop last uses of @returns:

Missed a couple of @returns

GVariant: drop use of @returns