Revert "Optimize g_[s]list_free_full a bit"

This reverts commit 98b667d052b1274f80b8898a32d0753e9e2e5c1a.

The commit was not actually an optimization, since g_list_free is
pretty smart.

Added TLS PEM parser unit test

Rework PEM parsing to not be order dependent

Some valid PEM file would not work because the private key was put
before the certificate.

Docs: Correct the namespace for g_file_query_filesystem_info()

Updated glib-zip.in a bit further

Missed the bin/gsettings.exe part (corrected from bin/gsettings.)

utf8: annotate the end pointer in g_utf8_validate as out + allow-none

More gdbus-codegen fixed to build on non-Unix

https://bugzilla.gnome.org/show_bug.cgi?id=655148#c6

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

gtester: small Windows fix

This code was unconditionally present in the gtester Makefile:

test-nonrecursive: ${TEST_PROGS}

On Windows, our testcases are compiled with a .exe suffix.  That means
that if we had 'foo' in TEST_PROGS, running "make check" would depend on
'foo' (not foo.exe) being compiled.

We could bring the EXEEXT in here to fix that up, but gtester doesn't
work on Windows at all, so better to just disable it in that case.

GDateTime test: 1970 doesn't exist on Windows

It is not safe to call the system library mktime() function on Windows
with a date in the 70s.  Use 1990 instead.

g_format_size: fix on Windows

The long format that displays the exact number of bytes with separators
(ie: "123,456,789 bytes") uses the ' format modifier, which is
unsupported on Windows.  Disable that for now, until we come up with a
better solution.

g_format_size: avoid silly GString use

We were using a GString for the purpose of doing a single printf().  Do
g_strdup_printf() instead.

Update glib-zip.in

glocalfile: don't bother with fstype on win32

Fixes the broken build on Windows.

Merge branch 'gwakeup'

GWakeup: fix Windows build breakage

...from the attempt to make it private.

.gitignore the gwakeup-fallback testcase

Remove g_wakeup_* from glib.symbols

GWakeup: make it private API

Colin requests that we keep this one private for now.

Include it at each point of use (libglib, libgio, tests).

GWakeup: test fallback case

We need to test the case of eventfd in the libc but no kernel support.

In order to do that, we add a separate compile of the GWakeup testcase
that interposes an 'eventfd' symbol that always returns -1 with errno
set.  That will trigger the fallback case.

GWakeup: add signal safety note

Note that g_wakeup_signal() is safe to call from a UNIX signal handler
(since this is a likely place to want to call it from).

More fixes for non-Unix builds

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

Fix the build on non-Unix

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

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

GWakeup .gitignore fixes

gtk-doc GWakeup

GCancellable: port to GWakeup

port GMainContext to GWakeup

add a couple of testcases for GWakeup

Add GWakeup

GWakeup is a utility class to handle the cross-thread signalling needs
of GMainContext and GCancellable.  It may find some other users as well.

The desire here is to properly hide the implementation details in a
module which can be properly unit tested and used in GMainContext and
GCancellable without a rats nest of #ifdef.

g_unix_open_pipe: fix some bugs

Fix some bugs in the fallback case of g_unix_open_pipe:

  - close both halves of the pipe on error (not just one)

  - set the cloexec flag on both halves of the pipe (instead of settings
    it twice on one half)

Windows (mingw32) .gitignore additions

Docs: Fix tiny typo.

Update Windows READMEs

-Tell people about that libFFI is now needed.
-Tell people not to compile GLib in paths containing spaces.

gio/tests/socket: add some basic IPv4 and IPv6 tests

GSocket: fix connected state after async connect

socket->priv->connected was only being set if g_socket_connect()
succeeded right away; in the case where it returns G_IO_ERROR_PENDING,
it never got set. Fix that by having g_socket_check_connect_result()
set it on success.

update .gitignore

Explicitly unset the session bus address for the peer tests

Otherwise, we may run into trouble as opening a peer-to-peer
connection uses a socket client, which uses a proxy resolver
which may end up using gsettings, whose dconf backend may end
up using the session bus to talk to dconfd...

Properly bring up a session bus for application tests

As pointed out in bug 644601, session_bus_up() requires
us to set up environment variables for things to work.

Fix a typo

Updated Russian translation

[l10n] Updated Italian translation

GDesktopAppInfo: Add g_desktop_app_info_get_show_in()

Necessary for rebasing gnome-menus on top of GDesktopAppInfo.

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

glib.symbols: Update for expected ABI changes

glib-2.0.pc: Add -lrt to private libraries to assist static linking

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

Revert addition of g_key_file_has_key_full

Per IRC discussion, we can just ask bindings to use
g_key_file_get_value() to test for the existence of a key.

I left the "fixed" code in the source tree as static because it makes
more sense to me.

Don't use deprecated G_UNICODE_COMBINING_MARK

gitignore tweaks

Rename a few arguments

To fix header/function/doc-string mismatches.

Have gtk-doc ignore some #defines

We prefer to have gtk-doc pick up the function rather than the macro.

Replace @Varargs with @...

to make gtk-doc happy.

Remove duplicate 'struct real_pcre' declaration

The redundant forward declaration of 'struct real_pcre' before the
typedef was tripping up gtk-doc.  Remove it.

Add deprecation guard to G_UNICODE_COMBINING_MARK

...since it's deprecated now and documented as such.

Reword some docs comments to avoid 'Returns ...'

If gtk-doc sees 'Returns ...' written in the text of the documentation
for a macro, it thinks that you are talking about the return value.

Avoid doing that and use 'Returns:' explicitly if we mean to.

move 'Since:' tags down

gtk-doc wants the Since: tag to be the absolute last thing in the docs
comment.

Add g_ptr_array_new_full

Fixes bug #654450

Bump the version

Release 2.29.14

Bug 655076 - Our normalization code misses some Full_Composition_Exclusion=True.

gdbus-codegen: Add support for new org.gtk.GDBus.C.UnixFD annotation

Also add convenience _with_unix_fd_list variants to GDBusConnection,
GDBusProxy and GDBusMethodInvocation types to easily support this.

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

Document how to free return values from g_key_file_get_*_list

Makes the docs consistent with the string list getters.  Add
GOI annotations while we're at it.

Bug 655025 - #define G_UNICODE_SPACING_MARK G_UNICODE_COMBINING_MARK

GDBus: Don't convert GVariant type 'h' to gint

The code-generator already uses GVariant* so generated code didn't
really work at all. We want that instead of gint to avoid confusion
because a 'h' instance is an _index_ into a GUnixFDList, not a file
descriptor.

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

gdbus-codegen: Use G_TYPE_STRV/gchar** for GVariant type 'ao'

This is possible now that we have better support for object path
arrays, see

 http://git.gnome.org/browse/glib/commit/?id=19878998bc386db78614f1c92ff8524a81479c7b

Note that this breaks the ABI of generated code but since
gdbus-codegen(1) has never yet been in a stable GLib release, this is
fine.

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

g_atomic_int_add: document that the return value is new

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=654988
Reviewed-by: Matthias Clasen <mclasen@redhat.com>

Advise against using GPrivate

GStaticPrivate is really a lot better in almost every way.

GParam: try to avoid further invalid uses

In an attempt to avoid some potential future abuses of the GParamSpec
API, qualify the 'name' field of the structure as 'const' and add a
comment noting that it is an interned string.

This is a theoretical API break, but it will only ever result in
warnings -- and even then, only if you were already doing something
questionable.

Clean up some of the warnings that were caused internally in gparam.c
from these changes.

Add g_unicode_script_from_iso15924()

And adjust g_unicode_script_to_iso1592().

glocalfile: Fix compilation without USE_STATFS and USE_STATVFS

Bug 648271 - Add g_unicode_script_to_iso15924()

Add g_unicode_script_to_iso15924() and tests.

Updated Spanish translation

Updated Spanish translation

Change order of GFormatSizeFlags

David requested that I change the order of the flags.

Also, assign numerical values to the flags in the usual way.  This
wasn't a bug yet, but only by chance.

g_format_size: just use GString

Matthias wasn't too impressed by the homebrew stack-allocated string building I
was doing.

Switch to GString.

g_format_size: Return 'gchar' instead of 'char'

I meant to do that in the first place.

[ast, es, fr, nn] Update byte unit translations

The Asturian, French, Norwegian Nynorsk and Spanish translations
incorrectly translated "MB" and friends to "MiB" (french: "Mio"), etc.

This was in response to the incorrect use of "MB" in the (now
deprecated) g_format_size_for_display() function.

These strings are now used (correctly) in g_format_size(), so I have
updated the translations accordingly.

Additionally, the Norwegian Nynorsk translation was incorrectly
translating several larger units to "KiB", so that has been corrected as
well.

Change GLib size units policy

This commit changes GLib size units policy.  We now prefer SI units and
allow for use of proper IEC units where desired.

g_format_size_for_display() which incorrectly mixed IEC units with SI
suffixes is left unmodified, but has been deprecated.

g_format_size() has been introduced which uses SI units and suffixes.

g_format_size_full() has also been added which takes a flags argument to
allow for use of IEC units (with correct suffixes).  It also allows for
a "long format" output which includes the total number of bytes.  For
example: "238.5 MB (238,472,938 bytes)".

g_format_size_for_display: some internal renames

Rename the size constants from KILOBYTE to KIBIBYTE (etc.) since that's
what they really are.

This is a strictly internal change with no externally-visible effect in
terms of API or functionality.

gdbusintrospection: fix introspection annotations

g_dbus_interface_info_lookup_* were incorrectly considered
(transfer full) by introspected bindings, and this caused memory
corruptions.

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

Stop using deprecated g_unicode_canonical_decomposition()

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

GVariant: document avoiding g_variant_iter_loop

The choice between g_variant_iter_next() and g_variant_iter_loop() is a
bit confusing for some people.  Add a note to the documentation of
g_variant_iter_loop() to clarify that it should be avoided except in a
few specific cases.

GVariant: better support for object path arrays

Add G_VARIANT_TYPE_OBJECT_PATH_ARRAY along with accessor functions
g_variant_new_objv, g_variant_get_objv and g_variant_dup_objv.  Also add
support for '^ao' and '^a&o' format strings for g_variant_new() and
g_variant_get().

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

GSettings: don't abort on missing schemas

Give a g_critical instead.

Add some blacklisted mount directories

Another long-carried Debian patch.  No reason that we should not have
these few extra items in the (already long) upstream list.

wait longer in threadpool test case

If we are going to fail an assert, wait for a bit longer before doing so
(up to 5 seconds, if needed).

This is a long-standing Debian patch to fix build failures on really
slow machines.

post-release bump

2.29.12

Drop the warnings.sgml template

Document how to unset attributes

Unsetting won't work on all attributes. Currently, only
metadata attributes are supported.

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

Pass in NULL instead of g_cclosure_marshal_generic

NULL is now a shortcut for g_cclosure_marshal_generic, so avoid
referencing it directly.

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

Use a generic marshaller if one is not specified

Since g_cclosure_marshal_generic is always enabled, it makes
sense to always use that instead of using generated ones.

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

GSettings: remove key length restrictions

The key length now stands effectively unlimited at 1024 characters.

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

Fix doc formatting errors

Move GError docs inline and ditch template

Move Unicode docs inline, ditch template

Remove queue.sgml

Simplify Hangul Jamo decomposition

The algorithm is not copy/paste from Unicode anymore, but it's easy
enough to follow the logic.

Bug 654651 - Better g_unicode_canonical_decomposition()

Add g_unichar_fully_decompose().
Deprecate g_unicode_canonical_decomposition().

Avoid failing arguments to statfs() test on systems which use statvfs.

- move choice of statfs vs statvfs from gio/glocalfile.c to configure.ac
- if statvfs is the choice, then don't check number of arguments to statfs()
- use choice in gio/gunixmounts.c as well

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

gtester: Count inability to run a test in a binary as a failure

Previously, when a binary did not run a single test - for whatever
reason, from the binary not existing to the binary not using the gtester
framework - no failures were recorded. Now we record a non-successful
run of a binary that did not start any tests as a failure, too.

Note that this does not change the behavior of any binaries that exit
successfully or that report the start of any gtester test.

gtester: Return a failure exit code when a test fails

Previously, gtester used the testcase_fail_count as the number of tests
that failed in the latest run testcase, but then use that as the return
value of main().

Now we count the failed testcases of the whole run.

Updated Norwegian bokmål translation