Tiny docs fix

Add g_value_take_variant

Turns out we do need g_value_take_variant(), so we can correctly
implement VARIANT:* marshalers.

Bug #621947.

Add g_variant_is_floating

Fix a preprocessor warning with --disable-visibility

Don't redefine DISABLE_VISIBILITY when already building with
--disable-visibility.

Fix gsettings tool crash, part 2

Same as previous commit, but for two other functions it missed.

Fix gsettings tool crash

When no path is provided for the schema, we have call
g_settings_new() instead of g_settings_new_with_path()
passing a NULL path.

This was crashing the tool on start since an assertion was
recently added to g_settings_new_with_path() to refuse NULL.

Fix bug in strinfo

We can't search for a larger needle inside of a smaller haystack, and
unsigned integer subtraction tends to result in very large numbers
rather than small ones.

Add a check for this case and abort out immediately.

Also add a test case (lifted directly from the docs) that demonstrates
the problem.

Issue discovered and tracked down by Milan Bouchet-Valat

Updated Galician translations

Add a note to docs about paragraphs in schemas

Conditional gsettings.m4 support, AC_PATH_PROG

Add new support for conditional enabling of GSettings (as per
bug #616718) and use AC_PATH_PROG instead of AC_PATH_TOOL (as per
bug #621172).

GDBus: Sink closures in _with_closures functions

See https://bugzilla.gnome.org/show_bug.cgi?id=621092#c12

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

[GApplication] Add working directory to platform data

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

Fix build on non-Unix

2.25.9

Fix a mismerge

Fix distcheck

Fix VARIANT out args

Use g_value_set_variant(), not the nonexistent g_value_take_variant().

Add VARIANT to glib-genmarshal docs

GDBusMessage: Fix bug when deserializing a message

See https://bugzilla.gnome.org/show_bug.cgi?id=621838 for the whole
story. The problem was that we ended up reading data from arrays of
arrays when we were just supposed to be aligning the buffers.

Also add a host of debug infrastructure that was needed to find the
root cause. For now it can be turned on only via defining
DEBUG_SERIALIZER. In the future we might want to make it work via
G_DBUS_DEBUG. In a nutshell, the added debug info looks like this

Parsing blob (blob_len = 0x0084 bytes)
  0000: 6c 01 00 01  3c 00 00 00  41 00 00 00  37 00 00 00    l...<...A...7...
  0010: 08 01 67 00  08 61 61 79  61 7b 73 76  7d 00 00 00    ..g..aaya{sv}...
  0020: 01 01 6f 00  08 00 00 00  2f 66 6f 6f  2f 62 61 72    ..o...../foo/bar
  0030: 00 00 00 00  00 00 00 00  03 01 73 00  06 00 00 00    ..........s.....
  0040: 4d 65 6d 62  65 72 00 00  00 00 00 00  34 00 00 00    Member......4...
  0050: 03 00 00 00  63 77 64 00  01 73 00 00  23 00 00 00    ....cwd..s..#...
  0060: 2f 68 6f 6d  65 2f 64 61  76 69 64 7a  2f 48 61 63    /home/davidz/Hac
  0070: 6b 69 6e 67  2f 67 6c 69  62 2f 67 69  6f 2f 74 65    king/glib/gio/te
  0080: 73 74 73 00                                           sts.

Parsing headers (blob_len = 0x0084 bytes)
  Reading type a{yv} from offset 0x000c: array spans 0x0037 bytes
    Reading type {yv} from offset 0x0010
      Reading type y from offset 0x0010: 0x08 '
      Reading type v from offset 0x0011
        Reading type g from offset 0x0014: 'aaya{sv}'
    Reading type {yv} from offset 0x001e
      Reading type y from offset 0x0020: 0x01 '\ 1'
      Reading type v from offset 0x0021
        Reading type o from offset 0x0024: '/foo/bar'
    Reading type {yv} from offset 0x0031
      Reading type y from offset 0x0038: 0x03 '\ 3'
      Reading type v from offset 0x0039
        Reading type s from offset 0x003c: 'Member'
Parsing body (blob_len = 0x0084 bytes)
  Reading type (aaya{sv}) from offset 0x0047
    Reading type aay from offset 0x0048: array spans 0x0000 bytes
    Reading type a{sv} from offset 0x004c: array spans 0x0034 bytes
      Reading type {sv} from offset 0x0050
        Reading type s from offset 0x0050: 'cwd'
        Reading type v from offset 0x0058
          Reading type s from offset 0x005b: '/home/davidz/Hacking/glib/gio/tests'
OK

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

Distcheck fix

Add a missing symbol to gobject.symbols.

Make g_settings_sync() a proper prototype.

Fix the build

Updates

GDBus: Complain via g_warning() if an invalid message is received

While we are already propagating the error to the user via the
GDBusConnection::disconnected signal (because the only safe thing is
to disconnect the other peer), changes are the user is simply not
listening to this signal.

This should never ever happen unless there's a bug in the
serializaer/deserializer so it's fine to complain via g_warning()
here.

Bug 621838, see

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

is related to this.

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

GDBusMessage: Properly check error and bail if set

Otherwise we may set a GError on top of it.

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

Documentation fixups

fix a typo

Fix GApplication for the G_TYPE_VARIANT change

Update api docs to current api

Catch up with G_TYPE_VARIANT changes

The GType for a GVariant is now a fundamental GType instead of a boxed
one so use the right marshaller.

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

Fix a typo

GSettings: remove "context" from backend too.

Note: this is a GSettingsBackend API/ABI change.

Add fundamental type and pspec for GVariant

Make G_TYPE_VARIANT a fundamental type instead of boxed, and add
g_variant_{set,get,dup}_variant.

Add GParamSpecVariant.

Bug #610863.

Revert "GVariant: add g_variant_default_value()"

This reverts commit b205dc77cb14f67818eaac23d0eb3f5d1b867921.

We decided to go the other route with the GParamSpec.

Only run the schema compiler from the test cases

Bug 621266  - GSettings "context" clarification

Remove the concept of "context" in favour of dealing with
GSettingsBackend directly.

Bug 621905 – Assume a ref when doing async work

When asynchronously acquiring the DBus connection, assume a reference to
the proxy object, to avoid destroying it in the middle of the operation.

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

GSettings-related distcheck fixups

Add 'if' that was absorbed by $(AM_V_GEN)

From commit 1b8ee5196eb8ed684b0ee0d2f0d26e6bd82656af

More GSettings doc fixups

<alias> takes mandatory target='' arg

GSettings: Fix comment formatting, whitespace.

Add new api

Add some schema examples

Typo fix

Update the dtd to match the schema parser

The aliases element was not mentioned here.

Updates

GSettings: enum/choices/aliases/range test cases

Fix a small bug that one of the tests uncovered.

move GSettings(Backend) to giotypes.h

Bug 621319 - more leaked GVariants in GSettings

Caught by Felix Riemann.

GVariant serialiser: fix small type error

GSettings: major refactor.  Add enums, range.

GVariant: add g_variant_default_value()

Returns various kinds of false, zero, empty arrays, etc.

Typo fix

Typo fix

Rework GApplication API to use GInitable

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

Add gdbus-proxy-well-known-name to the ignore file

binding: Use a hash table

Since an object instance might have a lot of bindings, using a list
might become a performance issue when removing them. Using a simple
hash table guarantees constant time lookup, which is the most common
operation.

tests: Add a chain binding

Test the case with a chain like A → B, B → C, and what happens when
switching to a direct A → C link.

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

Properly initialize GError

Pointed out by Florian Müllner in bug #621702

Sprinkle some $(AM_V_GEN) around to make the build more silent.

Correct some documentation mistakes in gunixmounts.c

Pointed out by Ross Burton in bug #618904

gobject: Add GBinding

GBinding is a simple, opaque object that represents a binding between a
property on a GObject instance (source) and property on another GObject
instance (target).

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

Introduce g_object_notify_by_pspec()

g_object_notify_by_pspec() will emit the "notify" signal on the given
pspec, short-circuiting the hash table lookup needed by
g_object_notify(). The suggested and documented way of using
g_object_notify_by_pspec() is similar to the way of emitting signals
with their ID.

Emission tests (with no handler attached to the notify signal) show a
10-15% speedup over using g_object_notify().

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

Prevent division by zero if no tests

Forgotten file

Add subunit support to gtester-report

This patch adds subunit support to gtester-report via a -s switch. Subunit
(https://launchpad.net/subunit) is a language neutral test activity protocol.
This can be used to integrate gtester tests with the Hudson CI tool amongst
other things.

Bug #611869.

Use optparse to parse gtester-report commandline options

See bug #611778.

 Add package and version to the test report XML

Reword a doc comment

To take out some awkward wording.

Let GLIB_SETTINGS macro use glib-compile-schemas from PATH when cross compiling

See bug #621172

Use native glib-compile-schemas when cross compiling

Switch to using variants for timestamps, split out signals

Like how we're handling activation, use GVariant for timestamps.  To
avoid polluting the GtkApplication API with GVariants, we rename the
GApplication signals to "quit-with-data" and "action-with-data".
GtkApplication will then wrap those as just "quit" and "action".

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

Add --xml to gdbus-tool to print raw introspected XML

Bug #621442.

Avoid a race in application registration

We need to register the object before taking the name.
Remove the workarounds for this race from the test suite.

Updated Galician translations

Accept no-reply errors

This can happen if the app quits before the dbus reply can be
sent out.

Make the tests not fall over every other time

Bug 621213 – GDBusProxy and well-known names

Allow constructing a GDBusProxy for well-known names as discussed here
http://mail.gnome.org/archives/gtk-devel-list/2009-October/msg00075.html
including test cases.

Make it possible to create a GDBusProxy for a GBusType instead of a
GDBusConnection. This requires G_BUS_TYPE_NONE so add that too.

Nuke g_bus_watch_proxy() since one can now more or less use GDBusProxy
for this.

Port gdbus-example-watch-proxy to this new API and include this
example in the GDBusProxy doc page.

Also nuke the GType parameter from the GDBusProxy constructors as
requested here: https://bugzilla.gnome.org/show_bug.cgi?id=621229

Also update the porting guide and other API docs for this change.

Also fix a bug in the signal dispatching code so each subscriber only
get notified once, not N times, for the same signal. Also add a test
case for this.

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

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

gitignore GApplication test-cases

Close memory leak in GSettings

Fixes bug #621252.

converter streams: make code more readable

The name buffer_availabile was kinda confusing, so its been renamed
to buffer_data_size() to match buffer_data().

Also I added a comment to buffer_ensure_space because its behaviour
wasn't obvious.

Use correct offset for buffer data in GConverterOutputStream

Otherwise we overwrite already converted data in the case that more
than one call to g_converter_convert is necessary in write() or
flush().

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

Ensure we always have some target space in GConverterOutputStream

When the converter fills the whole buffer without reading all input,
we need to enlarge the buffer. Otherwise we get an assertion failure
for `outbuf_size > 0' in g_converter_convert.

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

Add a regression test for GConverterOutputStream corruption

See bug #619945 for background

GSettings test case fixes

 - call g_settings_sync()
 - fix bug #618715 by not forking if a backend is specified via
   environment variable

Add g_settings_sync() and use it

[Gio] Merge in introspection annotations from gobject-introspection gio-2.0.c

This is not an exhaustive set, but covers everything we have so far.

GSettings: store (default, options) in gvdb

gvdb just dropped the ability to have a separate "options" field.  We
now store the options into a GVariant along with the default value.

For now, we use a small shim in GSettingsSchema in order not to touch
too much code.  A more complete rewrite will follow.

This represents a change to the schema file format with another likely
to follow.  glib-compile-schemas needs to be re-run after installing
this change.

Merge remote branch 'gvdb/master'

drop "options" support

Add _with_closures alternative functions for those in GDBus that
accept more than one callback.

g_bus_own_name_with_closures
g_bus_own_name_on_connection_with_closures
g_bus_watch_name_with_closures
g_bus_watch_name_on_connection_with_closures
g_bus_watch_proxy_with_closures
g_bus_watch_proxy_on_connection_with_closures

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

[GApplication] Tweak docs a bit

Rewrite apps test to ensure children are killed

Create a function run_with_application that both ensures the
app is running exactly while the test is running, which most
of the tests use.  We start it beforehand, and kill it after.
This avoids having any interdependence between the tests (and
there definitely was before, because we didn't wait for
the process to actually terminate after a kill() call).

Also, open a pipe between the two, and have the child app
monitor that pipe.  If it gets closed (e.g. because the parent
died), the child exits.  This is the most reliable way to
avoid stale children; before, if we failed an assertion, the
parent would abort, and not run kill().

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

Mention GVariant format string docs from iter docs

It's not entirely clear what @format_string in iter_next() and
iter_loop() should be.  Include a link to the GVariant format string
docs as a hint.

Plug memory leak on Windows

Intern the string returned from g_win32_getlocale() and then free it.
Fixes bug #621168.

Fix GApplication tests to run without a session bus

We reuse code from the GDBus tests here to launch a session bus.

GDBusConnection: Do not dispatch calls to unregistered objects or subtrees

There was a slight race where we ended up calling into user code if
the user managed to unregister an object (or subtree) in the window
between

 - processing the remote call on the worker thread; and
 - continuing handling it on the user code thread (via an idle handler)

This patch fixes the problem.

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

GDBusProxy: Fix error handling in synchronous initialization codepath

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

g_variant_builder_add_parsed: fix type error

Problem caught by Juan A. Suarez Romero