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

gbitlock: Drop @returns syntax from doc comments

GBuffer: Drop @returns syntax from doc comments

GSocketClient: Drop @returns syntax from doc comments

GSimplePermission: Drop @returns syntax in doc comments

GPermission: Drop @returns syntax from doc comments

Improve test coverage

Improve mainloop test coverage slightly

Improve test coverage

Fix a case conversion bug

For titlecase chars without uppercase variant, we were returning
0, contrary to the docs.

Improve test coverage

Add a test for g_unichar_isdefined

gio/Makefile.am

Filter out yet another *nix only source file from the Visual C++ project
file completion rules.

Update GLib Visual C++ projects

Also link to WinMM.lib since timeGetTime is used (commit 8d023c27).

Updated Galician translations

GFile: Add some introspection annotations

progress_callback in g_file_copy and g_file_copy_async may be NULL.
Annotations should reflect that. Pointed out by Carlo Teubner.

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

Document that g_list_sort is a stable sort.

This is a guarantee for the future so that applications can rely on it.

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

Updated Spanish translation

GApplication: unregister bus name

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

Trivial doc typo fix

Updated Norwegian bokmål translation

GTlsCertificate: support unencrypted PKCS#8 private keys

PKCS#8 is the "right" way to encode private keys. Although the APIs do
not currently support encrypted keys, we should at least support
unencrypted PKCS#8 keys.

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

[gio] Terminate connect call when application proxy is used

The connect_async() calls would never terminated when an application side
proxy was being used. Note we also skip over TLS handshake in this case,
as the application may have to do some proxy handshake before.

GMain: allow NULL context to g_source_attach

Documentation says it's fine and means default context, but the annotations
are missing (and thus bindings would complain).

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

GPollable*Stream: expose source methods to introspection

GSource has been introspectable for a while, so can stop skipping
methods of GPollableInputStream/OutputStream that return it.

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

gsignal.h: Remove trailing comma

We need <locale.h> in gsettingsschema.c now

for LC_TIME (since we moved a bunch of code over from gsettings.c).

Clear proxy address upon retry

The proxy address was not cleared between each attempt. That would lead
to leak or worse, trying to do the proxy handshake on the final
destination address. To make all this safer, I have regroup all the cleanup
where the iterations starts.

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

Silence compiler warnings

Fix several warnings for format strings (-Werror=format-security) and
set but unused variables (-Wunused-but-set-variable)

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

GKeyFile: Accept empty files

It already accepts files consisting solely of whitespace.

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

Document exactly what g_time_val_to_iso8601() produces

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

_g_dbus_get_machine_id(): check /etc/machine-id too

machine-id can be in /etc or in /var/lib/dbus.

[amended with slightly revised error handling -smcv]

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

Remove redefinition of typedef 'GSettingsSchemaSource'