Thanks for review Nicholas.
1) Oops, will fix long lines
2) I though of making exactly this change (the first version of the patch
contains this very change). However, it might be troublesome in some cases,
because fileSys.rename can return false in some cases (say, namespace won't
rename the path) and then we might stuck in this loop forever
That's why I've left it as is and put this comment in both (rename and delete)
cases. I'll appreciate to hear from you on this before I'll submit the update to
3) nice catch! Fixed along with a couple of other excessive castings to double.
Thanks in advance,