On Mon, Nov 04, 2013 at 04:09:29AM -0800, Daniel Murphy wrote:
> > @@ -593,6 +593,15 @@ const char *FileName::replaceName(const char *path, const char *name)
> >   * Free returned value with FileName::free()
> >   */
> >  
> > +const char *FileName::dup(const char *name)
> 
> Hmm, I forgot about that.  I have no problem with removing
> `FileName::free`, after using D it seems a little silly to have more
> than one allocation method for strings.  We're probably safe either
> way.

I'm fine either way (leaving FileName::dup() or removing
FileName::free()), but if we are going to remove FileName::free(), it
might as well be done in a separate pull request, as it has nothing to
do with this issue. So I'll wait for instructions before doing any
change to the pull request.