2008-11-29 01:07:33 +01:00
|
|
|
|
2008-11-28 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 547264 – Missing "no flags" flag
|
|
|
|
|
|
|
|
|
|
* glib/gregex.c: Mention 0 as value for 'no flags' in some places..
|
|
|
|
|
Pointed out by Bastien Nocera
|
|
|
|
|
|
2008-11-28 22:22:21 +01:00
|
|
|
|
2008-11-28 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 562544 – g_key_file_get_string and g_key_file_get_value
|
|
|
|
|
documentation does not explain the difference
|
|
|
|
|
|
|
|
|
|
* glib/gkeyfile.c (g_key_file_get_string): Explain the difference
|
|
|
|
|
to g_key_file_get_value(). Pointed out by Mart Raudsepp.
|
|
|
|
|
|
2008-11-28 21:29:42 +01:00
|
|
|
|
2008-11-28 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
* glib/gkeyfile.c: Refer to @locale parameters as locale identifier
|
|
|
|
|
in docs.
|
|
|
|
|
|
2008-11-28 21:17:15 +01:00
|
|
|
|
2008-11-28 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 557603 – carbon check output misplaced
|
|
|
|
|
|
|
|
|
|
* configure.in: Call AC_PROG_CPP early to avoid it messing up
|
|
|
|
|
carbon check output. Fix proposed by Christian Persch
|
|
|
|
|
|
2008-11-28 07:24:51 +01:00
|
|
|
|
2008-11-28 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 559110 – Do not include libintl.h after glibintl.h
|
|
|
|
|
|
|
|
|
|
* glib/glibintl.h: Define bind_textdomain_codeset in the DISABLE_NLS
|
|
|
|
|
branch. Patch by Peter Kjellerstedt.
|
|
|
|
|
|
|
|
|
|
* glib/gutil.c: Don't include libintl.h directly.
|
|
|
|
|
|
2008-11-28 06:17:27 +01:00
|
|
|
|
2008-11-28 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 562378 – callback return value not respected for callback option
|
|
|
|
|
with no arg
|
|
|
|
|
|
|
|
|
|
* glib/goption.c (parse_long_option): Return the parse_arg return
|
|
|
|
|
value even for no-arg callbacks. Patch by Christian Persch
|
|
|
|
|
|
|
|
|
|
* glib/tests/option-context.c: Add a test for a callback which
|
|
|
|
|
returns FALSE.
|
|
|
|
|
|
2008-11-23 22:16:06 +01:00
|
|
|
|
2008-11-23 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 559413 – g_option_group_set_error_hook docs buglet
|
|
|
|
|
|
|
|
|
|
* glib/goption.c: Doc fix.
|
|
|
|
|
|
2008-11-23 22:16:01 +01:00
|
|
|
|
2008-11-23 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 560568 – gkeyfile docs buglet
|
|
|
|
|
|
|
|
|
|
* glib/gkeyfile.c: Clarify the docs.
|
|
|
|
|
|
2008-11-23 22:15:57 +01:00
|
|
|
|
2008-11-23 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 560569 – gkeyfile doesn't use the set list_separator in some cases
|
|
|
|
|
|
|
|
|
|
* glib/gkeyfile.c: (g_key_file_get_locale_string_list),
|
|
|
|
|
(g_key_file_set_locale_string_list), (g_key_file_set_integer_list),
|
|
|
|
|
(g_key_file_set_double_list): Use the key file's list separator character,
|
|
|
|
|
not the default one.
|
|
|
|
|
|
|
|
|
|
* glib/tests/keyfile.c: (test_lists), (test_reload_idempotency): Test
|
|
|
|
|
this.
|
|
|
|
|
|
2008-11-21 11:30:43 +01:00
|
|
|
|
2008-11-21 Christophe Fergeau <teuf@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 561212 – GFileReadMoreCallback API doc refers to non-existant function
|
|
|
|
|
|
|
|
|
|
* gio/giotypes.h: fix name of function referred to in
|
|
|
|
|
GFileReadMoreCallback API documentation
|
|
|
|
|
|
2008-11-21 08:28:50 +01:00
|
|
|
|
2008-11-21 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
* configure.in: Bump gtk-doc dependency to 1.11 for
|
|
|
|
|
nicer index-generation.
|
|
|
|
|
|
2008-11-14 20:59:34 +01:00
|
|
|
|
2008-11-14 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
* NEWS: Updates
|
|
|
|
|
|
2008-11-04 18:01:19 +01:00
|
|
|
|
2008-11-04 Christian Dywan <christian@imendio.com>
|
|
|
|
|
|
|
|
|
|
Bug 558672 – NULL key lookup using g_hash_table_lookup_extended()
|
|
|
|
|
|
|
|
|
|
* glib/ghash.c:
|
|
|
|
|
Clarify g_hash_table_lookup_extended
|
|
|
|
|
|
2008-11-02 20:43:52 +01:00
|
|
|
|
2008-11-02 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
* configure.in: Expand also build/win32/vs9/Makefile.
|
|
|
|
|
|
2008-10-31 10:56:19 +01:00
|
|
|
|
2008-10-31 Christian Dywan <christian@imendio.com>
|
|
|
|
|
|
|
|
|
|
Bug 558513 – g_warn_if_fail FIXME in gtestutils
|
|
|
|
|
|
|
|
|
|
* glib/gtestutils.c (g_test_log_send):
|
|
|
|
|
Turn g_assert into g_warn_if_fail as advised
|
|
|
|
|
|
2008-10-31 03:23:55 +01:00
|
|
|
|
2008-10-31 Grahame Bowland <grahame@angrygoats.net>
|
|
|
|
|
|
|
|
|
|
Bug 558185 – 'parent' variable in g_local_file_get_child_for_display_name()
|
|
|
|
|
hits g_object_unref(NULL) assertion
|
|
|
|
|
|
|
|
|
|
* gio/glocalfile.c:
|
|
|
|
|
- remove unused variable. Patch by Matt Johnston <matt@ucc.asn.au>
|
|
|
|
|
|
2008-10-29 16:14:06 +01:00
|
|
|
|
2008-10-29 16:11:14 Tim Janik <timj@gtk.org>
|
|
|
|
|
|
|
|
|
|
* glib/gmacros.h: added G_PASTE() and G_STATIC_ASSERT(), based on
|
|
|
|
|
patches by Christian Persch and Christian Dywan. Fixes:
|
|
|
|
|
Bug 558381 - Add support for compile time assertions
|
|
|
|
|
|
2008-10-27 23:59:42 +01:00
|
|
|
|
2008-10-27 Joseph Pingenot <gnomesvn@digitasaru.net>
|
|
|
|
|
|
|
|
|
|
* glib/ghash.c: changed "#GHashTableIterator" to "#GHashTableIter"
|
|
|
|
|
in the documentation-comments above g_hash_table_foreach_remove()
|
|
|
|
|
and g_hash_table_foreach_steal() to correctly name and link to
|
|
|
|
|
GHashTableIter. This affects the GNOME documentation website
|
|
|
|
|
as well, so further steps are likely necessary beyond this subproject..
|
|
|
|
|
|
2008-10-24 06:10:50 +02:00
|
|
|
|
2008-10-23 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
2008-10-24 06:11:26 +02:00
|
|
|
|
Bug 557210 – g_compute_checksum_for_* asserts with less than 2 bytes
|
|
|
|
|
|
2008-10-24 06:10:50 +02:00
|
|
|
|
* glib/gchecksum.c (g_compute_checksum_for_data): Accept
|
|
|
|
|
lengths < 2. Patch by Tommi Komulainen
|
|
|
|
|
|
|
|
|
|
* tests/checksum-test.c: Add a test for this
|
|
|
|
|
|
2008-10-24 05:55:29 +02:00
|
|
|
|
2008-10-23 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 556921 – gpoll.h breaks hal compilation
|
|
|
|
|
|
|
|
|
|
* glib/gpoll.h: Include gtypes.h. Pointed out by Anis Elleuch
|
|
|
|
|
|
2008-10-20 18:49:16 +02:00
|
|
|
|
2008-10-20 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 557087 – mem leak in g_content_types_get_registered
|
|
|
|
|
|
|
|
|
|
* gio/gcontenttype.c: Plug a mem leak.
|
|
|
|
|
|
2008-10-19 22:38:14 +02:00
|
|
|
|
2008-10-19 Michael Natterer <mitch@imendio.com>
|
|
|
|
|
|
|
|
|
|
Bug 556186 – gpoll.h breaks gmain.h inclusion
|
|
|
|
|
|
|
|
|
|
* glib/gpoll.h: different fix: disallow direct inclusion
|
|
|
|
|
unconditionally again but make an exception if included from
|
|
|
|
|
gmain.h to maintain compatibility.
|
|
|
|
|
|
2008-10-16 06:53:25 +02:00
|
|
|
|
2008-10-16 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
2008-10-17 02:06:03 +02:00
|
|
|
|
* configure.in: Bump version to 2.19.1
|
|
|
|
|
|
2008-10-17 01:59:50 +02:00
|
|
|
|
* === Released 2.19.0 ===
|
|
|
|
|
|
2008-10-16 06:53:25 +02:00
|
|
|
|
* NEWS: Updates
|
|
|
|
|
|
2008-10-14 12:11:29 +02:00
|
|
|
|
2008-10-14 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 556186 – gpoll.h breaks gmain.h inclusion
|
|
|
|
|
|
|
|
|
|
* glib/gpoll.h: Only disallow direct inclusion when
|
|
|
|
|
G_DISABLE_SINGLE_INCLUDES is defined.
|
|
|
|
|
|
2008-10-13 14:54:13 +02:00
|
|
|
|
2008-10-13 Sven Herzberg <sven@imendio.com>
|
|
|
|
|
|
|
|
|
|
Bug 556101 – static mutex yields warnings with g++
|
|
|
|
|
Reviewed by Tim Janik
|
|
|
|
|
|
|
|
|
|
* configure.in: added an intermediate cast to gpointer
|
|
|
|
|
|
2008-10-13 14:43:37 +02:00
|
|
|
|
2008-10-13 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 555311 – format not a string literal and no format arguments
|
|
|
|
|
|
|
|
|
|
* glib/gmarkup.c:
|
|
|
|
|
* glib/gshell.c: Use literal errors where appropriate.
|
|
|
|
|
|
2008-10-11 02:04:51 +02:00
|
|
|
|
2008-10-10 Behdad Esfahbod <behdad@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 551355 – [PATCH] Make glib build with libtool 2.2
|
|
|
|
|
|
|
|
|
|
* autogen.sh: Accept libtool 2.2. We are moving towards having
|
|
|
|
|
it working.
|
|
|
|
|
|
2008-10-11 01:55:38 +02:00
|
|
|
|
2008-10-10 Behdad Esfahbod <behdad@gnome.org>
|
|
|
|
|
|
|
|
|
|
* configure.in: With libtool 2.x, regenerate libtool early.
|
|
|
|
|
Based on bug 552107, patch by Patryk Zawadzki
|
|
|
|
|
|
2008-10-10 06:54:51 +02:00
|
|
|
|
2008-10-10 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 554557 – Patch to fix gcc warnings about missing format
|
|
|
|
|
specifiers
|
|
|
|
|
|
|
|
|
|
* glib/gmarkup.c:
|
|
|
|
|
* glib/gshell.c:
|
|
|
|
|
* glib/grel.c: Fix gcc warnings. Patch by Kjartan Maraas
|
|
|
|
|
|
2008-10-10 06:44:53 +02:00
|
|
|
|
2008-10-10 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 552861 – glib-2.0.m4 calls system(3) without storing its result
|
|
|
|
|
|
|
|
|
|
* m4macros/glib-2.0.m4: Cosmetic change to make -Werror happy.
|
|
|
|
|
Patch by Andreas Köhler
|
|
|
|
|
|
2008-10-08 22:54:39 +02:00
|
|
|
|
2008-10-08 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 555313 – GFileAttribute boxed type get_type function should
|
|
|
|
|
use usual get_type pattern
|
|
|
|
|
|
|
|
|
|
* gio/gfileattribute.c (g_file_attribute_info_list_get_type): Use
|
|
|
|
|
g_once_init_enter/leave.
|
|
|
|
|
|
2008-10-08 22:54:35 +02:00
|
|
|
|
2008-10-08 Christian Persch <chpe@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 555314 – mem leak in gmarkup
|
|
|
|
|
|
|
|
|
|
* glib/gmarkup.c: (g_markup_parse_context_parse): Plug a mem leak.
|
|
|
|
|
|
2008-10-08 22:35:39 +02:00
|
|
|
|
2008-10-08 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
Bug 554790 - g_convert() misbehaves with winiconv versions
|
|
|
|
|
|
|
|
|
|
* glib/win_iconv.c (kernel_mbtowc): If converting from ASCII,
|
|
|
|
|
explicitly check for and reject 8bit chars. MultiByteToWideChar()
|
|
|
|
|
doesn't, at least not on XP.
|
|
|
|
|
|
2008-10-07 20:52:18 +02:00
|
|
|
|
2008-10-06 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
* glib/gtypes.h: Properly include gmacros.h
|
|
|
|
|
|
2008-10-07 00:57:49 +02:00
|
|
|
|
2008-10-06 Behdad Esfahbod <behdad@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 555309 – giochannel breaks on error
|
|
|
|
|
Patch from Christian Persch
|
|
|
|
|
|
|
|
|
|
* glib/giounix.c (g_io_unix_read), (g_io_unix_write),
|
|
|
|
|
(g_io_unix_seek), (g_io_unix_close), (g_io_unix_set_flags):
|
|
|
|
|
Don't shadow err. Oops!
|
|
|
|
|
|
2008-10-06 20:50:39 +02:00
|
|
|
|
2008-10-06 Christophe Fergeau <teuf@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 555224 – Improve g_format_size_for_display doc
|
|
|
|
|
|
|
|
|
|
* glib/gfileutils.c: change g_format_size_for_display API doc to
|
|
|
|
|
explicitly say that the returned string has to be freed. Change
|
|
|
|
|
spelling of "newly allocated" to "newly-allocated" in g_file_read_link
|
|
|
|
|
API doc to be more consistent with what is done in that file.
|
|
|
|
|
|
2008-10-01 19:46:57 +02:00
|
|
|
|
2008-10-01 David Zeuthen <davidz@redhat.com>
|
|
|
|
|
|
|
|
|
|
* README.in: Add "Notes about glib 2.20" section detailing the
|
|
|
|
|
ramifications of the patch from bug #528670.
|
|
|
|
|
|
2008-09-30 22:40:31 +02:00
|
|
|
|
2008-09-30 Behdad Esfahbod <behdad@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 554092 – glib doesn't return G_FILE_ERROR_NOENT et al on OS X
|
|
|
|
|
|
|
|
|
|
* glib/giounix.c (g_io_unix_read), (g_io_unix_write),
|
|
|
|
|
(g_io_unix_seek), (g_io_unix_close), (g_io_unix_set_flags),
|
|
|
|
|
(g_io_unix_get_flags), (g_io_channel_new_file):
|
|
|
|
|
Like mclasen says: "well, thats the way errno works...,
|
|
|
|
|
save it or loose it". Save errno.
|
|
|
|
|
|
2008-09-30 16:46:45 +02:00
|
|
|
|
2008-09-30 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
* Makefile.decl
|
|
|
|
|
* glib/tests/Makefile.am: Bypass gtester related stuff on Windows.
|
|
|
|
|
|
2008-09-30 16:04:35 +02:00
|
|
|
|
2008-09-30 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
* glib/gprintf.c
|
|
|
|
|
* glib/gnulib/vasnprintf.c: Don't define _GNU_SOURCE on Windows,
|
|
|
|
|
as _GNU_SOURCE has unintended side effects when compiling against
|
|
|
|
|
newest mingw headers.
|
|
|
|
|
|
2008-09-27 03:43:29 +02:00
|
|
|
|
2008-09-26 Dan Winship <danw@gnome.org>
|
|
|
|
|
|
|
|
|
|
Bug 553447 – g_assert_no_error()
|
|
|
|
|
|
|
|
|
|
* glib/gtestutils.h (g_assert_no_error, g_assert_error): Macros to
|
|
|
|
|
assert that a GError is not set, or else is set to a particular
|
|
|
|
|
error.
|
|
|
|
|
|
|
|
|
|
* glib/gtestutils.c (g_assertion_message_error): utility for
|
|
|
|
|
those macros
|
|
|
|
|
|
|
|
|
|
* glib/tests/keyfile.c:
|
|
|
|
|
* tests/asyncqueue-test.c:
|
|
|
|
|
* tests/bookmarkfile-test.c:
|
|
|
|
|
* tests/convert-test.c:
|
|
|
|
|
* tests/file-test.c: Use g_assert_error/g_assert_no_error
|
|
|
|
|
|
2008-09-26 18:00:45 +02:00
|
|
|
|
2008-09-26 Dan Winship <danw@gnome.org>
|
|
|
|
|
|
|
|
|
|
* glib/gthreadpool.c (wakeup_thread_marker): make this a "const
|
|
|
|
|
gpointer" rather than a gconstpointer to avoid warnings later
|
|
|
|
|
|
|
|
|
|
* glib/pcre/pcre_ucp_searchfuncs.c:
|
|
|
|
|
* glib/pcre/pcre_valid_utf8.c: #include "config.h"
|
|
|
|
|
|
|
|
|
|
* glib/tests/printf.c (test_d): fool gcc into not warning about
|
|
|
|
|
some printf format strings that we know are dubious
|
|
|
|
|
|
2008-09-26 16:33:48 +02:00
|
|
|
|
2008-09-26 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 553857 – gbacktrace.h requires signal.h
|
|
|
|
|
|
|
|
|
|
* glib/gbacktrace.h: Include signal.h for raise().
|
|
|
|
|
Pointed out by Sebastien Bacher
|
|
|
|
|
|
2008-09-26 16:28:25 +02:00
|
|
|
|
2008-09-26 Matthias Clasen <mclasen@redhat.com>
|
|
|
|
|
|
|
|
|
|
Bug 553724 – python interpretter path not patched in correctly
|
|
|
|
|
|
|
|
|
|
* glib/Makefile.am: Fix the sed magic to replace python.
|
|
|
|
|
|
|
|
|
|
2008-09-26 Matthias Clasen <mclasen@redhat.com>
|
2008-09-26 15:55:51 +02:00
|
|
|
|
|
2008-09-26 16:05:03 +02:00
|
|
|
|
* glib/gmain.c: Add some more docs.
|
|
|
|
|
|
2008-09-26 15:55:51 +02:00
|
|
|
|
* glib/giochannel.c: Move more docs inline, and improve them
|
|
|
|
|
on the way.
|
|
|
|
|
|
2008-09-25 21:59:49 +02:00
|
|
|
|
2008-09-25 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
Bug 553820 - gpoll.c: undeclared identifier
|
|
|
|
|
|
|
|
|
|
* glib/gmain.c
|
|
|
|
|
* glib/gpoll.c: Make the g_poll() function non-static also on
|
|
|
|
|
Windows. Prefix an underscore to the g_main_poll_debug variable
|
|
|
|
|
and make it non-static in gmain.c so that it can be used in
|
|
|
|
|
gpoll.c. Add back missing variable declaration.
|
|
|
|
|
|
2008-09-25 10:05:41 +02:00
|
|
|
|
2008-09-25 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
* glib/gspawn-win32.c (do_spawn_with_pipes) (do_spawn_directly):
|
|
|
|
|
Just ignore the child_setup function, never call it. The is no
|
|
|
|
|
situation in which it could be useful on Windows. Do print a
|
|
|
|
|
warning, like before.
|
|
|
|
|
|
|
|
|
|
* glib/gspawn.c (g_spawn_async_with_pipes): Corresponding change
|
|
|
|
|
in documentation.
|
|
|
|
|
|
2008-09-24 15:44:27 +02:00
|
|
|
|
2008-09-24 Sven Herzberg <sven@imendio.com>
|
|
|
|
|
|
|
|
|
|
Be a little more explcit in the docs. Includes Owen's requested
|
|
|
|
|
changes.
|
|
|
|
|
|
|
|
|
|
* glib/gmain.c: improved documentation for g_source_attach() and
|
|
|
|
|
g_source_destroy()
|
|
|
|
|
|
2008-09-23 20:56:31 +02:00
|
|
|
|
2008-09-23 Michael Natterer <mitch@imendio.com>
|
|
|
|
|
|
|
|
|
|
* glib/glib.h: #include <glib/gpoll.h>
|
|
|
|
|
|
|
|
|
|
* glib/gpoll.h: #error out if gpoll.h is included directly.
|
|
|
|
|
|
|
|
|
|
* glib/gpoll.c: remove trailing whitespace.
|
|
|
|
|
|
2008-09-23 18:41:37 +02:00
|
|
|
|
2008-09-23 Dan Winship <danw@gnome.org>
|
|
|
|
|
|
2008-09-23 20:56:31 +02:00
|
|
|
|
* glib/gpoll.[ch] (g_poll): Move this out of gmain.c and make it part
|
2008-09-23 18:41:37 +02:00
|
|
|
|
of the public API. (Part of Bug 505361 - gunixinputstream.c assumes
|
|
|
|
|
poll() available.)
|
|
|
|
|
|
2008-09-23 17:35:12 +02:00
|
|
|
|
2008-09-23 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
|
|
|
|
* glib/gmain.c (poll_rest) [Win32]: Fix embarrassing bug: I was
|
|
|
|
|
passing an incorrect third parameter to memmove(), had forgotten
|
|
|
|
|
to multiply by the size of the table entry. Just use a for loop
|
|
|
|
|
instead, clearer. Odd I didn't notice when testing this code.
|
|
|
|
|
|
Rewrite most of GHashTable to use open addressing with quadratic probing
2008-09-19 Hans Petter Jansson <hpj@novell.com>
Rewrite most of GHashTable to use open addressing with quadratic
probing instead of chaining. This has the potential to reduce memory
fragmentation significantly, while being slightly faster due to
better locality and no need to call alloc/free functions for nodes.
Benchmarks suggest it also uses less memory overall.
* glib/ghash.c (prime_mod): Table of suitable primes for
initial-probe distribution.
(g_hash_table_set_shift): New function.
(g_hash_table_find_closest_shift): New function.
(g_hash_table_set_shift_from_size): New function.
(g_hash_table_lookup_node_for_insertion): New function.
(g_hash_table_lookup_node): Rewritten to return node index instead of
pointer, use quadratic probe on flat table, and not return insertion
data. The latter saves some computation for read-only lookups.
(g_hash_table_remove_node): Rewrite to take a pointer directly to the
node structure to remove, and clear that. Remove unlinking code.
(g_hash_table_remove_all_nodes): Rewrite to not clear nodes
individually, but en masse using memset () after potentially calling
notify functions.
(iter_remove_or_steal): Use new data structure and algorithm. Vastly
simplified - now just a call to g_hash_table_remove_node ().
(g_hash_table_resize): New resize code, re-indexing with new prime
and cleaning up tombstones.
(g_hash_table_maybe_resize): Table may hold 8 buckets minimum, no less
than 1/4 load excluding tombstones, and no more than 15/16 load
including tombstones. These numbers are the results of a lot of
benchmarking with multiple complex applications, and should not be
changed lightly.
(g_hash_table_iter_next)
(g_hash_table_lookup)
(g_hash_table_lookup_extended)
(g_hash_table_insert_internal)
(g_hash_table_remove_internal)
(g_hash_table_foreach_remove_or_steal)
(g_hash_table_foreach)
(g_hash_table_find)
(g_hash_table_get_keys)
(g_hash_table_get_values): Use new data structure and algorithm,
fairly trivial changes.
svn path=/trunk/; revision=7518
2008-09-20 06:05:11 +02:00
|
|
|
|
2008-09-19 Hans Petter Jansson <hpj@novell.com>
|
|
|
|
|
|
|
|
|
|
Rewrite most of GHashTable to use open addressing with quadratic
|
|
|
|
|
probing instead of chaining. This has the potential to reduce memory
|
|
|
|
|
fragmentation significantly, while being slightly faster due to
|
|
|
|
|
better locality and no need to call alloc/free functions for nodes.
|
|
|
|
|
Benchmarks suggest it also uses less memory overall.
|
|
|
|
|
|
|
|
|
|
* glib/ghash.c (prime_mod): Table of suitable primes for
|
|
|
|
|
initial-probe distribution.
|
|
|
|
|
(g_hash_table_set_shift): New function.
|
|
|
|
|
(g_hash_table_find_closest_shift): New function.
|
|
|
|
|
(g_hash_table_set_shift_from_size): New function.
|
|
|
|
|
(g_hash_table_lookup_node_for_insertion): New function.
|
|
|
|
|
(g_hash_table_lookup_node): Rewritten to return node index instead of
|
|
|
|
|
pointer, use quadratic probe on flat table, and not return insertion
|
|
|
|
|
data. The latter saves some computation for read-only lookups.
|
|
|
|
|
(g_hash_table_remove_node): Rewrite to take a pointer directly to the
|
|
|
|
|
node structure to remove, and clear that. Remove unlinking code.
|
|
|
|
|
(g_hash_table_remove_all_nodes): Rewrite to not clear nodes
|
|
|
|
|
individually, but en masse using memset () after potentially calling
|
|
|
|
|
notify functions.
|
|
|
|
|
(iter_remove_or_steal): Use new data structure and algorithm. Vastly
|
|
|
|
|
simplified - now just a call to g_hash_table_remove_node ().
|
|
|
|
|
(g_hash_table_resize): New resize code, re-indexing with new prime
|
|
|
|
|
and cleaning up tombstones.
|
|
|
|
|
(g_hash_table_maybe_resize): Table may hold 8 buckets minimum, no less
|
|
|
|
|
than 1/4 load excluding tombstones, and no more than 15/16 load
|
|
|
|
|
including tombstones. These numbers are the results of a lot of
|
|
|
|
|
benchmarking with multiple complex applications, and should not be
|
|
|
|
|
changed lightly.
|
|
|
|
|
(g_hash_table_iter_next)
|
|
|
|
|
(g_hash_table_lookup)
|
|
|
|
|
(g_hash_table_lookup_extended)
|
|
|
|
|
(g_hash_table_insert_internal)
|
|
|
|
|
(g_hash_table_remove_internal)
|
|
|
|
|
(g_hash_table_foreach_remove_or_steal)
|
|
|
|
|
(g_hash_table_foreach)
|
|
|
|
|
(g_hash_table_find)
|
|
|
|
|
(g_hash_table_get_keys)
|
|
|
|
|
(g_hash_table_get_values): Use new data structure and algorithm,
|
|
|
|
|
fairly trivial changes.
|
|
|
|
|
|
2008-09-19 12:20:41 +02:00
|
|
|
|
2008-09-19 Tor Lillqvist <tml@novell.com>
|
|
|
|
|
|
2008-09-19 12:42:04 +02:00
|
|
|
|
* glib-zip.in: Look for man pages in share/man.
|
|
|
|
|
|
2008-09-19 12:20:41 +02:00
|
|
|
|
* glib/gutils.c (_glib_get_dll_directory)
|
|
|
|
|
* glib/gspawn-win32.c (do_spawn_with_pipes): Be a bit less
|
|
|
|
|
restrictive, look for the helper programs in the same folder where
|
|
|
|
|
the GLib DLL is, not necessarily in a "bin" subfolder of the top
|
|
|
|
|
GLib installation folder.
|
|
|
|
|
|
2008-09-18 16:47:14 +02:00
|
|
|
|
2008-09-18 Matthias Clasen <mclasen@redhat.com>
|
2008-09-18 00:52:48 +02:00
|
|
|
|
|
2008-09-18 16:47:14 +02:00
|
|
|
|
* configure.in: Bump version to 2.19.0
|
2008-09-18 01:58:48 +02:00
|
|
|
|
|
2008-09-18 16:47:14 +02:00
|
|
|
|
* ChangeLog.pre-2-18: rotate ChangeLog
|
2007-11-08 03:54:39 +01:00
|
|
|
|
|
2008-09-18 16:47:14 +02:00
|
|
|
|
* === branch for 2.18 ===
|