[pLog-svn] r6452 - in plog/trunk/class: . object

Mark Wu markplace at gmail.com
Fri May 9 14:01:02 EDT 2008


 
>  	Ok, I was wondering about that.  I think the script 
> should be changed to ignore the test directories.  I don't 
> care as much about the map file as the directory search 
> function - I think we should order that search process in a 
> better way than alphabetically.  Unless that function really 
> never gets called, since everything is in the loadermap.  
> Probably plugins still use it?

The loader map should re-generated before we release it, it won't include
test files in official release.
 
> > 2. Oscar remove "all" lt_include(), so, if you do this, 
> some test can 
> > not run well..
>  	If I am reading the code correctly, it looks through 
> the map, and if it can't be found, then looks through all of 
> the directories.  So, the tests should still work correctly.

Well, yes .... The side effect is "bad performance" coming back again.

I don't like this. I prefer use "a small memory usage" to get  "better
performance".

> 
> > 3. Please kindly make sure all scripts can run well before 
> you commit.
>  	I doubt the tests run at all at the moment, so I didn't 
> bother testing them.

As I said, this file should re-generate again before it released.  Removing
it from this file does not make any sense. Especially, we are in development
stage now.



More information about the pLog-svn mailing list