2005-07-29 Havoc Pennington <hp@redhat.com>

	* test/glib/test-profile.c (write_junk): initialize the junk
	buffer so valgrind doesn't have a breakdown

diff --git a/ChangeLog b/ChangeLog
index 6832d4e6ed944d0a3718917cc18fcecf33c3463b..2a53e5aa44addbeacaadcb3f04bc20a57ef881cd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2005-07-29  Havoc Pennington  <hp@redhat.com>
+
+       * test/glib/test-profile.c (write_junk): initialize the junk
+       buffer so valgrind doesn't have a breakdown
+
 2005-07-29  Havoc Pennington  <hp@redhat.com>
 
        * bus/signals.c (bus_signals_test): add match_rule_equal() tests

diff --git a/doc/TODO b/doc/TODO
index 0d68910a20504d5c951029001693a549a3811b3a..55ba922290f9f3ce8094362cd291449286eb57c4 100644 (file)
--- a/doc/TODO
+++ b/doc/TODO
@@ -28,6 +28,13 @@ Important for 1.0 GLib Bindings
 
  - Add support for getting sender
 
+ - format_version in the object info doesn't look like it's handled correctly. The creator
+   of the object info should specify some fixed number per struct version; the library
+   should handle only specific numbers it knows about. There's no assumption that all 
+   numbers >= the given one are compatible. The idea is that new versions of the lib
+   can offer totally different object info structs, but old versions
+   keep working.
+
 Important for 1.0 Python bindings
 ===
 

diff --git a/test/glib/test-profile.c b/test/glib/test-profile.c
index ea42bd0821a7beb491551be6e81c45f5e2cc1f6e..21d5885284ddfbcb907f517ff87746aa0a9e1d8c 100644 (file)
--- a/test/glib/test-profile.c
+++ b/test/glib/test-profile.c
@@ -48,7 +48,7 @@
  */
 #define N_CLIENT_THREADS 1
 /* It seems like at least 750000 or so iterations reduces the variability to sane levels */
-#define N_ITERATIONS 750000
+#define N_ITERATIONS 25000
 #define N_PROGRESS_UPDATES 20
 /* Don't make PAYLOAD_SIZE too huge because it gets used as a static buffer size */
 #define PAYLOAD_SIZE 0
@@ -654,7 +654,7 @@ write_junk (int fd,
   int val;
   char *buf;
   char *allocated;
-  char not_allocated[512+PAYLOAD_SIZE];
+  char not_allocated[512+PAYLOAD_SIZE] = { '\0', };
 
   g_assert (count < (int) sizeof(not_allocated));
   
@@ -978,6 +978,7 @@ plain_sockets_thread_func (void *data)
 
   cd.iterations = 1;
   cd.loop = g_main_loop_new (context, FALSE);
+  cd.vtable = data;
 
   channel = g_io_channel_unix_new (fd);