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'

start on the NEWS

g_settings_new_full(): more docs

add plugin example for GSettingsSchemaSource

Add gtk-doc for gsettingsschema{,source}

'make dist' fixes

settings-source test: Check returned error as well

Trivial typo fix

gsettings: add annotations for _new_full()

The path and backend arguments to g_settings_new_full are optional.

schema source: avoid introspection confusion

Any method that has its prefix'd argument as its first parameter will be
interpreted by introspection as a method. We don't want this, so we need
to swap the first two parameters.

gsettingsschema: Use the trusted parameter

Add test case and fix some bugs

Add the first test case for the schema source functionality and fix a
couple of bugs that got uncovered by that.

Add g_settings_schema_source_new_from_directory()

It is now possible for plugin loading systems to do the right thing.

Clarify docs/params for 'schema' vs 'schema id'

Clean up our parameter naming and documentation to draw a clearer line
between a schema and its identifier.

Add g_settings_new_full() taking GSettingsSchema

GSettings: add 'settings-schema' property

Ideally we'd have called this 'schema', but it is already used.

add boxed types for GSettingSchema{,Source}

Drop the 'schema_name' field of GSettings

This is strictly redundant now that we can get the ID from the schema
itself.  Its only other purpose was to get the schema name from the
set_property() call to the constructed() call and we can avoid that by
doing the schema lookup at the time of the property being set.

speak of 'schema id' rather than 'schema name'

Schemas are identified by id='' in the xml file, so we should use the
same language on the C and GObject property APIs.

GSettings: deprecate 'schema' property

This should have been called 'schema-name'.

drop the now-trivial g_settings_schema_new

Combine it into g_settings_constructed()

new public header: gsettingsschema.h

Expose some GSettingsSchemaSource and GSettingsSchema APIs.

These are mostly useless so far...

Add non-aborting g_settings_schema_source_lookup()

And rewrite g_settings_schema_new() in terms of it

GSettingSchemaSource: add refcounting