doc: various improvements

- g_subprocess_launcher_spawn() and spawnv(): there is no other way
  AFAIK to create a GSubprocess from a launcher. So these
  functions are not "convenience helper".

- annotate optional arguments for g_shell_parse_argv().

- other trivial fix

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

GActionEntry: one more docs fix

Hyperlink "GVariant text format".

GActionEntry: improve documentation

Document that giving %NULL for the activate handler is supported since
GLib 2.40.  We documented this on GSimpleAction itself (where the
default handler functionality is implemented) but expecting the user to
dig that up is asking a bit much.

Also, add some more explicit documentation about the conditions under
which each field is expected to be filled in.

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

Change gio/data-to-c.pl to /usr/bin/env perl.

On FreeBSD perl gets installed in /usr/local, And a compat symlink in
/usr/bin. However this symlink might go away in the future.

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

gcredentials: add NetBSD support

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

GNotification: add priority

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

gobject: move _g_type_debug_flags from gtype.h to gtype-private.h

_g_type_debug_flags is used internally to suport GOBJECT_DEBUG
environment variable. it should never be exposed to the outside.

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

glib/tests/mainloop: Acquire the context while asserting its state

The iteration methods presume that the context is acquired.

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

A simple test for polling on a file-like descriptor

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

giochannel: avoid setting uninitialised length

Our internal call to g_io_channel_read_line_backend() may return
G_IO_STATUS_ERROR, in which case two things will be true:

 - the GError will have been set (if appropriate)

 - the &got_length return value may not have been set

Since it's our convention to leave 'out' parameters untouched in
exception cases, this is perfectly fine.  Unfortunately,
g_io_channel_read_line(), in wrapping this internal function, always
promotes the length parameter, even in the case of error.

Stop doing that in order to avoid overwriting the callers's variable
with junk in the error case.

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

GDateTime: Add guards to g_date_time_new()

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

docs: Fix formatting of g_alloca documentation.

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

Convert multiple #if/#endif to single #if/#elif/#endif

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

gmessages: make g_assert_warning compiler friendly

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

all: remove use of 'register' keyword

We should have done this a decade ago...

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

unicode: Update to unicode 7.0.0

See bug https://bugzilla.gnome.org/show_bug.cgi?id=731929.

unicode: Switch compose_second_single to gunichar

This will be required for the update to unicode 7.0.0.

unicode: Move gscripttable.h generation into main script

So we just have to run one script when updating the unicode data, not two.

unicode: Simplify width table generation

Move width table generation into the gen-unicode-tables.pl script. This makes
updating the tables automatic without the previously required manual editing
required to insert the tables in the right place of the source code.

GObject: tweak property deprecation warnings

Don't emit property deprecation warnings for construct properties that
are being set to their default value during construction, but _do_ emit
them in all cases when the property was explicitly given to
g_object_new().

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

GTimeZone: remove some dead code

Remove code that parsed out and stored the unused isstd and isgmt
fields.

gtype: Remove unused header

GType does not use GBSearchArray any more.

Revert "gsettings: remove long-deprecated "schema" property"

This reverts commit cf9b162e0d0defaff2ad5c85aeaf7af0899ad22f.

It turns out that there are still a very large number of users of this
API.

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

GObject: warn on use of deprecated properties

By default G_PARAM_DEPRECATED means absolutely nothing.  We only emit a
warning if G_ENABLE_DIAGNOSTIC is set to '1' and then, only on sets.

Turn the logic on its head: emit the warning by default, unless
G_ENABLE_DIAGNOSTIC is set to 0.  In order to avoid a torrent of output, only
emit a warning once per property name.

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

gvalue: New g_value_init_from_instance

Used for the commonly used case (in signal emission) where we
initialize and set a GValue for an instance

Includes a fast-path for GObject

Overall makes it 6 times faster than the previous combination
of g_value_init + g_value_set_instance

Makes signal emission around 10% faster

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

gvariant tests: workaround libc/compiler "issue"

memcmp() is declared by glibc as follows:

  /* Compare N bytes of S1 and S2.  */
  extern int memcmp (const void *__s1, const void *__s2, size_t __n)
       __THROW __attribute_pure__ __nonnull ((1, 2));

despite the fact that it is valid to call it with a null pointer if the
size is zero.

gcc 4.9.0 contains a new optimisation that sees that we pass a pointer
to this function and concludes that it certainly must not be null,
removing a later check and thereby causing a crash.

We protect the invocation of memcmp() with a condition to prevent gcc
from making this false assumption (arguably under wrong advice from
glibc).

gsettings: remove long-deprecated "schema" property

This property has been deprecated for three years after only having
existed for one.  We've wanted to reuse the name for all that time, so
let's try to actually remove it now and see if we can get away with it.

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

gsettings test: stop testing long-deprecated API

We want to try taking this away now...

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

docs: fix typos

Bump version

2.41.1

gsocketlistener: Don’t explicitly close sockets on finalisation

Instead of closing the sockets explicitly, let them close themselves
when their final reference is dropped. This makes use of
g_socket_listener_add_socket() more natural.

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

gsocket: Document that g_socket_create_source() holds a socket ref

This is quite important, as it means you can safely let the GSocket drop
out of scope while maintaining a reference to the GSource, and the
socket will remain open. That means fewer closure structures, simpler
code, and fewer allocations.

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

gsignal: Add an example to the g_signal_connect_swapped() documentation

The precise behaviour of and motivation behind swapping parameters was
not entirely clear before.

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

Return folder as icon for directories

'folder' is the name of the folder icon in the incon naming spec,
and the Adwaita icon theme doesn't include an inode-directory icon.
This fixes folders appearing as generic file in the file chooser.

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

gtestdbus: Don’t close stdout for dbus-daemon

The stdout FD passed to dbus-daemon is propagated to all its child
processes, such as service activated processes. If we close the FD after
reading the bus address from the daemon, any child process which
subsequently writes to stdout (e.g. for logging) will get a SIGPIPE and
explode.

Instead of closing the stdout FD immediately after dbus-daemon has
spawned, keep it open until the daemon is killed.

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

gclosure: Remove an unused alloca() and assignment

Coverity issue: #1159511

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

goption: Remove an unused assignment

Coverity issue: #1159514

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

gwakeup: Clarify buffer sizing in g_wakeup_signal()

The code in g_wakeup_signal() is currently correct: it writes a 64-bit
counter increment value if the FD is an eventfd, and writes an arbitrary
8-bit value if using a normal pipe.

However, the reasoning behind these buffer sizes is not clear, and the
mismatch between the allocated buffer size and the length passed to
write() in the pipe case could be mistaken for a bug.

Coverity issue: #1159490

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

docs: Correct param to interface's default_init()

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

docs: fix a broken sentence

configure.ac: fix check for futex and eventfd

We were using AC_LANG_PROGRAM to build a program to test for our ability
to call syscall (__NR_futex, ...);.  This macro adds "main () { ... }"
around the provided code segment automatically.  The provided code
segment provided its own main() function, however.

The result looked something like:

int main (void) {
  int main (void) {
    ...
  }
}

which worked on GCC, but not on clang.

Let's fix that.  Let's fix the same mistake copied over for eventfd()
detection while we're at it.

gatomic: disable GCC extension atomics for clang

clang defines the macro that we use to test for GCC's extension support
for C11 atomics, but doesn't define the extension in the same way.
Check for clang and disable the macros again if we find it.

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

Try to make gobject.py work with both pythons

This change was suggested on the fedora test list, and has been
reported to work.

Prevent an invalid @CARBON_LIBS@ from appearing in the .pc files

Fixes static linking when using the glib pkgconfig file.
Regression was introduced in commit 8d037c6 (bz #668152)

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

gio/tests/socket: fix minor memleak

gparam: Remove unnecessary conditionals around preconditions

These cause a static analyzer to think we're trying to actually handle
them being NULL, which is not the case.  They both must not be NULL,
period.

No idea why the code was like this originally.

Reviewed by mclasen on IRC.

gbookmarkfile: Cleaner error handling code to pacify static analysis

A static analyzer flagged the g_file_get_contents() call as not
checking its return value.  While the code here is actually correct,
it's verbose at best.

I think the "goto out + cleanup" code style is substantially cleaner,
less error prone, and easier to read.  It also will pacify the static
analyzer.

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

gparam: change value of G_PARAM_EXPLICIT_NOTIFY

GParamSpec has a possibility of user-introduced flags, and we didn't
respect that with the addition of _EXPLICIT_NOTIFY.

Change the documentation for the maximum number of user flags to 10,
just to pick a somewhat random number.  The documentation here was never
correct anyway -- it previously claimed that as many as 38 flags were
possible.

Meanwhile, move G_PARAM_EXPLICIT_NOTIFY next to _DEPRECATED in order to
avoid conflicts with low-numbered user flags (which are in use by at
least evolution-data-server).

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

gnextstepsettingsbackend: Use same token for function-call and its availability

Availability of the g_nextstep_settings_backend_get_type() prototype
is controlled by HAVE_COCOA in gsettingsbackendinternal.h and the
actual implemenation by OS_COCOA in Makefile.am. Therefore, the
giomodule.c call to that function should also be protected by a COCOA
token rather than an CARBON token (cocoa and carbon are independent
autoconf tests).

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

assert-msg-test: Explicitly quit gdb session to avoid zombie

gdb is run in batch mode, and can leave leave the program being
executed/debugged running when the batchfile is finished. Explicitly
"quit"ing the subprocess prevents it from leaving the stray subprocess
when gdb finishes.

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

gtype: remove interface-after-init exceptions

A year ago, we tried to remove support for adding interfaces on
already-initialised types.  There were problems with the C++ and C#
bindings at the time, so we added exceptions to give them a bit more
time to catch up.

It's already one cycle after when these exceptions were planned to be
removed, so let's take them out now.

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

MSVC 2010+ Projects: Update "Installation" Process

Currently, due to the way that Visual Studio 2010+ projects are handled,
the "install" project does not re-build upon changes to the sources, as it
does not believe that its dependencies have changed, although the changed
sources are automatically recompiled.  This means that if a part or more
of the solution does not build, or if the sources need some other fixes
or enhancements, the up-to-date build is not copied automatically, which
can be misleading.

Improve on the situation by forcing the "install" project to trigger its
rebuild, so that the updated binaries can be copied.  This does trigger an
MSBuild warning, but having that warning is way better than not having an
up-to-date build, especially during testing and development.

trivial: docs: add some missing commas

docs: warn about installing properties after init

Leave ourselves a little wiggle room: if people install properties after
initialisation then we reserve the right to handle that in a way that
may not be threadsafe.

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

Revert "GObject: prevent installing properties after init"

This reverts commit ddb0ce14215cd62c7a2497d6cf9f2ea63c40ebb5.

Conflicts:
gobject/gobject.c

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

gthread: use inline keyword for _get_impl() functions

Give compiler a hint that these should be inlined,
which doesn't seem to happen by default with -O2.
Yields 5% speedup in artificial benchmarks, and
1% speedup in a real-world test case doing a lot
of mutex locking and unlocking.

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

gatomic: use GCC C11-style atomics, if available

GCC does not yet support ISO C11 atomic operations, but it has
compatible versions available as an extension.  Use these for load and
store if they are available in order to avoid emitting a hard fence
instruction (since in many cases, we do not need it -- on x86, for
example).

For now we use the fully seqentially-consistent memory model, since
these APIs are documented rather explicitly: "This call acts as a full
compiler and hardware memory barrier".

In the future we can consider introducing new APIs for the more relaxed
memory models, if they are available (or fall back to stricter ones
otherwise).

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

gatomic: whitespace fixes

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

gatomic: fix typo in deprecation attribute

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

gtype: guard uses of new fundamental type check

g_type_is_fundamentally_a (see bug 730984) is a new API/ABI and is
marked with a version macro.  We should therefore avoid its
unconditional use from G_IS_OBJECT() and G_IS_PARAM_SPEC() which are
APIs that have been around for a long time.

This prevents deprecation warnings from being emitted when these
functions are used with an older GLIB_VERSION_MAX_ALLOWED and also
prevents linking to the new ABI in that case (so that it's possible to
use the resulting binary with an older version of GLib).

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

test: add test for explicit-notify properties

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

gobject: add 'explicit notify' GParamSpec flag

Add a flag to prevent the automatic emission of the "notify" signal
during g_object_set_property().

If this flag is set then the class must explicitly emit the notify
for themselves.  This is already standard practice on most classes, but
we cannot simply remove the existing behaviour because there are surely
many cases where it is needed.

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

Add a test for GMenuAttributeIter

valgrind.h: Update to latest upstream version

Avoid crashes in g_type_free_instance() on mingw64.

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

Support Building Using Visual Studio 2013

Like the Visual Studio 2012 project files, the Visual Studio 2013 files are
largely the same as the Visual Studio 2010 project files, so support
Visual Studio 2013 by updating the autotools scripts that is used for
Visual Studio 2012.  This means that project files for Visual Studio 2012
and Visual Studio 2013 can be maintained by simply maintaining the Visual
Studio 2010 project files, adding minimal maintenance overhead.

docs: Add missing language annotations

docs: GType documentation cleanup

Still some markup left.

docs: Replace <function> XML tag with back ticks

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

docs: Replace <structname> tags with back ticks

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

docs: Add an index for 2.40 api

Minor documentation additions and corrections

Going for 100%.

docs: Fill in the gspawn long description

docs: Fill in gshell long description

Document g_signal_handlers_destroy

docs: Fix up varargs documentation

gtk-doc expects varargs to be documented in a specific way,
otherwise it complains.

docs: Add some missing apis

docs: Add a 2.42 api index

GParamSpec: Use new fundamental instance check

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

gobject: Use fast fundamental instance type check

Speeds up g_object_ref/_unref by 50%-65% (i.e. takes 60-65% of the time
it used to take).

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

gtype: Add check for fundamental instance type

When checking whether an instance is of a given fundamental type (such
as G_TYPE_OBJECT), we can avoid over 60%+ of the cost of checking types.

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

Fix the closure test in continuous

This test has the same problem as the mapping-test - it uses
SIGUSR1 without checking the signal mask. Apply the same fix
here.

Fix the mapping test in continuous

It turns out that due to a recent gdm change, the inherited
signal mask has SIGUSR1 blocked - which is bad news for
tests using SIGUSR1. Fix the test by explicitly checking the
signal mask before using SIGUSR1.

gconvert: mention that the g_convert len should be in bytes

Some charsets have each char with more than one byte, make it clear that
the length should be in bytes

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

Add more debug spew to the mapping-test

gvariant: Clarify type and format strings in the docs

'@' and '&' are only used in format strings as prefixes to type
strings and not to full format strings.

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

mapping-test: Add debug spew

This might help figuring out why the test is failing in continuous.

gio/tests/network-address: fix when no network is available

GResolver doesn't do full validation of its inputs, so in some of
these tests, the fact that we were getting back
G_RESOLVER_ERROR_NOT_FOUND is because the junk string was getting
passed to an upstream DNS resolver, which returned NXDOMAIN. But if
there's no network on the machine then we'd get
G_RESOLVER_ERROR_INTERNAL instead in that case.

GLib 2.41.0

gio tests: rm xdgconfighome on 'make clean'

Revert "gvariant: Fix confusion between type and format strings in the docs"

This reverts commit fca1b83be74c9840037cfee84015fd8460cc33f9.

The original documentation was correct.

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

gtestutils: Error out if /dev/null stdin redirection fails

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

gio: port annotations from the Vala metadata.

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

config.h.win32.in: Define _WIN32_WINNT Conditionally

This is done so that _WIN32_WINNT may be overridden in the project files,
if needed, so that one can access the Vista+ (or so) Windows APIs easier
by using "preprocessor defines" (or so) in the Visual C++ project files.

Try to fix mapping-test

The mapping-test is failing under gnome-continuous. I suspect this
is simply due to running many tests in parallel, and mapping-test
being racy. Replace the blind sleep by signals, to avoid the
races.

GDBusError enum: Remove trailing comma.

This is helpful to people using the g++ --pedantic option.

gobject: unref unused class

If g_type_class_ref() returns a class which is not
a GObjectClass we need to unref it before return in
object_interface_check_properties().

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

Trivial: fix a guard comment

The conventional comment for the #endif at the bottom of the
header was not matching the #ifdef at the top in glib-private.h.
Fix that.

gdbus-tool: Remove dead variables

Coverity issues: #1159447, #1159446

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

gthread: Fix use of a local variable after it went out of scope

This could theoretically cause problems, although in practice we would
have seen them by now (the bug was introduced in 2012).

Coverity issue: #1159486

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