On Fri, Nov 09, 2001 at 07:45:55PM -0500, Glenn Maynard wrote:
> Easy one: "550 Not a directory." spams the output.  Any reason not to move
> 550 for CWD to 4?

Ok, no problem.

> Second, chdir results aren't cached, so it keeps doing the test chdirs
> every time.  Of course, most chdir results shouldn't be cached; it's only
> useful for this case.  Do you think we could use a more generalized cache?
> Perhaps a simple CacheObject base class, containing a type and a virtual
> destructor; allow searching for cached objects by name (path or whatever
> is appropriate) and type.  (It'd mean using MI, but I don't think that's
> a problem.)
>
> Third, "glob -a cls *" is slow for large directories: cls has to
> re-parse the directory for each file.  This is another incentive to have
> generalized caching: we could cache FileSets, too.  We could also cache
> FileSets inside LsCache explicitely, alongside ls text, but that's a hack.

Do it carefully not to use wrong cached object, e.g. you would have
to store all options of ListInfo along with the resulting FileSet.

> Not going to bother you with a patch until I work out these issues, since
> it's not useful yet.  (I've attached the class source, if you want to look
> at it; don't drop it in yet, though.)

The message was without an attachement.

-- 
   Alexander.

Reply via email to