[Revisionmanager-devel] Re: RevisionManager and ZopeTree on sourceforge now

[Philipp v. W. <ph...@we...>]

Arthur,

(I'm CC-ing the developing list)

> btw, I have a quick question about RM.py
> *****************************************
> In  addChildPaths(self, paths):
> I changed the following:
> 
>             for module_path in module_paths:
>                 if module_path.startswith(path+'/') and module_path not in all_paths:
>                     sub_paths.append( module_path)
>                     
>             #sub_paths = [module_path for module_path in module_paths
>             #             if module_path.startswith(path+'/')]

If your code fixes a problem, it should go in. However, I recommend 
keeping the list comprehension. List comprehensions on large lists are 
much faster than for loops.

Philipp