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

GSettingsSchema: use our own linked lists

Switch from GSList for the list of schema sources to using our own
linked list type called GSettingsSchemaSource.

GSettingsSchema: alter our 'reverse' technology

Instead of building a reversed linked list by prepending in order and
then reversing it at the end, prepend in reverse by iterating backwards
through the directories (to get a list in-order when we're done).

move GSettingsSchemaKey to gsettingsschema.c

These functions no longer have anything to do with GSettings itself, so
they should not be in that file anymore.

GSettings still wants direct access to the GSettingsSchemaKey structure,
so put that one in gsettingsschema-internal.h.

GSettingsSchemaKey: store the GSettingsSchema*

g_settings_schema_key_init: take GSettingsSchema*

instead of of GSettings *

add internal g_settings_schema_get_name()

rename GSettingsKeyInfo to GSettingsSchemaKey

GSettingsKeyInfo: rename field 'key' to 'name'

remove GSettings* from GSettingsKeyInfo

This way GSettingsKeyInfo depends strictly on the information in the
schema.  Pass the GSettings* around separately where we need it.

store some extra info in GSettingsKeyInfo

GSettingsKeyInfo: drop unused variable

rename gsettingsschema.h to -internal.h

unGObjectify GSettingsSchema

Fix the fix for G*_TO_POINTER casts on 32 bits

The 'fix' from commit 16292dd753cb63f8ccb2267644aefbd2632dd52c broke the
build on 32 bit because it was missing the parentheses around "gint" / "guint"
in glib_gp[u]i_cast.

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

Updated Spanish translation

Updated Spanish translation

Fix doc formatting

Deal gracefully with NULL log domains

[notify] remove some rather bogus 'inline' use

[notify] add warning to gobjectnotifyqueue.c

Mention that this file gets installed, so people should probably not
modify it.

[notify] dispatch 'notify' directly if not frozen

Avoid the notify queue in the case that we're not already frozen.

[notify] add 'conditional' to _notify_queue_freeze

When the 'conditional' parameter is TRUE, the queue will only be frozen
(ie: have its freeze count increase by one) if it is already frozen.

This will allow us to avoid a freeze-notify-thaw in the case that we
just want to notify on a single property.

Another approach may have been to add an is_frozen() type call and avoid
even increasing the freeze count at all in this case.  Unfortunately,
I'm not totally sure what is the exact expected semantics of
simultaneous notifications in multiple threads and this may interact
badly with someone freezing or thawing in between our check and
emission.

[notify] Remove GObjectNotifyContext indirection

This silly abstraction is causing more trouble than it's worth -- just
use the relevant bits directly.

[notify] lift some logic out of _notify_queue_add

Lift the check-if-READABLE and redirect-target logic from out of
g_object_notify_queue_add() into its own function, get_notify_pspec().

Use that function at the site of our two calls to
g_object_notify_queue_add().

[notify] remove an obviously false comment