summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add session supportwip/sessionMatthias Clasen2011-12-177-4/+572
| | | | | This is a port of the eggsmclient dbus implementation, plus an inhibit api.
* GApplication: receiving end of GRemoteActionGroupRyan Lortie2011-12-173-8/+100
| | | | | | | | Use the fact that the action group exporter now speaks to GRemoteActionGroup to get the platform data into GApplicion without hacks. https://bugzilla.gnome.org/show_bug.cgi?id=665737
* action group exporter: kill GApplication hackeryRyan Lortie2011-12-171-25/+13
| | | | | | Use the GRemoteActionGroup interface, if available, instead. https://bugzilla.gnome.org/show_bug.cgi?id=665737
* introduce GRemoteActionGroupRyan Lortie2011-12-1713-148/+303
| | | | | | | | | | | | | | | This interfaceifies the extra functions that were on GDBusActionGroup for dealing with platform data. The two main benefits of doing this: - no longer have to do a silly song and dance in GApplication to avoid calling GDBusActionGroup API from non-dbus-aware code - the interface can be reused by the action group exporter to avoid ugly and unbindable hook callbacks https://bugzilla.gnome.org/show_bug.cgi?id=665737
* GApplication: send platform data for actions againRyan Lortie2011-12-163-9/+39
| | | | | Use the _full variants of the GDBusActionGroup API to send platform data for action invocations once again.
* GDBusActionGroup: add _full variants of activationRyan Lortie2011-12-164-13/+95
| | | | This allows the platform_data to be explicitly specified.
* Revert "GDBusActionGroup: add static platform registration"Ryan Lortie2011-12-164-52/+5
| | | | This reverts commit fcc9902e98a6a6568a1150441c8680fd4615d437.
* GApplication: emit signals on action changesRyan Lortie2011-12-161-1/+16
| | | | | | | | | | Now that we're a GActionMap the story about propagating signals from our (now-constant) internal action group is vastly simplified. If someone calls g_application_set_action_group() then signals will stop working -- but this function is deprecated and they never worked before, so no big loss there. https://bugzilla.gnome.org/show_bug.cgi?id=643736
* windows XP threads: fix hilariously obvious raceRyan Lortie2011-12-161-7/+12
| | | | | | | | | I tried to do a double-checked lock without the double check. Rodrigo Rivas Costa caught the problem and suggested the (obviously correct) fix. https://bugzilla.gnome.org/show_bug.cgi?id=666296
* GDBusActionGroup: add static platform registrationRyan Lortie2011-12-164-5/+52
| | | | | | | | | | | | | | | We provide a mechanism by which a 'platform' (eg: Gtk) can register some hook functions to be called to collect platform-data at the point of sending an outgoing action activation request and also to inform the platform of this data on incoming requests (before and after dispatching the actual request). This can be used for forwarding timestamp and startup-notification information (as is presently done in GApplication) but the before/after hook could also be used for acquiring/releasing the Gdk lock or other similar things. https://bugzilla.gnome.org/show_bug.cgi?id=665737
* Added API g_queue_free_full().Ravi Sankar Guntur2011-12-164-0/+77
| | | | | | | | | | | | | | g_queue_free_full(), to free a Queue including its dynamically-allocated elements. On similar lines to List and Slist. void g_queue_free_full (GQueue *queue, GDestroyNotify free_func); Test case covering g_queue_free_full() is added. Added export symbol to glib.symbols. Closes Bug: https://bugzilla.gnome.org/show_bug.cgi?id=657433 Signed-off-by: Ravi Sankar Guntur <ravi.g@samsung.com>
* GDBusActionGroup: Fix a typo in docsColin Walters2011-12-151-1/+1
|
* configure.ac: robustify netlink.h checkDan Winship2011-12-151-1/+2
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=666173
* GBytes: add a size argument to g_bytes_get_dataStef Walter2011-12-156-24/+41
| | | | | | | * An out size argument so that this is more easily bindable by gobject-introspection. https://bugzilla.gnome.org/show_bug.cgi?id=665879
* Initialize service_loop before running the service threadChris Coulson2011-12-141-1/+1
| | | | Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666129
* menu markup: tolerate not having a hash table of objectsMatthias Clasen2011-12-141-4/+5
| | | | | | | | Various places in the code were assuming that the hash table was always available. Fix this, and also avoid leaking strings now that the hash table may be NULL. Based on a patch by Simon McVittie, bug 666167
* g_array_free, g_ptr_array_free: decrement refcount if not the last refSimon McVittie2011-12-141-2/+2
| | | | | | | | | | | | | | | foo_free is conceptually "worth" one unref; not decrementing the refcount here means the GArray or GPtrArray wrapper (but not its contents) would leak in the following call sequence: p = g_ptr_array_new (); g_ptr_array_ref (p); g_ptr_array_free (p, TRUE); g_ptr_array_unref (p); Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* GArray, GPtrArray: factor out the actual freeingSimon McVittie2011-12-141-14/+43
| | | | | | | | | Depending how the array is freed, we may want to free the underlying array (the "segment"), the struct wrapper or both. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* g_key_file_get_string_list: don't leak the pieces on errorSimon McVittie2011-12-141-0/+1
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* tls-interaction test: use a weak pointer instead of a deliberate use-after-freeSimon McVittie2011-12-141-1/+6
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666115 Reviewed-by: Dan Winship <danw@gnome.org>
* gdbus tests: remove buggy use of GMainLoopRyan Lortie2011-12-141-25/+0
| | | | | | | | | | | | g_main_loop_quit() only quits mainloops that are currently running -- not ones that may run in the future. The way the gdbus-threading tests are written can possibly result in a call to g_main_loop_quit() before g_main_loop_run() has started. The mainloops aren't actually used for anything other than signalling the completion of the threads, so just use g_thread_join() for that. https://bugzilla.gnome.org/show_bug.cgi?id=666129
* two test fixes for ARMRyan Lortie2011-12-142-12/+12
| | | | | | | First, some ARM systems are not fast enough to meet the 30 second deadline in gwakeuptest.c, so increase that to 60. Second, we have some signed/unsigned woes in the gparam transform tests.
* Plug some leaks in the GIO testsSimon McVittie2011-12-145-0/+14
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666115 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* hash test: avoid leaking various keys and valuesSimon McVittie2011-12-141-7/+75
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666115 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* GOptionContext test: free all arguments, not just the remaining onesSimon McVittie2011-12-141-88/+265
| | | | | | | | | | | | | | On success, g_option_context_parse alters argv by removing options that it understood, so g_strfreev is insufficient. Instead, take a shallow copy and free all of the arguments in that, then free the array argv but not its contents. Also, improve the checks in error cases, by checking that argv has not been altered in this way. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666115 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* various GLib tests: plug memory leaksSimon McVittie2011-12-1426-20/+137
| | | | | | | | | These don't really matter, since it's test code, but they do obscure real leaks in the library. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666115 Acked-by: Matthias Clasen <mclasen@redhat.com>
* testglib: test_file_functions: don't close fd if it's -1Simon McVittie2011-12-141-1/+2
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666115 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* Revert "g_menu_markup_start_element: tolerate not having a hash table of ↵Simon McVittie2011-12-141-1/+1
| | | | | | | objects" This reverts commit bcdb60641196f98482b5e17fcc96a92589deba35, accidentally pushed before it was reviewed.
* g_menu_markup_start_element: tolerate not having a hash table of objectsSimon McVittie2011-12-141-1/+1
| | | | | I don't know whether this is the right fix, but gio/tests/gmenumodel crashes out, which can't be right.
* g_variant_byteswap: don't leak serialised.dataSimon McVittie2011-12-141-1/+1
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* GDBusActionGroup: don't leak param_strSimon McVittie2011-12-141-1/+1
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* g_data_set_internal: avoid use-after-free if datalist is in datasetSimon McVittie2011-12-141-2/+8
| | | | | | | | | | Removing the last thing in a dataset frees the dataset, and if the datalist was in a dataset, we can't safely unlock it after the dataset has been freed. Unlock it sooner. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* g_strcompress: check that source is non-NULL rather than just crashingSimon McVittie2011-12-141-2/+7
| | | | | | | | | | | | Calling this function with a NULL argument is considered to be invalid, but one of the regression tests does it anyway (to watch it crash), which seems a good indication that it's expected to be somewhat common. Let's check it rather than segfaulting. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com> Reviewed-by: Matthias Clasen <mclasen@redhat.com>
* g_dbus_action_group_changed: don't leak iterator and its contentsSimon McVittie2011-12-141-0/+1
| | | | | | Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* GKeyFile: free group comments when the group is removedSimon McVittie2011-12-141-0/+6
| | | | | | | | These were leaked. Valgrind was sad. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* g_hmac_get_string: don't allocate and leak an unused bufferSimon McVittie2011-12-141-1/+5
| | | | | | | | Also document why we're not actually using the buffer for anything. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* g_hmac_copy: initialize the refcountSimon McVittie2011-12-141-0/+1
| | | | | | | | | In practice, the uninitialized refcount will typically mean that the copy is never freed, and leaks. Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugzilla.gnome.org/show_bug.cgi?id=666113 Reviewed-by: Emmanuele Bassi <ebassi@linux.intel.com>
* Add an explicit deprecation note to g_thread_initMatthias Clasen2011-12-141-0/+4
| | | | https://bugzilla.gnome.org/show_bug.cgi?id=666145
* Add another menu parser testMatthias Clasen2011-12-141-16/+46
| | | | This one concerns itself with objects.
* GMenuModel: Flesh out the docs some moreMatthias Clasen2011-12-131-1/+11
|
* GMenu: purge references to GMenuProxy from the docsMatthias Clasen2011-12-132-3/+3
|
* documentation fixesStef Walter2011-12-1328-127/+169
| | | | | | | | Fixes for gtk-doc warnings. http://bugzilla.gnome.org/show_bug.cgi?id=66469 https://bugzilla.gnome.org/show_bug.cgi?id=664699
* Improve GNetworkMonitor docsMatthias Clasen2011-12-132-3/+23
| | | | | Make sure we get signal and property docs, and mention the extension point in the overview.
* gnetworkmonitornetlink.c: Fix compilation on RHEL 6.2Christophe Fergeau2011-12-131-2/+5
| | | | | | | | | | | | | | | | | | | When trying to compile glib master on a RHEL 6.2 system, it fails with: make[4]: Entering directory `/home/teuf/gnome/src/glib/gio' CC libgio_2_0_la-gnetworkmonitornetlink.lo In file included from gnetworkmonitornetlink.c:25: /usr/include/linux/netlink.h:35: error: expected specifier-qualifier-list before 'sa_family_t' gnetworkmonitornetlink.c: In function 'g_network_monitor_netlink_initable_init': gnetworkmonitornetlink.c:99: error: 'struct sockaddr_nl' has no member named 'nl_family' gnetworkmonitornetlink.c:100: error: 'struct sockaddr_nl' has no member named 'nl_pid' gnetworkmonitornetlink.c:100: error: 'struct sockaddr_nl' has no member named 'nl_pad' gnetworkmonitornetlink.c:101: error: 'struct sockaddr_nl' has no member named 'nl_groups' make[4]: *** [libgio_2_0_la-gnetworkmonitornetlink.lo] Error 1 sa_family_t is defined in sys/socket.h, this commit makes sure this header is included before netlink.h This fixes bgo bug #666001
* bump versionRyan Lortie2011-12-131-1/+1
|
* Update VS property sheetsChun-wei Fan2011-12-132-3/+39
| | | | | "Install" the newly-added public headers in GLib and GIO, such as the ones for GMenuModel, etc.
* Fix wordingAlexandre Franke2011-12-121-1671/+1791
| | | | délai épuisé has been replaced with délai dépassé
* glib 2.31.42.31.4Ryan Lortie2011-12-122-4/+15
|
* GActionGroupExporter: stop using signal IDsRyan Lortie2011-12-121-16/+16
| | | | | | | | | | | | | | | GDBusConnection recently changed to dispatching its GDestroyNotify calls from an idle instead of on-the-spot. Under the previous regime, we would destroy-notify the action group export of a GtkApplicationWindow at the point it was removed from the application (ie: slightly before being disposed). With the destroy notify now deferred to an idle, the window has already been disposed, so the signal handlers have already been disconnected. Avoid the problem by dropping our use of signal IDs and just do g_signal_handlers_disconnect_by_func(), which doesn't complain if there is no connection.
* GMenuMarkup parser: small typo fix in docsRyan Lortie2011-12-121-1/+1
|