* bus/test-main.c (main): Initialize threading during tests

* dbus/dbus-connection.c (_dbus_connection_new_for_transport):
  Unlock connection on error
  (generate_local_error_message): static method for generating
  an error message when we don't have a message to reply to
  (_dbus_connection_block_pending_call): Send a disconnect error
  instead of just a timeout (NULL) when the bus gets disconnected
  while blocking for a reply.

* dbus/dbus-connection.c (dbus_connection_dispatch): Properly remove
 filters (Patch from Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)

* dbus/dbus-connection-internal.h: Add prototype for
  _dbus_connection_test_get_locks to remove warning

* dbus/dbus-spawn.c (_dbus_spawn_async_with_babysitter): Make sure we
  close child_err_report_pipe[WRITE_END] on exec (Patch from
  Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)

* Clean up close calls to use _dbus_close (Patch from
  Kimmo Hämäläinen <kimmo dot hamalainen at nokia dot com>)

* doc/TODO:
- Remove pending call locking todo item
- dbus_connection_open now holds hard ref.  Remove todo item
- do proper locking on _dbus_bus_check_connection_and_unref
  and handle DBUS_BUS_STARTER. Remove todo item
- Warn on closing of a shared connection.  Remove todo item

* bus/bus.c, bus/connection.c, bus/dispatch.c, dbus/dbus-bus.c,
  dbus/dbus-connection.c: Use the dbus_connection_close_internal
  so we don't get the warning when closing shared connections

* test/test-service.c, test/test-shell-service.c: Applications
  don't close shared connections themselves so we unref instead of
  close

* test/test-utils.c (test_connection_shutdown): Close the connection

* dbus/dbus-bus.c (_dbus_bus_check_connection_and_unref): Changed to
  _dbus_bus_check_connection_and_unref_unlocked since we only call this
  method on a locked connection.
  Make sure we call _dbus_connection_unref_unlocked instead of
  dbus_connection_unref also.
  Handle DBUS_BUS_STARTER correctly

* dbus/dbus-connection.c (connection_record_shared_unlocked):
  Mark as shared and hard ref the connection
  (connection_forget_shared_unlocked): Remove the hard ref from the
  connection
  (_dbus_connection_close_internal_and_unlock):  New internal function
  which takes a locked connection and unlocks it after closing it
  (_dbus_connection_close_internal): New internal function which acts
  like the origonal dbus_connection_close method by grabbing a connection
  lock and calling _dbus_connection_close_internal_and_unlock
  (dbus_connection_close): Public close method, warns when the app
  trys to close a shared connection

* bus/driver.c:
  (bus_driver_generate_introspect_string): New method for populating
  a DBusString with the introspect data
  (bus_driver_handle_introspect): Move introspect generation code to
  bus_driver_generate_introspect_string

* bus/main.c:
  (introspect): New function which prints out the intropect data and
  exits
  (main): Add a --introspect switch

* doc/TODO: Removed dtd publishing item.
  It seems the dtd has already been added at
  http://www.freedesktop.org/standards/dbus/1.0/introspect.dtd

* doc/TODO, various source files: Audited todo's and FIXME's and
  prepended the ones we should be looking at with 1.0.  Those
  prepended with 1.0? need clerification or might not be needed
  for 1.0

* dbus/dbus-pending-call.c: Add some checks so we don't crash on
  NULL in the public API (Patch from Kimmo Hämäläinen
  <kimmo dot hamalainen at nokia dot com>)

* configure.in: Fix for OS-X compile time endian issues (patch from
  Benjamin Reed <ranger at befunk dot com>)

* configure.in: Check for no-common compiler flag (OS-X fix from
  Benjamin Reed <ranger at befunk dot com>)

* tools/dbus-launch.c: Add a sigterm handler (patch from Frederic Crozat
  <fcrozat at mandriva dot com>)

2006-08-29  Havoc Pennington  <hp@redhat.com>

* test/test-service.c (path_message_func): fix lack of return value

* dbus/dbus-sysdeps.c (_dbus_printf_string_upper_bound): fix
formatting, remove #ifdef, and fix docs. #ifdef doesn't make
any more sense than on anything else in this file.
(_dbus_get_tmpdir): add const to return value, and keep the
results of the various getenv around in a static variable.

2006-08-29  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-sysdeps-util.c, dbus/dbus-sysdeps-util-unix.c: change
from Ralf Habacker to move UNIX-specific sysdeps into a separate file.

* test/Makefile.am: change find to use syntax that works with non
  gnu versions of find

* dbus/dbus-transport.c: fix invalid deref when checking if
  a vtable method exists (Patch from Christian Ehrlicher
  <ch dot ehrlicher at gmx dot de>)

* configure.in, dbus/Makefile.am, test/name-test/run-test.sh,
  test/name-test/Makefile.am: Fixed some issues with getting get dbus
  to build with builddir != srcdir (Taken from a patch by Bernard Leak
  <bernard at brenda-arkle dot demon dot co dot uk>)

* configure.in: Fix DBUS_DAEMONDIR to use EXPANDED_BINDIR for the
  default case

- document --enable-kqueue

 * configure.ac, bus/Makefile.am: Generalize kqueue support so that
   it works on any system providing this interface, not only FreeBSD.
   For example, NetBSD. (Patch by Julio M. Merino Vidal  <jmmv at NetBSD dot org>)

add some items related to shared connections

2006-08-20  Havoc Pennington  <hp@redhat.com>

* doc/dbus-faq.xml, doc/dbus-tutorial.xml: some improvements to
the docs

- bump release

* Released 0.92

* dbus/dbus-threads.c (dbus_threads_init): change the documentation
  to reflect the init late change

* bus/bus.c (bus_context_new): Check user before we fork so we can
  print out an error message a user will be able to see

Patch provided by Ralf Habacker (ralf dot habacker at freenet dot de)

* dbus/dbus-sysdeps.c, dbus/dbus-threads.c, dbus/dbus-internals.h:
  Add two more global locks for use on windows platforms.  These are
  unused on non-windows platforms but are not ifdefed out to avoid
  potential bugs (i.e. the few bytes lost does not warrent the extra
  maintanence and complexity that having seperate sets of locks would
  cause)

2006-08-17  Alp Toker  <alp@atoker.com>

* doc/dbus-specification.xml: Fix some minor typos.

    * configure.in: use  instead of
        to determine where to store the pid since the init scripts manipulate
        the pid file (patch from Marcelo Ricardo Leitner
        <marcelo dot leitner at terra dot com dot br>.

* commit the rest of the ChangeLog

    * dbus/dbus-threads.c: Add static DBusList *uninitialized_mutex_list and
        static DBusList *uninitialized_condvar_list to support new late
        initialization threading model.  In this model threads can be initialized
        even after the D-Bus API has been used but still needs to be initialized
        before the second thread has been started.  Mutexes and condvar addresses
        are stored in the two static lists and are replaced with actuall locks
        when threads are initalized.
        (_dbus_mutex_new_at_location): New method for creating a mutex and placing
        the location into the static list
        (_dbus_mutex_free_at_location): New method for removing a mutex location
        from the static list and freeing the mutex
        (_dbus_condvar_new_at_location): New method for creating a conditional
        variable and placing the location into the static list
        (_dbus_condvar_free_at_location): New method for removing a conditional
        variable location from the static list and freeing the conditional variable
        (init_uninitialized_locks): Atomic method which goes through the static
        lists of mutex and condvar location and updates them with actuall locks
        (init_global_locks): changed to init_locks

        * dbus/dbus-connection.c:
        (_dbus_connection_test_get_locks): New method for tests to check connections
        (_dbus_connection_new_for_transport): Use the new at_location mutex and
        condvar API
        (dbus_connection_allocate_data_slot): Pass in the global lock address
        to _dbus_data_slot_allocator_alloc

        * dbus/dbus-dataslot.c:
        (_dbus_data_slot_allocator_alloc): Use the address of the mutex
        instead of the mutex itself

        * dbus/dbus-message.c:
        (dbus_message_allocate_data_slot): Pass in the global lock address
        to _dbus_data_slot_allocator_alloc

        * dbus/dbus-pending-call.c:
        (dbus_pending_call_allocate_data_slot): Pass in the global lock address
        to _dbus_data_slot_allocator_alloc

        * dbus/dbus-server.c:
        (_dbus_server_init_base): Use the new at_location mutex API
        (dbus_server_allocate_data_slot): Pass in the global lock address
        to _dbus_data_slot_allocator_alloc

        * test/name-test/test-threads-init.c: New test case for late thread
        initialization

 * dbus/dbus-dataslot.c (_dbus_data_slot_allocator_alloc):
        Change _dbus_abort to _dbus_assert_not_reached because _dbus_abort
        causes compile problems when asserts are turned off
        Keeping _dbus_warn for printing out the message so even if
        asserts are turned off the user gets the messages that something is
        wrong

Patches by Kjartan Maraas <kmaraas at gnome dot org>

        * bus/services.c (bus_service_list_queued_owners):
        Add a pointer cast to fix compiler warning

        * dbus/dbus-dataslot.c (_dbus_data_slot_list_get):
        return a NULL instead of FALSE since the return type
        is not expecting a boolean

        * dbus/dbus-marshal-basic.c (_dbus_marshal_test):
        Remove unused variable

        * dbus/dbus-marshal-recursive-util.c (node_new):
        return a NULL instead of FALSE since the return type
        is not expecting a boolean

        * dbus/dbus-server-debug-pipe.c (_dbus_transport_debug_pipe_new):
        Send a NULL into _dbus_transport_new_for_fd instead of a FALSE
        because we are expecting a pointer not a boolean

        * dbus/dbus-sysdeps-util.c (_dbus_get_tmpdir):
        add void as the parameter so some compilers
        don't complain

        * dbus/dbus-transport-unix.c (_dbus_transport_new_for_domain_socket,
          _dbus_transport_new_for_tcp_socket):
        Send a NULL into _dbus_transport_new_for_fd instead of a FALSE
        because we are expecting a pointer not a boolean

        * test/shell-test.c (test_command_line):
        cast the second argument to _dbus_list_append to avoid compiler
        warnings

        * test/test-names.c (main): remove unused variable

        * test/test-service.c (check_hello_from_self_reply):
        Initialize echo_message and echo_reply to NULL

        * test/test-shell-service.c (handle_echo):
        Remove unused variable and cast the third parameter passed to
        dbus_connection_get_object_path_data to avoid compiler warrnings

        * test/name-test/test-names.c (clear_message_queue):
        Remove unused function

        * test/name-test/test-pending-call-dispatch.c:
        Fix format string in printf

    * dbus/dbus-bus.c:
        * test/name-test/test-names.c:
        Remove test method dbus_bus_connection_get_unique_name because
        we already have public method dbus_bus_get_unique_name that
        does the exact same thing

* fix typos in Julio's previous patch which cause make check to fail

* fix mismatching paren typo

* dbus/dbus-address.c (_dbus_address_test): Revert leaking strcmp.
  In any case it was wrong since this is a test checking to see if
  address parsing is correct.  There was no need to get the true
  tmp directory.

* dbus/dbus-macros.h: Revert the addition of stddef.h
  as we should not be adding it to library headers

* dbus/dbus-signature.c:
  Fix the unit tests so that they work if assertions are not enabled.
  (patch from Julio M. Merino Vidal  <jmmv at NetBSD dot org>)

* tools/run-with-tmp-session-bus.sh:
* test/name-test/run-test.sh:
  Remove bashisms (patch from Julio M. Merino Vidal
  <jmmv at NetBSD dot org>)

- configure.in doesn't seem to like >= 2.6.0 in a warning message

* configure.in: add a version (>= 2.6.0) check for libxml2

* forgot to save the dbus/dbus-macros.h file for a previous checkin

* configure.in: make sure the compiler supports -Wfloat-equal

* bus/dir-watch-dnotify.c: move functional code after
  variable declerations (C99 fix)  (patches from Jens Granseuer
  <jensgr at gmx dot net>

* dbus/dbus-macros.h:
  add #include <stddef.h> so that Sun compilers don't complain about
  the defining NULL

* dbus/dbus-sysdeps.c:
* dbus/dbus-address.c:
* bus/activation.c:
* test/shell-test.c:
  don't hardcode tmp directory (patch from Dave Meikle
  <dmk at davemeikle dot co dot uk>)

* Change an assert to a detailed warning and abort.
  When allocator->lock != mutex it indicates that the user has failed
  to initalize threads before using the D-Bus library.  This warning
  helps the user identify the issue and fix their app.

* missing semicolon

These are all patches from Kjartan Maraas <kmaraas at gnome dot org>
with cleanups of bugs found from Coverity reports:

* dbus/dbus-sysdeps-util.c (_dbus_write_pid_file):
  close the file on error to avoid a leak

* bus/expirelist.c (bus_expire_list_test):
  Check for NULL on dbus_new0

* bus/activation.c (update_directory):
  remove dead code

* bus/config-parser.c (merge_service_context_hash, start_selinux_child):
  Fix some leaks

* bus/bus.c (process_config_every_time):
  Fixed a leak

* bus/desktop-file.c (parse_key_value):
  Fixed leak

* bus/selinux.c (bus_selinux_id_table_insert):
  Fixed leak

* dbus/dbus-object-tree.c (_dbus_object_subtree_new):
  remove dead code

* tools/run-with-tmp-session-bus.sh: use #!/bin/sh instead of
  #!/bin/bash

* Doxyfile.in: fix the INPUT line so we can generate docs
  when sourcedir != builddir (patch from Cygwin Ports maintainer
  <yselkowitz at users dot sourceforge dot net>

* dbus/dbus-sysdeps.h:
* dbus/dbus-sysdeps.c:
* dbus/dbus-string.c:
  s/_dbus_printf_length/_dbus_printf_string_upper_bound to comform with
  GLib's function which does the same thing

* configure.in:
* bus/Makefile.am:
* bus/dir-watch-default.c:
* bus/dir-watch-dnotify.c:
* bus/dir-watch-kqueue.c:
  Add kqueue directory watching for freebsd and split the directory
  watching code into seperate files per method/arch
  (patches from Timothy Redaelli <drizzt at gufi dotorg>)

* configure.in:
* tools/Makefile.am:
* tools/dbus-launch.c:
* bus/Makefile.am:
  allow --with-dbus-daemondir switch to be used to make the
  daemon install to a seperate bindir like /usr/libexec
  (patch from Brian Cameron <brian dot cameron at sun dot com)

* bus/Makefile.am (install-data-hook): removed the slash after
  $(DESTDIR) so we don't get the double slash which does not work
  in windows (patch from Andras Porjesz
  <andras dot porjesz at siemens dot com>

* dbus/dbus-sysdeps.h:
* dbus/dbus-sysdeps.c:
  add _dbus_printf_length (patch from Peter Kümmel
  <syntheticpp at gmx dot net>)

* dbus/dbus-internals.c: unistd.h is not used
  (_dbus_verbose_real): only enable verbose printing
  if DBUS_VERBOSE environment var is set to '1'
  (patch from Peter Kümmel <syntheticpp at gmx dot net>

* confifure.in: add a GCC flag check for -Wdeclaration-after-statement
  so we still compile on older gcc (patch from Frederic Peters
  <fpeters at 0d dot be>

* configure.in: add -Wdeclaration-after-statement

* dbus/dbus-connection.c: change all the pending call stuff to
  reflect the fact that pending call operations use the connection
  lock

* dbus/dbus-pending-call.c: add locking here

* dbus/dbus-errors.c (struct DBusRealError): don't make the name
  field const consistent with how message field is done

* s/D-BUS/D-Bus/g

* dbus/dbus-object-tree.c: Patch by Benjamin Otte
  <in7y118 at public.uni-hamburg dot de> - fix invalid
  read/write reported by valgrind

- bump release

- Released 0.91

- Correct release location

* dbus/dbus-connection.c:
  (_dbus_connection_attach_pending_call_unlocked):
  (connection_timeout_and_complete_all_pending_calls_unlocked):
  Make sure we set timeout_added on pending calls to FALSE when
  we remove the timeout from the connection

add note about fixing locking on pending call

* Removed some extra bindings stuff lingering around (thanks timo)
* dbus-pendingcall.c (_dbus_pending_call_new):
  s/dbus_connection_ref/_dbus_connection_ref_unlocked fixes assertion
  when we tried to take a lock on an already locked connection

- Bump release in CVS

* Release 0.90

2006-07-17  Havoc Pennington  <hp@redhat.com>

        * dbus/dbus-marshal-basic.c (_dbus_type_to_string): support 64-bit
        ints, reported by Owen Taylor

* doc/TODO:
* dbus/dbus-bus.c:
* dbus-errors.c:
* dbus/dbus-marshal-validate.c:
  Removed TODO items which were fixed or are no longer relevent

* Remove stray files from the stripped bindings

2006-07-16  Havoc Pennington  <hp@redhat.com>

* dbus/dbus-pending-call.c (_dbus_pending_call_set_timeout_error):
Improve the error message on reply timeout

* Update ChangeLog for binding removal commit

* Remove all bindings

* dbus-connection.c (dbus_connection_send_with_reply): return TRUE
  and set pending_reply out arg to NULL is connection is disconnected
  (connection_timeout_and_complete_all_pending_calls_unlocked): New
  static method for cleaning up pending calls on disconnect
  (_dbus_connection_get_dispatch_status_unlocked): If we have pending
  calls queued timeouts on disconnect

* dbus/dbus-pending-call.ci (_dbus_pending_call_set_connection):
  Remove

* bus/activation.[ch] (bus_activation_list_services): new function to
  get the list of services that can be activated

* bus/dispatch.c: test coverage for the new bus method
  ListActivatableNames

* bus/driver.c: new bus method ListActivatableNames to get the list of
  services that can be activated

* doc/dbus-specification.xml: ListActivatableNames method documentation

* dbus/Makefile.am: add dbus-pending-call-internal.h to the list of
  source files

* dbus/dbus-message-factory.c:
  Fix index into an array (patch by Peter Kümmel <syntheticpp at gmx.net>)

- Make DBusPendingCall an opaque type even to D-Bus internals

* dbus/dbus-connection.h: remove connection_disconnect and replace with
  connection_close

* dbus/dbus-connection.c: include dbus-bus.h
  (_dbus_connection_read_write_dispatch): make static

* dbus/dbus-connection.c (dbus_connection_close): removed deprecated
  function
  (dbus_connection_dispatch): On disconnect unref any shared connections

* dbus/dbus-bus.c (_dbus_bus_check_connection_and_unref): new function
  for cleaning up shared connections on disconnect
  (internal_bus_get): get a hard refrence to shared connections when
  they are created

* doc/TODO: Remove items which are no longer relevent or have been fixed
  Split 1.0 todo items with a 0.90 freeze todo list

2006-06-14  Ross Burton  <ross@openedhand.com>

* glib/dbus-gobject.c:
Free a leaking GArray (surely not!) in dbus_g_method_return.

2006-06-14  Ross Burton  <ross@openedhand.com>

* tools/Makefile.am:
* tools/dbus-monitor.c:
Don't use the GLib bindings in dbus-monitor (patch from Ralf
Habacker).

2006-06-14  Ross Burton  <ross@openedhand.com>

* tools/dbus-print-message.c:
Also print the object path when outputting signals or method calls.

add dbus_bus_get refcount item to TODO
link to mail list post about pending call threading problems

    * qt/src/Makefile.am: install the qdbus.h header.
        This allows people to actually use the installed code.

2006-06-12  Ross Burton  <ross@openedhand.com>

* glib/dbus-gproxy.c:
Don't leak a GArray when firing signals (thank Rob Taylor for review).

* Released 0.62
(update ChangeLog and NEWS)

* dbus/dbus-arch-deps.h.in: Remove spurious semi-colons that
        break pedantic builds. Closes bug 6043 (patch approved by
        Havoc back in February).

* qt/src/qdbusintegrator.cpp: Fix bug in parsing async methods
        that took a QDBusMessage parameter.
* qt/src/qdbusbus.h: Add a default flag for RequestName.
* qt/tools/dbus.cpp: Don't use automatic call because we might
        be calling an async method: request a reply.

No changelog: fix distcheck.

* test/qt/*: Update the testcases, including testing the new
        functionality of sending null QByteArray and QString over the
        bus. Add new headertest test and restore the old
        qdbusxmlparser test.

* qt/tools/dbuscpp2xml.cpp: Compile on Windows.
* qt/tools/dbusidl2cpp.cpp: Add missing newline.

* qt/examples/Makefile.am:
* qt/examples/chat.h: Use UI-generated files with the ui_*.h
form.

* qt/src/qdbusmarshall.cpp: Allow sending of QString() and
QByteArray() (nulls) over the bus.
* qt/src/qdbusabstractinterface.cpp: Use the correct variable,
the one that has the signature suffix stripped.
* qt/src/qdbusreply.h: Make some methods const.

    Patch from Timo Hoenig <thoenig@suse.de>.

        * qt/dbus/Makefile.am: New file.  Fix "make dist", add all
        headers required during build to EXTRA_DIST.
        * qt/src/Makefile.am: Fix "make dist", add 'qdbus.h' to
        EXTRA_DIST.
        * qt/Makefile.am: Fix "make dist", add 'dbus' to DIST_SUBDIRS.
        * configure.in: Fix "make dist", take care that the Makefile
        for qt/dbus is being generated.

* bus/bus.c: Fix eavesdropping on method calls

* configure.in:
* dbus/dbus-userdb-util.c:
Add Solaris console owner patch from Artem Kachitchkine

Managed to get the year wrong in April as well as yesterday...

* qt/Makfile.am:
* qt/src/Makefile.am: Fix the EXTRA_DIST after the
reorganisation. Thanks to Timo Hoenig for pointing this out.

2005-05-06  Robert McQueen  <robot101@debian.org>

* glib/dbus-gtype-specialized.c: Fix obvious leak of GArray in every
  call to dbus_g_type_get_struct.

2005-05-06  Robert McQueen  <robot101@debian.org>

* glib/dbus-gvalue-utils.c: Fix the failing test where static string
  pointers were put into a GPtrArray-based specialised collection, and
  then freed along with the array. GValues which you add into
  collections or maps which have the NOCOPY flag set are assumed to not
  belong to the caller, so rather than the existing pointer-stealing
  semantics, they are copied instead. Given that the main consumers of
  this abstraction are the bindings themselves, I don't think this is
  too bad, but others should watch their choice of take vs set_static.