On Tue, Oct 12, 2004 at 12:47:42PM -0400, Andy Dougherty wrote: > On Tue, 12 Oct 2004, Andy Dougherty wrote: > > > On Fri, 8 Oct 2004, Leopold Toetsch via RT wrote: > > > > > Andy Dougherty <[EMAIL PROTECTED]> wrote: > > > > > > > The biggest problem with the message is that it never tells you exactly > > > > *which* file was not found. > > > > > > Is the error message now more informative? > > Without the patch below, I just get a core dump. With the patch below, > yes, it's more informative.
Indeed :-) > --- parrot-current/src/dynext.c Fri Oct 8 13:41:12 2004 > +++ parrot-andy/src/dynext.c Tue Oct 12 12:42:53 2004 > @@ -217,7 +217,7 @@ > } > err = Parrot_dlerror(); > fprintf(stderr, "Couldn't load '%s': %s\n", > - full_name, err ? err : "unknown reason"); > + file_name, err ? err : "unknown reason"); Would it be better as full_name ? full_name : file_name ? Assuming that full_name when non-NULL has even more information. Nicholas Clark
