Hi Tom,

Thanks much for the feedback!


> Just "skip_whitespace" sounds more natural to me.
> I think "whitespace" is commonly used for both singular and plural.

Fixed.

> Joel> +skip_until_character (char *str, char delimiter)
> Joel> +skip_until_whitespace (char *str)
> Joel> +skip_space_delimited_token (char *str)
> Joel> +get_space_delimited_token (char *str, char **token)
> 
> I think all of these (including the others above) could be in util.c.

Agreed - moved thusly.

> Joel> +void *load_shared_lib (char *lib_name)
> 
> Formatting.
> 
> Also, this should mention how errors are to be detected.

All fixed.  These two routines are the two left in remote-wtx-utils
after the exercise. I was thinking of moving them to a non-wtx context,
but it turns out I forgot that one of them (load_shared_lib) is more
than just a dlopen wrapper. What I had forgotten is that function takes
a library name (eg: "wtx40", and build the DSO filename depending on
the host filesystem -> either libwtx40.so or wtx40.dll). It would have
been too easy if it was just a matter of using a .so vs .dll extension...
So it really belongs in a vxworks-specific unit.

I thought it might have been useful to have a gdb_dlfcn.h module that
abstracts out shared library support. It's relatively easy to write,
the code itself, but it's the packaging that raises questions. See the
revision history that I wrote for patch 0003.2 (attached).  In any case,
I made it easier for this to happen, when/if we want to.

Here are 2 new patches, replacing patch 0003...

-- 
Joel