On Fri, 9 Nov 2007, Alexandru Stanoi wrote:

> Which is the best solution? (You've guessed, voting time again!)
> 
> 1) Keep the copyRecursive method as it is.
> 
> 2) Split it into copyRecursive and chmodRecursive (is there a BC break 
> here as we need to remove the last 2 arguments of copyRecursive?).

There is a major BC break, as suddenly copyRecursive() doesn't do what 
it did before. So we should stick to 1.

Adding chmodRecursive might be a good idea, and in that case we should 
probably refactor the code a bit. Feel free to add a feature request for 
this.

regards,
Derick
-- 
Components mailing list
[email protected]
http://lists.ez.no/mailman/listinfo/components

Reply via email to