Stop using glib-genmarshal at build time

To help cross compilation, don't use glib-genmarshal in our
build.  This is easy now that we have g_cclosure_marshal_generic().

In gobject/, add gmarshal.[ch] to git (making the existing entry
points stubs).

In gio/, simply switch to using g_cclosure_marshal_generic().

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

GDBus: Unref worker from worker-thread to avoid race

... otherwise we might end up using the worker after it has been
freed. Reported by Dan Winship and Colin Walters.

This fix uncovered a bug in the /gdbus/nonce-tcp test case so "fix"
that as well to use a better way of having one thread wait for another
(using quotes for the word "fix" since it's pretty hackish to
busy-wait in one thread to wait for another).

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

gthread tests: Remove spurious return;

gclosure: Add G_TYPE_ENUM and G_TYPE_PARAM for g_cclosure_marshal_generic()

These are fundamental types missed in the original commit.

gdbus-test-codegen: Allocate full integer space for uint16 properties

Because there is no G_TYPE_[U]INT16, we need to use integer types,
otherwise we'll end up overwriting memory (in this case on the stack).

configure.ac: Generate xz tarballs by default

Visual C++ 2010 project files update

Consolidate my previous commit into property file... should have realized
this sooner...

Update Visual C++ 2010 Projects

-Seperate intermediate directories for each project to avoid intermittent
 MSBuild errors that a build log cannot be written while in use, and
 update the property sheet as necessary.
-Minor cleanups of uneeded tags in the projects/properties

Clarify nul-termination of g_utf8_to_ucs4[_fast] result

The docs for g_utf8_to_ucs4_fast didn't mention that the resulting
string is terminated by a 0 character.

Bug #652897.

Add tests for GCache

asyncqueue: improve test coverage

utils: silence tests unless verbose

logging: improve test coverage

hash: Improve test coverage

Update gsettings/glib-compile-schemas VS Projects

These tools require the use of GModule headers also, so update the include
directories so that the correct gmodule.h will be included instead of the
system-installed version.

gvariant: Mark g_variant_new_variant as constructor

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

gdatainputstream: Add Since: tags and update gio-sections.txt

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

GDataInputStream: Add _utf8() variants of _read_line

These will validate the resulting line, and throw a conversion error.
In practice these will likely be used by bindings, but it's good
for even C apps too that don't want to explode if that text file
they're reading into Pango actually has invalid UTF-8.

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

GDataInputStream: Clarify read_line() docs, mark as byte array

g_data_input_stream_read_line() and
g_data_input_stream_read_line_finish() don't do any encoding checks,
so we shouldn't call the returned value a "string" (which I'd like to
mean UTF-8).  Annotate them as byte arrays and add encoding warnings
to the docstrings.

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

Fix build due to trivial typos in ab0e9dbf.

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

G_STATIC_REC_MUTEX_INIT: Add a 0 to the initialization.

This avoids a compiler warning about an incomplete initialization
when using this.

Drop dead code in GVariant parser

There is no chance that an unsigned integer value will be negative after
we do the bounds check that enforces its non-negativity.

Caught by Matthias running Coverity.

Eliminate some dead code

Remove a redundant NULL check

Silence a compiler warning

Add some argument checks to filename conversion functions

gmain: Close race condition in _g_main_wake_up_all_contexts()

Running gthread/tests/spawn-multithreaded in a loop, I very easily hit:

GLib-CRITICAL **: g_main_context_wakeup: assertion `g_atomic_int_get (&context->ref_count) > 0' failed

Testing the refcount still left a window where we would fall into the
assertion.  Fix this by just locking the context.

gmain: Only run through signal delivery once per read()

This is what I intended to do before.

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

gmain: Use sigset_t for keeping track of installed signals

Minor code cleanup.

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

gspawn: Reset signal handlers for synchronous spawning

We should by default reset signal handlers; particularly with the
ability to install them via g_unix_signal_source_new(), we don't
want to call a user callback inside a fork()ed helper process.

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

glib/tests/unix.c: Also test SIGTERM

We expect this to not kill the process, so it'd be a good one to test.

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

spawn-multithreaded: Clean up IO channel code

I modeled the new bits after how gunixmount.c handles GIOChannel;
it's apparently easier not to look at the condition.

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

gspawn: Handle EINTR in a few more cases

I was debugging gthread/tests/spawn-multithreaded.c, and while I
don't think I actually hit EINTR in any of these cases, it'd be
good to fix them anyways.

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

spawn-singlethread.c: New test

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

spawn-multithreaded: Also look for lt-test-echo

I hate libtool =(

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

If dup fails don't call close()

Remove a redundant assertion

A gsize is never going below 0.

Don't compare unsigned numbers with 0

It does not work.

GDBusMessage: Optimize serializer and deserializer

... by using a switch instead of if-then-else.

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

gclosure: Add missing include

Don't do < 0 on an unsigned value

It won't work. read() returns a signed value, anyway.

Silence a compiler warning

Even though we are confident the filename will always
end in .gmarkup, the compiler doesn't know that...

Don't leak resources in error cases

Don't forget to free resources in an early return

Avoid a deref-before-NULL-check warning

Remove unneeded NULL check

We ensure 10 lines up that parameters is not NULL.

Remove an unneeded NULL check

data->proxy cannot be NULL here, or we'd have crashed 10 lines up.

Silence an uninitialize variable warning

Fix up symbol lists and docs

gclosure: add missing fundamental types to generic marshaler

for GObject type the ffi->gvalue conversion was not handled. The
GFlags type was completely missing.

GDesktopAppInfo: Fix declaration of g_desktop_app_info_get_nodisplay

GDesktopAppInfo: Add g_desktop_app_info_get_nodisplay

Necessary for rebasing gnome-menus on top of GDesktopAppInfo.

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

bump version

2.29.8

Fix doc generation

It helps to use the same file name on both ends...

Use a realistic timeout in a test

Who knew that g_test_trap_fork() takes a timeout in microseconds...

Update NEWS

gdbus-codegen tests: Fix usage of top_srcddir

Based on a patch from Rico Tzschichholz.

Formally deprecate G_CONST_RETURN

Drop a few more G_CONST_RETURN

update .gitignores

gspawn: Fix annotations

g-ir-scanner wants guint8.

Fix regression in qsort_r BSD detection

We can use AC_CHECK_FUNCS to detect if qsort_r is available on
the system or not since it will unconditionnally define
HAVE_QSORT_R, which we don't want since on BSD, qsort_r isn't usable
for us, so we don't want to have HAVE_QSORT_R defined on such platforms.
By using AC_CHECK_FUNC instead, we can defer defining HAVE_QSORT_R until
we have actually tested it's usable.

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

Updated Hebrew translation.

[l10n] Updated German translation

Added UG translation

Updated Spanish translation

gioenums.h: Remove trailing commas

gdbus: Don't pass an unintialized GError pointer

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

gdbus: Add --recurse and --only-properties options

These options are useful when debugging D-Bus services and working
with bug reports.

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

gdbus: Clarify the --xml option in the man page

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

Bug 652197 – Improper handling of double values in GDBusMessage

Matthew Bucknall pointed out

  GDBusMessage does not serialize/deserialize double values correctly
  on platforms with strict alignment constraints (in my particular
  case, ARM926EJ-S).

This was reported in

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

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

gdbus-codegen: Fix build

See https://bugzilla.gnome.org/show_bug.cgi?id=651998 for details.

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

Document the deprecatation of G_CONST_RETURN

...but don't actually deprecate it yet.

Remove all uses of G_CONST_RETURN

Just use 'const'.

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

gobject: remove -DG_DISABLE_CONST_RETURNS

GObject had a few cases of depending on G_CONST_RETURN not being defined
to 'const'.  Remove those in preparation for deprecation of
G_CONST_RETURN.

Updated Bulgarian translation

Move gmain docs inline

Move gutils docs inline

Remove unused template

bitlock: don't use asm goto on older gcc

asm goto was addded in gcc 4.5 so don't try to use it on gcc versions
older than this one. This is achieved by explicitly checking gcc
version, an alternative would be to try to compile a program using
asm volatile in configure.

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

Test the previous fix

Just make one of the existing GApplication test cases use
a dash in the id.

GApplication: allow '-' in application ID

By converting it to _ before trying to shove it into an object path.

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

Fix a typo in GBinding

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

gconverter: Improve cross-references

Fix export_symbols variable in gio/Makefile.am

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

Fix "Since" tags on g_data_input_stream_read_upto{,_async}

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

gutils, gspawn: fix introspection annotations

Fix various (out) arguments, (allow-none) and (array zero-terminated=1)
for g_spawn_*() and some others.

Some additional fixes by Colin Walters <walters@verbum.org>

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

Update annotations from gobject-introspection/gir/glib-2.0.c

This covers most of them.

glib-types.h: Include glib.h, since we use G_BEGIN_DECLS

This header is included individually by g-ir-scanner.

Generate $module-public-headers.txt file, feed it to gtk-doc

Rather than having the gtk-doc build machinery have a list of header
files to exclude, change the GLib build to dump a list of public
header files generated from the maintained Makefile.am files for
each of glib/, gobject/, gio/.

Also, for glib, always install glib-unix.h, even on non-Unix
platforms, for the same reason we install gwin32.h even on Unix.

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

gdbus-codegen: Clean config.pyc to make distcheck happy

g_get_user_data_dir(): Don't warn if XDG_RUNTIME_DIR isn't set

We have a meaningful fallback, and warning causes the tests to break.

Updated Norwegian bokmål translation

Updated Czech translation

Add a testcase for the previous fix

GDataList: don't hold the bitlock over callbacks

g_datalist_id_clear_i was dropping the dataset lock around
the destroy notifies, but kept the bitlock. This was causing
deadlocks when finalizing widgets.

Update VS 2008/2010 property sheets

"Install" the new gobject/glib-types.h as well... it is needed.

Update gsettings utilities in GIO

Avoid C99-style variable declaration

Update config.h.win32.in

-Make contents more like the current config.h(.in)
-vsnprintf is included in VS 2008+