commit 80e37faeac62a4694514410ec17c2ab4a75d50e6
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-21 17:36:11 +0100

    dbus-glib 0.96

 configure.ac |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

commit 4bc172fd5f31123ee883c4027e729b38c2777d7d
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 17:43:15 +0100

    test-dbus-glib: make lose() abort, and make lose_gerror() more
    informative

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 test/core/test-dbus-glib.c |   21 ++++-----------------
 1 files changed, 4 insertions(+), 17 deletions(-)

commit 9e97f9744e7ffeb04a105087da6143931195ce77
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 16:25:15 +0100

    get_all_object_properties: if _dbus_gvalue_marshal fails, bail out

    This isn't necessarily OOM: _dbus_gvalue_marshal can fail due to
    programming errors. If so, raise a critical warning, then (if
    that wasn't
    fatal) return a D-Bus error to be sent to the caller.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |   27 +++++++++++++++++++++++++--
 1 files changed, 25 insertions(+), 2 deletions(-)

commit bfd1c05174b86c9dfcf1b22a4fa0b2c552dfc902
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 18:02:47 +0100

    get_all_object_properties: note which operations can only fail via OOM

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |   12 ++++++++++++
 1 files changed, 12 insertions(+), 0 deletions(-)

commit 30198a2368c9cd19db4b8f595bfad7a719ada7e9
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 16:21:12 +0100

    get_all_object_properties: skip invalidly-named properties, with
    a critical

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

commit a2a3315ac9747dcc7dda860ceb1d1f94ab56d29b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-08-17 19:15:05 +0100

    dbus-gobject: add and use connection_send_or_die to check for OOM

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Conflicts:
	dbus/dbus-gobject.c

 dbus/dbus-gobject.c |	 40 +++++++++++++++++++++++++---------------
 1 files changed, 25 insertions(+), 15 deletions(-)

commit 22262581b0956f69f321b9e2eb702e3f15b49823
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:50:38 +0100

    emit_signal_for_registration: assert that signal headers are all valid

    This is an assertion, not a return_if_fail, because we already checked
    the object path in dbus_g_connection_register_g_object and the others
    in export_signals.

    After these checks, dbus_message_new_signal can really only fail
    on OOM.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

commit cccf1e6672fba6b649fe02082c2f087cdf022dcd
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:49:18 +0100

    export_signals: check interface, signal names for validity

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 14 ++++++++++++++
 1 files changed, 14 insertions(+), 0 deletions(-)

commit d09dfcb130be0a17f337bd198a9fb43aab7dfa28
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:43:30 +0100

    object_registration_message: check for OOM when a property is absent

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  8 +++++---
 1 files changed, 5 insertions(+), 3 deletions(-)

commit 7b92ac1bfd1e50e842549dbb14b845478d114652
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:42:49 +0100

    object_registration_message: make logic/assertions slightly clearer

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 13 ++++++-------
 1 files changed, 6 insertions(+), 7 deletions(-)

commit b5b27e4ca0d8ca250fa7649bcf2613e8d889a722
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:42:29 +0100

    object_registration_message: handle remote caller errors better

    If the remote process sends us a wrong message, that's its fault, not
    ours; we should send back a comprehensible D-Bus error, and not spam
    to stderr.

    Previously, in each of these cases libdbus would have sent back
    NoReply,
    because we declined to handle the method call.

    One case that's still wrong is passing extra arguments to Get,
    GetAll or
    Set, like so:

	Get("com.example.Iface", "MyProperty", "extra")
	Set("com.example.Iface", "MyProperty", Variant("foo"), "extra")
	GetAll("com.example.Iface", "extra")

    dbus-glib has historically warned, ignored the extra argument(s)
    and sent
    back a reply as if they hadn't been there, whereas a stricter
    implementation (like telepathy-glib's TpDBusPropertiesMixin) would
    have sent back an error reply and done nothing.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 20 ++++++++++++++------
 1 files changed, 14 insertions(+), 6 deletions(-)

commit e5cf2be0e03af4df19618076015cc2a6556ca5d4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:36:09 +0100

    get_object_property: add brief documentation

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  5 +++++
 1 files changed, 5 insertions(+), 0 deletions(-)

commit d3aa6c7942cce0cf7b602c6b0854488938f0548e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:25:55 +0100

    get_object_property: unwind on errors, and avoid returning NULL

    Also treat all errors here as programming errors (because this method
    should never fail), upgrading them from warning to critical; return a
    D-Bus error reply anyway, to be nicer to our callers.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 33 +++++++++++++++++++++++----------
 1 files changed, 23 insertions(+), 10 deletions(-)

commit 98b3b3f323fd4001be0630c3e0a1dc4fb83cdeab
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:21:56 +0100

    check_property_access: centralize error handling (and check for OOM)

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 69
 +++++++++++++++++++++++++++++++-------------------
 1 files changed, 43 insertions(+), 26 deletions(-)

commit 6293849bf0469ce8d6d122b38db5091c21304a30
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-08-17 19:11:14 +0100

    dbus-gobject: check for NULL when producing messages

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Conflicts:
	dbus/dbus-gobject.c

 dbus/dbus-gobject.c |	  6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

commit 8ec3166b6a40508a91144efedc10dcb32964498e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 14:56:16 +0100

    dbus-gobject: factor out reply_or_die, error_or_die and add
    error-checking

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 83
 ++++++++++++++++++++++++++++++--------------------
 1 files changed, 50 insertions(+), 33 deletions(-)

commit 7a1abf5243fe5b7adf7211fd7d8c82851f448192
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 16:52:58 +0100

    invoke_object_method: if marshalling an out argument fails, discard
    message

    We shouldn't report this as OOM, because it probably wasn't (it's much
    more likely to be programming error); we should also continue
    through the
    arguments, so that we don't leak them.

    By abandoning the message as soon as we detect a programming error,
    we can use reply == NULL as an indicator of whether to keep appending.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 47
 ++++++++++++++++++++++++++++++++++++-----------
 1 files changed, 36 insertions(+), 11 deletions(-)

commit 4ba637e2ae32b2daf224265c22cc8b43cf0a0bbf
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 15:45:07 +0100

    invoke_object_method, dbus_g_method_return_error: handle sending
    failure

    I just made it fatal, since it's either programming error or OOM.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)

commit cd7e9a1c6dd0eef6e867cac80339a38758d0cd6f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 15:44:28 +0100

    gerror_to_dbus_error_message: guarantee to return non-NULL

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

commit c239a810543bde85dd458012612874cdc81a4151
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 15:41:32 +0100

    invoke_object_method: if dbus_message_new_method_return fails,
    fail hard

    There's no point in doing graceful unwinding here, since it really
    shouldn't happen.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

commit 480c480b024a77dd8f4efecdb6639d45de9767ce
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 15:40:40 +0100

    arg_iterate: document

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 40 ++++++++++++++++++++++++++++++++++++++++
 1 files changed, 40 insertions(+), 0 deletions(-)

commit 36b55c3aac5de873a69c9bd0fbe003200a03ba32
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-16 15:33:01 +0100

    dbus_g_proxy_manager_unregister: if GetNameOwner failed, don't assert

    got_name_owner_cb never adds the proxy to the unassociated_proxies
    list
    if there is a D-Bus error other than NameHasNoOwner.

    Based on a patch from Janne Karhunen, for Maemo.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=38408
    Bug-NB: NB#116862
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gproxy.c |	10 ++++++----
 1 files changed, 6 insertions(+), 4 deletions(-)

commit 18705c42ca6b6a583b64451055eec4dad3aaa3b4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-17 14:40:28 +0100

    Add a regression test for fd.o #38406

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=38406
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 .gitignore		|    1 +
 test/core/Makefile.am	|    7 ++
 test/core/proxy-peer.c |  274
 ++++++++++++++++++++++++++++++++++++++++++++++++
 3 files changed, 282 insertions(+), 0 deletions(-)

commit 5d800ca9352b40354a0f4a905c1c0b111b81f030
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-17 14:53:09 +0100

    MyObject: register marshallers so individual tests don't have to

    These marshallers are actually dual-use: they're used to call certain
    methods on MyObject, and also to bind to its signals.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=38406
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 test/core/Makefile.am	    |	 4 +++-
 test/core/my-object.c	    |	12 ++++++++++++
 test/core/my-object.h	    |	 1 +
 test/core/test-dbus-glib.c |	11 ++++-------
 4 files changed, 20 insertions(+), 8 deletions(-)

commit 70e09208ab13812037b3cf133df312aacd24d036
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-17 12:18:57 +0100

    DBusGProxy: deal with errors in DBUS_G_VALUE_ARRAY_COLLECT_ALL

    G_VALUE_COLLECT can fail (admittedly, it's a programming error if it
    does). Previously, we leaked the resulting g_malloc'd string; now we
    emit a critical warning, free it, and abort the call.

    Based on patches from Kimmo Hämäläinen and Christian Dywan
    in Maemo's
    dbus-glib.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=38406
    Bug-NB: NB#86280
    Bug-NB: NB#180486 (CID-34419 to CID-34424 inclusive)
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gproxy.c |	93
 +++++++++++++++++++++++++++++++++++++---------------
 1 files changed, 66 insertions(+), 27 deletions(-)

commit 2731a2f7f824f9c1cd11dd14e88ff1e0875f280b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-17 12:15:20 +0100

    dbus_g_proxy_call: simplify error handling now that call_id may be 0

    There's no need to special-case it any more - we can just use
    dbus_g_proxy_end_call_internal and rely on it to cope.

    This reverts the fix for fd.o #12675, since the commit before this one
    is more general.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=38406
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gproxy.c |	15 +++------------
 1 files changed, 3 insertions(+), 12 deletions(-)

commit 1b38cecf7bbdb8a99ef13616b8c661dfb0bc060f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-17 12:06:11 +0100

    DBusGProxy: cope gracefully with call_id == 0

    As well as happening on a programming error or OOM (which both provoke
    critical warnings), dbus_g_proxy_begin_call_internal() can
    legitimately
    fail if we got disconnected from D-Bus (so can't send any more),
    but are
    still working through our backlog of incoming messages (so we haven't
    got round to the Disconnected message yet, so the DBusGProxy hasn't
    emitted ::destroy).

    fd.o #12675 fixed dbus_g_proxy_call(), but
    dbus_g_proxy_call_with_timeout()
    and the asynchronous API still need similar treatment.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=38406
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gproxy.c |	16 ++++++++++++++++
 1 files changed, 16 insertions(+), 0 deletions(-)

commit af31c9c21efeec8a38c73de24d82a2038d80f74e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-08-03 18:38:49 +0100

    registrations test: fix race condition in adding match rules

    If you don't block waiting for the reply to AddMatch, and you're
    using two
    connections to the bus, then messages sent from connection 2 might not
    be matched by connection 1, because they arrive before the AddMatch
    call
    from connection 1. This manifests itself as the /registrations/twice
    test *sometimes* failing (dependent on timing).

    Only tests with two parallel bus connections need this, but to
    be safe,
    I've added it throughout this test.

    Reviewed-by: Vivek Dasmohapatra <vivek@collabora.co.uk>

 test/core/registrations.c |   28 +++++++++++++++++++++++++---
 1 files changed, 25 insertions(+), 3 deletions(-)

commit 6d06dd358b22057dca1808c1432e620998202c41
Author: Tim Waugh <twaugh@redhat.com>
Date:	2011-07-22 15:13:42 +0100

    dbus-binding-tool: fixed "pretty" mode output for methods.

    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=39475

 dbus/dbus-glib-tool.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

commit e294ba3e579db1c12158ce95ad6ff34391cf571a
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-16 18:12:27 +0100

    Show warning if marshalling in dbus_g_method_return fails

    Based on a patch from Christian Dywan <christian.dywan@lanedo.com>.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29884
    Bug-NB: NB#180486
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)

commit cce0887278ae105f010d35636fc5f6d654c2d41f
Author: Christian Dywan <christian.dywan@lanedo.com>
Date:	2010-12-14 15:07:27 +0100

    remove minor dead code in dbus-gparser

    There's no way retval can be false at this location.

    Bug-NB: NB#180486 CID-643
    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

 dbus/dbus-gparser.c |	  3 ---
 1 files changed, 0 insertions(+), 3 deletions(-)

commit f446cb80f91b4cf584883b47477295d07dd32eac
Author: Derek Foreman <derek.foreman@collabora.co.uk>
Date:	2011-06-02 17:16:42 +0100

    remove remaining inclusions of <glib/gi18n.h> from library

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=36428
    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

 dbus/dbus-binding-tool-glib.c |    1 -
 dbus/dbus-gproxy.c	       |    1 -
 2 files changed, 0 insertions(+), 2 deletions(-)

commit 2a85bd434e6d8dba9615a53d288a0a72d6b5e0cf
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-02 13:50:20 +0100

    Regression test for fd.o #37852

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37852
    Reviewed-by: Colin Walters <walters@verbum.org>

 test/core/my-object.c		 |   17 ++++++++++
 test/core/my-object.h		 |    2 +
 test/core/registrations.c	 |   68
 +++++++++++++++++++++++++++++++++++++++
 test/core/test-service-glib.xml |    4 ++
 4 files changed, 91 insertions(+), 0 deletions(-)

commit 3e0828f57c3925ea9b63d22ab82d991a0fea0536
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-02 13:49:51 +0100

    Fix regression in marshalling objects as object paths

    This regressed while fixing fd.o #36811. NetworkManager apparently
    uses
    this idiom.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37852
    Bug-Debian: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=628890
    Tested-by: Michael Biebl <biebl@debian.org>
    Reviewed-by: Colin Walters <walters@verbum.org>

 dbus/dbus-gobject.c |	  8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

commit f534276eb20d8035566a206d1e38fd651a1eeab9
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-31 17:03:59 +0100

    registrations test: only listen for signals, not all messages

    Otherwise, we'd reply with an error to messages not intended for us.

    Reviewed-by: Colin Walters <walters@verbum.org>

 test/core/registrations.c |	4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

commit 867d670daad983f8b4416528e69d3ad2e0596e88
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 19:44:42 +0100

    0.95

 configure.ac |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

commit 469df67525a9e5363644ac730ac8ec7580a8ebe0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 19:07:30 +0100

    Prepare 0.94 release

 configure.ac |    7 ++++---
 1 files changed, 4 insertions(+), 3 deletions(-)

commit 5a780af947f0141f4e0524aaf62fdd06abe6cd5f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 13:46:11 +0100

    dbus-binding-tool: forbid ReturnVal annotation after the first
    OUT <arg>

    It has never actually worked correctly (invoke_object_method would
    always
    treat the ReturnVal as if it had been the first OUT argument),
    so let's
    only allow the situation that worked in practice.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35952
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 dbus/dbus-binding-tool-glib.c |   15 +++++++++++++++
 1 files changed, 15 insertions(+), 0 deletions(-)

commit a8605cf4c6c9dbc4a933dd744d339744e0102765
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:33:48 +0100

    Fix compiler warnings in dbus-binding-tool

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 dbus/dbus-glib-tool.c |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

commit 0c162cc63b073a9f50dd11ea99db72fe95a16064
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:33:07 +0100

    Remove all support for Doxygen

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=10890

 Doxyfile.in	|  180
 --------------------------------------------------------
 Makefile.am	|    2 -
 README		|    1 -
 configure.ac	|   34 +----------
 doc/api/README |    3 -
 5 files changed, 1 insertions(+), 219 deletions(-)

commit 6e20efb5c4c094915d2b6e86c3cab486a0562a95
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:32:58 +0100

    Use single star in doc-comments not intended for gtk-doc

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=10890

 dbus/dbus-glib.h	   |	4 ++--
 dbus/dbus-gloader-expat.c |	2 +-
 dbus/dbus-gproxy.c	   |	2 +-
 dbus/dbus-gtest.c	   |	2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

commit 1472d195bd201d219aab5bc4b31d80d8340cee5c
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:32:06 +0100

    Remove Doxygen droppings from source code

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=10890

 dbus/dbus-gidl.c    |	  4 ----
 dbus/dbus-gidl.h    |	  4 ----
 dbus/dbus-glib.c    |	  3 +--
 dbus/dbus-gmain.c   |	 22 +++-------------------
 dbus/dbus-gobject.c |	  3 +--
 dbus/dbus-gparser.c |	  4 ----
 dbus/dbus-gproxy.c  |	 26 ++------------------------
 dbus/dbus-gthread.c |	  6 ------
 dbus/dbus-gutils.c  |	  4 ----
 dbus/dbus-gutils.h  |	  4 ----
 dbus/dbus-gvalue.c  |	  5 +----
 11 files changed, 8 insertions(+), 77 deletions(-)

commit 4c3a346d296789cf3a81f6cc98f64d84ea892247
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:24:36 +0100

    Remove _dbus_g_proxy_test, which isn't called and doesn't do anything

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37790

 dbus/dbus-gproxy.c |	17 -----------------
 1 files changed, 0 insertions(+), 17 deletions(-)

commit cded5996b02ff937a73654788d1f3a7ca46ca0a9
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:24:29 +0100

    Remove _dbus_gutils_test, which didn't test anything

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37790

 dbus/dbus-gtest.c  |	 6 +-----
 dbus/dbus-gutils.c |	17 -----------------
 2 files changed, 1 insertions(+), 22 deletions(-)

commit d81722def291666a3c557a18d66be061f8f55e31
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:24:22 +0100

    _dbus_gidl_test: remove unused function which does nothing

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37790

 dbus/dbus-gidl.c |   17 -----------------
 1 files changed, 0 insertions(+), 17 deletions(-)

commit 1f371d3c75a5c2e9979350380e4243e3c445e093
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:23:45 +0100

    dbus-glib-tool: remove regression-test boilerplate which does nothing

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37790
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 dbus/dbus-glib-tool.c |   48
 ------------------------------------------------
 1 files changed, 0 insertions(+), 48 deletions(-)

commit 8abfa07b8acd2d8527785db3711d8a88a213a60e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-06-01 17:18:52 +0100

    dbus-binding-tool: clean up temporary file on error

    This fixes distcheck, which otherwise fails because temporary
    files are
    left over after distclean. We could delete them with the build system,
    but leaving them behind is a bug, so...

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37812
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 dbus/dbus-glib-tool.c |   78
 ++++++++++++++++++++++++++++++++++++------------
 1 files changed, 58 insertions(+), 20 deletions(-)

commit d5b9aa101d61bcd2321f0266fa638b644e4cba96
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-31 14:35:01 +0100

    Remove useless HAVE_GLIB conditional - GLib is mandatory

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37789

 configure.ac |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

commit 601b51279e7fbb08ff6ce8c98eaf4bcee21d46a5
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-03-28 14:40:12 +0100

    Remove unused DBUS_DAEMONDIR

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37789

 configure.ac |   10 ----------
 1 files changed, 0 insertions(+), 10 deletions(-)

commit 1a966d070e71378590fb0b64c5ec7a2e250d397b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-03-28 14:39:57 +0100

    Remove vestigial dbus-glib-error-*.h and the machinery to generate
    them

    This was the only use of DBUS_INCLUDEDIR, so remove that too.

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37789

 .gitignore			     |	  4 ----
 configure.ac			     |	  2 --
 dbus/Makefile.am		     |	 16 ++--------------
 dbus/make-dbus-glib-error-enum.sh   |	  2 --
 dbus/make-dbus-glib-error-switch.sh |	  2 --
 5 files changed, 2 insertions(+), 24 deletions(-)

commit 98d7cb8c869b53686a742fb48d1f189c74ff1484
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-31 14:33:18 +0100

    Simplify the check for dbus-1

    As in commit 0f782b4, omitting the action-if-found and
    action-if-not-found
    results in much better error messages if this mandatory dependency
    isn't
    found.

    Nothing checks HAVE_DBUS (as they shouldn't - it's mandatory!),
    so there's
    no point in keeping that either.

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37789

 configure.ac |   12 +++---------
 1 files changed, 3 insertions(+), 9 deletions(-)

commit b00d53dd647cf456ddb67c1dfdf44a8d56be90b4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-01-04 18:35:41 +0000

    Add more auto-generated gtkdoc droppings to .gitignore

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37789

 .gitignore |	 2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

commit bd92f266609fef162b6028fd249250b0ab9ae689
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-31 14:46:33 +0100

    Remove support for i18n completely

    As far as I can tell, dbus-glib has never actually been translated
    into
    another language, or even had the necessary infrastructure so
    people could
    do that. The translated strings are all low-level errors in
    "programmer
    English" anyway, and most of them only make sense to D-Bus experts.

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=36428

 configure.ac		       |   25 +-----------------
 dbus/dbus-binding-tool-glib.c |    8 +++---
 dbus/dbus-glib-tool.c	       |   35 +++++++++----------------
 dbus/dbus-gparser.c	       |   56
 +++++++++++++++++++----------------------
 dbus/dbus-gproxy.c	       |    8 +++---
 dbus/dbus-gvalue.c	       |   41 ++++++++++++++---------------
 6 files changed, 68 insertions(+), 105 deletions(-)

commit c37bd4357d50d1cd683ef04f05ee427c9a0f7e9f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-03-28 14:43:15 +0100

    Remove i18n macros from files where they're unused

    dbus-glib-tool (dbus-binding-tool) and dbus-gparser are theoretically
    localizable, but the library doesn't make any use of gettext.

    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=36428

 dbus/dbus-glib.c  |	5 -----
 dbus/dbus-gmain.c |	4 ----
 2 files changed, 0 insertions(+), 9 deletions(-)

commit e8a4f0496d0eec10fc9c76e0ef20e761bb2256c6
Author: Dan Williams <dcbw@redhat.com>
Date:	2011-05-31 11:25:59 -0500

    Log error message from failed object registration

    See https://bugzilla.redhat.com/show_bug.cgi?id=708686 for a case
    of this; it
    would be extremely helpful to figure out what the error actually
    was since it
    appears not to be a double-registration or other such issues after
    inspection
    of nm-applet and libnm-glib.

    Reviewed-by: Colin Walters <walters@verbum.org>
    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37795

 dbus/dbus-gobject.c |	 15 ++++++++++-----
 1 files changed, 10 insertions(+), 5 deletions(-)

commit eeb1c50aa03a95c39487e9331107142f27412958
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-20 16:41:28 +0100

    Remind maintainers to update the libtool versioning

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=22854
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

 HACKING |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

commit 9e2031148fd0ee114ec67c681d2b9bcfd84cdaa4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-20 16:41:00 +0100

    add a Makefile rule to upload a tarball and up-to-date docs

    Simplified from telepathy-glib, which has more elaborate machinery to
    automate releases.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=26952
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

 HACKING     |	  3 ++-
 Makefile.am |	 12 ++++++++++++
 2 files changed, 14 insertions(+), 1 deletions(-)

commit 61f48c290176c007ff0aef6ca4d752491763e4b0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-30 15:41:27 +0100

    Force a full rebuild for new gcc versions if gcov is enabled

    This avoids a lot of "Version mismatch - expected 406p got
    405*". Because
    we're changing the content of each file after preprocessing, it even
    defeats ccache, which is desirable here - otherwise we'd get old
    versions
    of the profiling code being resurrected from the cache.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37060
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

 configure.ac |   17 ++++-------------
 1 files changed, 4 insertions(+), 13 deletions(-)

commit 0f320c5224fe39df44648ac75ecce9e0a01079d0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-30 15:41:01 +0100

    Include config.h in generated C files, and regenerate dbus-gmarshal.c

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37060
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

 dbus/Makefile.am			|    3 ++-
 dbus/dbus-gmarshal.c			|    7 +++++--
 dbus/examples/statemachine/Makefile.am |    4 +++-
 test/core/Makefile.am			|    4 +++-
 4 files changed, 13 insertions(+), 5 deletions(-)

commit f91828abb160e44b53a32823516d55d8a5d634bc
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-30 15:39:28 +0100

    Consistently include config.h in all non-generated .c files

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37060
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

 dbus/dbus-gidl.c				  |    2 ++
 dbus/dbus-gloader-expat.c			  |    2 ++
 dbus/dbus-gparser.c				  |    3 +++
 dbus/dbus-gproxy.c				  |    3 +++
 dbus/dbus-gthread.c				  |    2 ++
 dbus/dbus-gtype-specialized.c			  |    2 ++
 dbus/dbus-gvalue-parse-variant.c		  |    2 ++
 dbus/examples/example-client.c			  |    2 ++
 dbus/examples/example-service.c		  |    2 ++
 dbus/examples/example-signal-emitter.c		  |    2 ++
 dbus/examples/example-signal-recipient.c	  |    2 ++
 dbus/examples/statemachine/statemachine-client.c |    2 ++
 dbus/examples/statemachine/statemachine-server.c |    2 ++
 dbus/examples/statemachine/statemachine.c	  |    2 ++
 test/core/30574.c				  |    2 ++
 test/core/peer-client.c			  |    2 ++
 test/core/peer-server.c			  |    2 ++
 test/core/test-dbus-glib.c			  |    2 ++
 test/core/test-gvariant.c			  |    2 ++
 test/core/test-service-glib.c			  |    2 ++
 test/core/test-thread-client.c			  |    2 ++
 test/core/test-thread-server.c			  |    2 ++
 test/core/test-types.c				  |    2 ++
 test/core/test-variant-recursion.c		  |    2 ++
 test/interfaces/test-client.c			  |    2 ++
 test/interfaces/test-objects.c			  |    2 ++
 test/interfaces/test-server.c			  |    2 ++
 test/test-service.c				  |    2 ++
 28 files changed, 58 insertions(+), 0 deletions(-)

commit 6d6d9a61e47f5bc28b5d61ae0d6cb072a81b5569
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-13 19:00:41 +0100

    dbus_g_proxy_add_signal: document error cases

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=36216
    Reviewed-by: Guillaume Desmottes <guillaume.desmottes@collabora.co.uk>

 dbus/dbus-gproxy.c |	 5 ++++-
 1 files changed, 4 insertions(+), 1 deletions(-)

commit 52f4e09d49551f3a8ece6578a6cb6e016c03a9d8
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-10 12:00:15 +0100

    dbus-gobject: check various preconditions for methods

    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=36811

 dbus/dbus-gobject.c |	 29 ++++++++++++++++++++++++++---
 1 files changed, 26 insertions(+), 3 deletions(-)

commit 6a61c922cedad9f5af5c32a0d783e9fbd53f3487
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-05-04 12:52:25 +0100

    dbus-gobject: move weakref to ObjectExport, with ObjectRegistration
    pointing there

    This has the following results:

    * each exported object only needs one weak ref to itself, however many
