On Wed, Feb 08, 2006 at 08:48:15AM -0800, Paddy Sreenivasan wrote:

> > But I am not sure whether I correctly understand the proposal.  Is the
> > following the proposed fix?
> 
> Yes. Please add test results to the bug tracker.

I have arranged for the test tonight.  I'll report tomorrow.

Thanks!

BTW: Checkin 1.111->1.112 of server-src/taper.c looks a little bit strange
     to me.  Especially two hunks I've appended to the end of this mail.
     When ``str ? str : "(null)"´´ results in "(null)" the str was the NULL
     pointer.  Will the called function newvstralloc() be able to cope with
     the passed NULL?


@@ -1428,7 +1428,7 @@
                                      "[sec ", walltime_str(runtime),
                                      " kb ", kb_str,
                                      " kps ", kps_str,
-                                     " ", str ? str : "(null)",
+                                     " ", str,
                                      "]",
                                      NULL);
@@ -1469,7 +1467,7 @@
                                              "[sec ", walltime_str(curdump_rt),
                                              " kb ", kb_str,
                                              " kps ", kps_str,
-                                             " ", str ? str : "(null)",
+                                             " ", str,
                                              "]",
                                              NULL);
                        putresult(DONE, "%s %s %d %s\n", handle, label,