gsocket: fix g_socket_condition_timed_wait() recovery after EINTR

After getting an EINTR, g_socket_condition_timed_wait() has to adjust
its timeout, but it was trying to convert from nanoseconds to
microseconds by multiplying by 1000 rather than dividing... Oops.

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

gtestutils: make the new assert messages more detailed

g_assert_true(), g_assert_false(), g_assert_null(), and
g_assert_nonnull() simply printed out the expression they were
checking, without any further explanation of what went wrong. (In
particular, "g_assert_true(x)" and "g_assert_false(x)" would both
print the same thing on failure.) Add a little bit more context.

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

gio/tests: fix race when generating code

There is a race condition in the makefile that can result in build failures like this in parallel builds:

| ./gdbus-test-codegen-generated.h:7:0: error: unterminated #ifndef
|  #ifndef __GDBUS_TEST_CODEGEN_GENERATED_H__

This is because a rule like this:

x.c x.h: prerequisites
@commands

doesn't consider x.c and x.h together. Instead, it expands to two rules, one to
generate x.c and one to generate x.h, which happen to run the same commands.  In
the worst case they execute in parallel, overwriting each other's output.

Signed-off-by: Ross Burton <ross.burton@intel.com>
https://bugzilla.gnome.org/show_bug.cgi?id=723616

subprocess: Init and clear the mutex

Fixes the leak of the GMutexImpl allocated inside the first call to g_mutex_lock()
on an uninitialized GMutex.

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

Annotate g_application_add_main_option_entries

g_simple_async_result_is_valid: fix for NULL source tag

If a GSimpleAsyncResult has a NULL source tag, allow it to compare
valid to a non-NULL source tag in g_simple_async_result_is_valid(), to
simplify cases where, eg, g_simple_async_result_new() and
g_simple_async_result_report_error_in_idle() are both used.

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

Fix the build on FreeBSD by replacing EAI_NODATA with EAI_NONAME.

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

win32: fixup lib.exe invocation

We have a configure.ac check for lib.exe that attempts to enable
creation of .lib files for our 5 public libraries.  That has been broken
for a long time for two reasons:

 1) the Makefiles hardcode 'lib' instead of 'lib.exe'

 2) we dropped generation of .def files quite some time ago (except for
    in gthread where we have the two-symbol file under version control)

Add new rules for creating .def files from dumpbin.exe (which you should
have if you have lib.exe) and fix the .lib rules to use lib.exe.

Add a bit of $(AM_V_GEN) all around, as well.

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

only '#pragma GCC' outside of functions

Don't use #pragma GCC inside of function scope.

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

gio: don't accept nonstandard IPv4 "numbers-and-dots" addresses

In addition to the standard "192.168.1.1" format, there are numerous
legacy IPv4 address formats (such as "192.168.257",
"0xc0.0xa8.0x01.0x01", "0300.0250.0001.0001", "3232235777", and
"0xc0a80101"). However, none of these forms are ever used any more
except in phishing attempts. GLib wasn't supposed to be accepting
these addresses (neither g_hostname_is_ip_address() nor
g_inet_address_new_from_string() recognizes them), but getaddrinfo()
accepts them, and so the parts of gio that use getaddrinfo()
accidentally did accept those formats.

Fix GNetworkAddress and GResolver to reject these address formats.

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

gobject: re-allow finalization from constructor()

Although returning NULL from constructor is strongly discouraged, some
old libraries need to keep doing it for ABI-compatibility reasons.
Given this, it's rude to forbid finalization from within
constructor(), since it would otherwise work correctly now anyway (and
the critical when returning NULL should discourage any new uses of
returning NULL from constructor()).

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

Fix g_socket_get_available() with TCP on Windows

Windows needs a special inefficient hack to implement
g_socket_get_available() correctly for UDP sockets, but that hack
isn't needed for TCP, and in fact, might give the wrong answer in that
case. Fix it to only use the hack with UDP.

Also, fix that case to handle non-blocking sockets as well.

And add a test case for g_socket_get_available() with TCP.

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

docs: Remove a few trailing <literal>s

docs: Remove some unneeded decorations

@var is not expanded inside literal `` blocks.
Just remove those @ characters.

docs: let go of &ast;

Since we are no longer using sgml mode, using /&ast; &ast;/ to
escape block comments inside examples does not work anymore.
Switch to using line comments with //

configure.ac: tweak inotify check

Our check for inotify_init1() being defined is broken.  We happily
declare that inotify is supported, even if the check fails.

This was originally intended to check for inotify_init1 in the libc so
that we could fall back to inotify_init if it was not yet defined.

FreeBSD has a libinotify that emulates the inotify API via kqueue.  It
installs a <sys/inotify.h> header and requires linking to -linotify.  We
don't want to falsely detect working inotify in this case.

Treat the lack of inotify_init1() in the libc as a lack of inotify
support.  This requires only a new libc -- we still support old kernels:
in the case that inotify1_init() fails, we fall back to inotify_init().

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

gtask: fix trivial doc typo

g_test_run: return 0 if all tests are skipped in TAP mode

Exit status 77 is special to Automake's default test driver, but is
treated as an error by TAP.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=724124
Reviewed-by: Dan Winship <danw>

glib/tests/collate.c: run to completion when skipping all tests

Otherwise, we don't produce valid TAP output, and fail with:

    ERROR: collate - missing test plan

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=724124
Reviewed-by: Dan Winship <danw>

gsocketconnection: Document closing connections with g_io_stream_close()

It’s not enough to close a connection by calling g_input_stream_close()
and g_output_stream_close() on its two substreams: to close the
underlying socket, one must use g_io_stream_close(). Document that.

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

gsocketservice: Clarify g_socket_service_stop() documentation

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

g_child_watch_source_new: POSIX pid must be positive

If we used a non-positive pid, we'd call waitpid(that_pid, ...)
which is exactly the situation this function can't deal with.

On Windows, GPid is a HANDLE (pointer), so I don't think the same thing
applies.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=723743
Reviewed-by: Ryan Lortie

Be more clear that g_return_if_fail is undefined behaviour

In particular, it is not incorrect to g_return_if_fail (..., FALSE)
in a function returning a "success" gboolean and a GError: "failure to
meet the preconditions is an error" takes precedence over the
GError documentation's guarantee that the error will be set on failure.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=660809
Reviewed-by: Emmanuele Bassi

po/nb.po: Fix mismatched translation

Missing '\n' at the end of the string broke building under CI.

Updated Norwegian bokmål translation

Updated Indonesian translation

gsubprocess: Fix ‘Since’ lines in documentation to read 2.40

GSubprocess was introduced in GLib 2.40, not 2.36.

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

gsubprocess: Fix a broken link in the documentation

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

Updated Norwegian bokmål translation

[l10n] Updated Italian translation.

Fix a misformatting in GVariant docs

The + at the beginning of the line was misinterpreted
as markdown for a list.

Docs: Drop entities, switch away from sgml mode

Since all element markup is now gone from the doc comments,
we can turn off the gtk-doc sgml mode, which means that from
now on, docbook markup is no longer allowed in doc comments.

To make this possible, we have to replace all remaining
entities in doc comments by their replacement text, &amp; -> &
and so on.

Convert remaining examples to links

REmove another table

Remove some informalexamples

Another stray <programlisting>

Another stray litreal

Strip out a remaining programlisting

Convert another table to a list

Convert more xincluded examples to external links

Docs: Remove an example tag that has snuck back in

Docs: convert another table to a list

Convert GVariantType table to a list

Remove a new literal tag that has crept in

Remove a few leftover <simplelist>s

Remove a stray <em> tag

Remove tables from GFileAttribute docs

Eradicate links and xrefs

These are all replaced by markdown ref links.

Updated Spanish translation

Updated Galician translations

Remove a few overlooked literals

Stop using replaceable tags

Stop using <para> for ids

Instead, use the id support in markdown headings.

Updated Brazilian Portuguese translation

Docs: replace <literal> by `

Docs: use quotes instead of firstterm

Docs: don't use option tags

Stop using starttag elements

Use markdown for images

Use a code block instead of <screen>

GApplication: parse command line options

Add support for parsing command line options with GApplication.

You can add GOptionGroup and GOptionEntry using two new APIs:
g_application_add_option_group() and
g_application_add_main_option_entries().

Also add a "handle-local-options" signal that allows handling of
commandline arguments in the local process without having to override
local_command_line.

As a special feature, you can have a %NULL @arg_data in a GOptionEntry
which will cause the argument to be stored in a GVariantDict.  This
dictionary is available for inspection and modification by the
"handle-local-options" signal and can be forwarded to the primary
instance in cases of command line invocation (where it can be fetched
using g_application_command_line_get_options()).

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

GOption: fix bug in strv mode

We are a bit too aggressive about freeing memory in strv mode.  Only
free it in the case that we actually set the pointer to NULL.

Uncovered by the GApplication tests.

Updated Thai translation

Convert external links to markdown syntax

Don't use <filename> in docs

Switch to simpler markdown, `foo`.

Don't use <envar> in docs

Switch to simpler markdown, `foo`.

updated kn.po

gvariant: Document the need to cast varargs when constructing GVariants

Slightly expand on the documentation about casting varargs when
constructing GVariants, and link to it from all the functions where it’s
a necessary consideration.

Add an example of passing flags to a ‘t’ type variable (guint64).
Assuming the flags enum does not have many members, the flag variable
will be 32 bits wide, and needs an explicit cast to be passed into
g_variant_new() as a 64-bit value.

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

return_if_fail vs. return_val_if_fail fix

gobject: box GVariantDict

We will want to use this in GApplication for a signal and a property.

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

add GVariantDict

...the long-requested mutable dictionary helper for GVariant.

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

Bump version

2.39.4

Updates

Fix a typo

gmain: Note that g_source_destroy() can be called multiple times

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

Updated Traditional Chinese translation(Hong Kong and Taiwan)

tests: Remove a compiler warning

array: Remove a compiler warning

The GRealPtrArray variable is not necessary: we're accessing only public
fields of GPtrArray.

array: Fix compilation

gio: extend the system_internal lists with BSD paths

GVariant: Convert docs to markdown

Specifically, convert the sections to markdown syntax.

Don't use the varname tag

This is just an environment variable, after all.

gmain: Convert docs to markdown

Specifically, convert sections to markdown syntax and
drop all the para tags.

Don't use the quote tag

It was only used in two places, and we can easily do without.

Remove an unnecessary paragraph

Make gtk-doc find another symbol

GTree: formatting fixes

gstrfuncs: Fix up gtk-doc warnings

grand: formatting cleanups

Formatting cleanups

Avoid another gtk-doc warning

Avoid a gtk-doc warning

GIOChannel: Move some docs where they are looked for

Avoid some gtk-doc warnings

Drop another use of xinclude

GArray: Documentation cleanups

Docs: Avoid a 'returns' at the beginning of the line

This confuses gtk-doc.

Add some more deprecation guards to shut up gtk-doc

Drop use of xinclude in GTestDBus docs