Improve test coverage in gobject/

Lines:          6631    8862    74.8 %
Functions:      747     893     83.7 %

GIcon: NULLify the `type' out param in the sync methods too

Both g_[file|bytes]_icon_load() leave the `type' out parameter
untouched, while the async methods g_[file|bytes]_icon_load_finish()
always set it to NULL.

For consistency's sake NULLify it in the sync methods too.

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

Updated Slovenian translation

updated kn translations

Update Visual Studio Project Sheets

"Install" the newly-introduced gio/gbytesicon.h...

GLib 2.37.1

More test fixes for builddir != srcdir

Fix make check with builddir != srcdir

This broke when the tests were converted to be installable.
My apologies.

gio/tests: Make gdbus-proxy-well-known-name handle srcdir != builddir

Fix yet more test regressions...

One more broken test....

More srcdir != destdir tests fallout

tests: fix a srcdir vs. destdir issue

The target file is a script, so it'll always be in SRCDIR for the uninstalled
case.  Just look there and avoid the libtool trickery.

Updated Odia Translation.

Tamil Translation Updated

Fix the GObject Visual Studio Projects

Update G_LOG_DOMAIN to be "GLib-GObject" so that we are consistent with
the autotools builds, and that tests expecting the log domain to be
"GLib-GObject" would not fail.

Fix the GLib Visual Studio Projects

Define the G_LOG_DOMAIN of the GLib DLL as "GLib", because:
-This makes it consistent with the autotools builds
-Some tests expect the log domain to be "GLib"

Update config.h.win32(.in)

Make the entries of config.h.win32(.in) consistent with the entries
that are generated from the autotools build (config.h.in).

Avoid a segfault in gdbus tool

When the interface name is invalid, we don't get an error
back from g_dbus_connection_call_sync.

build: Fix usage of %.test again

We actually need the first dependency because it includes the
final executable name.  Rather, fix the original bug by using
the variable $(EXEEXT).

tests: Drop unnecessary % from .test pattern match rule

On Windows, the executables will have .exe, so this won't
match.  Furthermore, they aren't actually dependent on the
executable to build.

Updated Spanish translation

Assamese translation updated

Updated Gujarati Translations

G_GNUC_FORMAT: documentation error

Attribute should be placed just before the semicolon, not after.
As can be seen in the example.

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

spelling fixes of 'runtine' and 'adresses' in cross.xml and running.xml, respectively

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

gsignal: remove some pointless locking

We previously hold a lock in the loop that collects the arguments for
g_signal_emit(), which we drop before calling into the argument
collection functions and reacquire again at the bottom of the loop (ie:
one release/acquire pair for each argument collected).  To make matters
worse, the lock is just released again after the loop.

Presumably that was done to protect the access to the parameter array,
but it's pretty unlikely that this is needed because the only way it
changes is if the signal is unloaded.  That only happens when unloading
types which is quite unlikely to happen while we are emitting on an
instance of that type (and, as an aside, never happens anymore anyway).

If we move the unlock below the loop up above it and remove the
acquire/release pair from the loop, we improve performance in the new
arg-collecting performance tests by ~15% (more like ~18% in the case
where we only emit to one handler -- where argument collection dominates
more).

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

performance test: add signal test with args

Add a signal that has some typical arguments (a uint and a pointer)
since all of the other signal performance tests are for signals with no
args.

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

performance test: share some code

The handled and unhandled cases share the same data and _run()
functions.  Refactor into a common section.

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

gunixconnection: added ngettext to support plural forms

While those strings ("Expecting 1 control message, got %d" and
"Expecting one fd, but got %d\n") have same singular/plural form
in english, it is not necessarily the case in other languages.

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

GLocalFileOutputStream: Further deduplicate error code path

Just code cleanup.

https://bugzilla.gnome.org/699959

GLocalFileOutputStream: Deduplicate stream creation code

Lots of copy/paste of the error handling path, let's deduplicate
so I can sanely patch this code later.

https://bugzilla.gnome.org/699959

gdbus-peer: Drop some usage of g_thread_yield()

It's a recipe for race conditions and error; on some hardware
architectures one thread isn't guaranteed to see the results
of writes from another thread without a cache flush.

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

glib/tests/cond: Fix race condition

The producer could push two values with the first one being
lost.  Fix this by blocking the producer until a consumer
reads.

gtestutils: Use stdio rather than write() to be Windows-friendly

Windows doesn't define STDOUT_FILENO and STDERR_FILENO, and they're
not even guaranteed to be 1 and 2. So just use stdio instead. Also fix
a counting error. Pointed out on gtk-devel-list.

Timeout the test if dbus sevice has not appeared in due time.

The test /gdbus/connection/large_message waits for a dbus name to appear.
The dbus name is created by a another process executed in the background.
If for some reason this fails, the test will likely wait forever.
This will avoid this situation by making the test fail if the dbus service
has not appeared after 10 seconds.

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

po: update for quote changes

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

Use 'dumb quotes' rather than `really dumb quotes'

Back in the far-off twentieth century, it was normal on unix
workstations for U+0060 GRAVE ACCENT to be drawn as "‛" and for U+0027
APOSTROPHE to be drawn as "’". This led to the convention of using
them as poor-man's ‛smart quotes’ in ASCII-only text.

However, "'" is now universally drawn as a vertical line, and "`" at a
45-degree angle, making them an `odd couple' when used together.

Unfortunately, there are lots of very old strings in glib, and also
lots of new strings in which people have kept up the old tradition,
perhaps entirely unaware that it used to not look stupid.

Fix this by just using 'dumb quotes' everywhere.

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

tests: fix a few recently-broken tests

gio/tests: Fix gdbus-connection when run from "make check"

Convert more tests to installed tests

This makes the tests in tests installed, except for
the performance tests.

Convert more tests to installed tests

gio/tests: gsettings is a developer-only test

It tries to run glib-compile-schemas and glib-mkenums, which
we won't have in the runtime tree.

Anyways it's kind of a dumb test since the best test for
compilation tools is...compiling things, which we already
do frequently.

gmessages: Fix up testing commit to handle fatal masquerading

This makes glib/tests/testing pass again.

gio/tests: Disable desktop-app-info test in installed mode for now

It wants a writable copy of the applications, and appears to
depend on a prior execution of "mimeapps".  This will take
a bit of work to untangle.

gio/tests: Make gdbus-peer work in installed mode

There's no /etc/hosts in gnome-ostree...

gio/tests: Fix gapplication test in installed mode

gio/tests: Some more fixes for installed tests

file passes now, appinfo needs a bit more work, but is getting
there.

gio/tests: Deduplicate Makefile.am a bit

We can just add all the static test data to EXTRA_DIST.

gtestutils: Ensure test subprocesses don't dump core

Since we expect them to crash, let's not spam the system
core dump collection (systemd, abrt).  At the moment
systemd is not very robust against programs crashing
in loops.

Instead of aborting, we exit(1).

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

Updated Norwegian bokmål translation

Convert remaining gio tests to installed

Convert some gio tests to installed tests

Make gobject tests installable

This makes the gobject tests run as part of the ostree integration
tests.

[l10n] Updated Italian translation.

Add --enable-installed-tests configure option

See https://live.gnome.org/GnomeGoals/InstalledTests for more
information.

The tests now support being run both uninstalled and installed, so
'make check' works for those who want it.  For tests which need data
files, the way this works is they look in the compiled in value of
SRCDIR by default, and the generated tests use "env G_TEST_DATA=" to
override that.

This patch only converts glib/tests for now; if this patch looks good,
I'll do the rest of the tests.

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

glib/tests: Drop unnecessary SRCDIR definitions

It's always going to be defined, we don't support compiling these
tests "by hand".  This is a cleanup commit for the installed tests
work.

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

glib/tests: Use explicit file listings

This is a preparatory commit for installed tests work; having things
cleanly categorized helps.

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

glib/tests/mappedfile: Copy test file before writing to it

This way we're *always* testing writability rather than only sometimes
doing it in the source directory.

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

Added Aragonese translation

m4macros/glib-gettext.m4: Don't use AC_HEADER_STDC

AC_HEADER_STDC is obsolete (pre-C89 headers are no longer an issue),
and the resulting STDC_HEADERS define is anyway not used.

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

updated kn translations

tests/slice: use G_GNUC_BEGIN_IGNORE_DEPRECATIONS

rather than implementing it by hand

gtestutils: use g_clear_pointer

gtestutils: deprecate g_test_trap_fork()

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

tests: port from g_test_trap_subprocess() to g_test_trap_fork()

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

tests/protocol: redo a bit

Rather than overloading --verbose, just skip the tests that aren't
supposed to be run in the parent process (so that if you do run the
toplevel test with --verbose, it doesn't immediately error out).

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

gtestutils: add g_test_trap_subprocess()

g_test_trap_fork() doesn't work on Windows and is potentially flaky on
unix anyway given the fork-but-don't-exec. Replace it with
g_test_trap_subprocess(), which re-spawns the same program with
arguments telling it to run a specific (otherwise-ignored) test case.

Make the existing g_test_trap_fork() unit tests be unix-only (they
never passed on Windows anyway), and add a parallel set of
g_test_trap_subprocess() tests.

Also fix the logic of gtestutils's "-p" argument (which is used by the
subprocess tests); previously if you had tests "/foo/bar" and
"/foo/bar/baz", and ran the test program with "-p /foo/bar/baz", it
would run "/foo/bar" too. Fix that and add tests.

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

g_test_trap_fork: don't blow away the SIGCHLD handler

Not sure why it was doing this, but it's not necessary (all of glib's
tests pass fine without it), and it breaks tests that try to use
g_spawn_sync() or GChildWatchSource after doing a g_test_trap_fork().

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

tests/spawn-*.c: fix on Windows

Need to append ".exe" to the spawned binary name on Windows

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

app: Document when floating variants are consumed

Bug #700203.

Updated Odia Translation.

Clarify GValueArray docs

Don't refer to Quicksort in the documentation of
g_value_array_sort, but just to qsort().

Updated Telugu Translations

Fix minor typos in GSocketConnectable example code

Declare res_init() for builds against Android's Bionic

res_init() exists in the C library here and as such is detected
by configure, but it is not declared in any header.

GProxyAddressEnumerator: add default-port property

Although none of the in-tree GSocketConnectable types need it, other
types (like SoupAddress) may find it useful to be able to pass a URI
and a default-port to GProxyAddressEnumerator separately (the same way
you can with GNetworkAddress). So add a default-port property.

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

Document all gtester-report options

The --subunit option was missing from the man page.

Document all glib-mkenums placeholders

The @ENUMPREFIX@ placeholder was missing in the man page.

Document all glib-genmarshal options

The --stdinc option was missing from the man page.

Fix a typo

Document all gdbus-codegen options

The --help and --xml-files options were missing from the man page.

gbitlock: fix this to not unconditionally use futex emulation

Ryan accidentally committed some debugging code a long time ago,
causing this file to always use futex emulation even when real futex
support was available. I noticed this a while later and pointed it out
to him, and assumed he was going to fix it, but I guess he assumed I
was going to fix it, and then neither of us did...

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

Silently handle icon being NULL

While an emblemed icon without a base icon is not very
useful, no need to crash here.
This was crashing the object finalization test in gtk.

binding: Add an explicit unbind()

Higher order languages with garbage collection can have issues releasing
a binding, as they do not control the last reference being dropped on
the binding, source, or target instances.

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

tests/mappedfile: Also handle ENOMEM

The RHEL6.4 kernel gives me that instead of EINVAL.

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

gobject: rename an unused parameter to make AIX happy

GSocks5Proxy: don't crash if parsing negotiation reply fails

The GError should be initialized to NULL, otherwise we'll
"pile up" errors, then try to free an uninitialized pointer.

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugzilla.gnome.org/show_bug.cgi?id=699493

hindi update

Updated Slovenian translation

Bump version to 2.37.1

Add 2.38 indexes to the docs

Updates

gio: fix small leak

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

GApplication: don't leak 'hint' on remote Open

We were using format string "s" to deconstruct the open hint into a
'const gchar *' which, of course, we never freed.  Fix that.

bytesicon: don't use g_object_unref() on GBytes

We need to use g_bytes_unref()

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

bytesicon: fix a memory leak

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

configure: Assume C90 compatible malloc() prototype

This ancient code was attempting to cope with (unknown) systems whose
malloc() prototype was incompatible with the standard.  This test was
fragile; it would break if the build environment provided -Wall in
CFLAGS.

Now that it's 2013, let's assume that target systems have a sane
malloc().  If someone complains, we can revisit this.

https://bugzilla.gnome.org/698716

g_object_new: check for NULL from _constructor()

There is some code in the wild (like in gnome-session) that does this
from its custom _constructor() implementation:

{
  GObject *obj;

  obj = ((chain up));

  if (!object_is_viable (obj))
    {
      g_object_unref (obj);
      return NULL;
    }
  else
    return obj;
}

This has never been a valid use of GObject and this code has always
caused memory to be leaked[1] by growing the construction_objects list.
The ability to legitimately return NULL from a constructor was exactly
the reason that we created GInitable, in fact.

That doesn't change the fact that the g_object_new() rewrite will crash
in this case, so instead of doing that, let's emit a critical and avoid
the crash.  This will allow people to upgrade their GLib without also
upgrading their gnome-session.  Meanwhile, people can fix their broken
code.

[1] not in the strictest sense of the word, because it's still reachable

Tamil Translation Updated