[MC BUG, PATCH] moved files are accounted twice
Wed Jul 23 06:20:37 UTC 2003
On Tue, Jul 22, 2003 at 11:38:58PM -0400, Pavel Roskin wrote:
> On Wed, 9 Jul 2003, it was written:
>
> > When moving >1 files every file is accounted twice in "count" progress bar.
> > Each file is copied with copy_file_file() (which does accounting) and
> > then removed.
> > Following patch fixes this (don't know if this has any side effects).
> > Consider applying. Patch is against current CVS.
>
> Test 1. Moving one large directory. There is no "count" bar at all.
> Obviously a bug.
>
> Test 2. Moving three large directories. The "count" bar stays at 0.
> Obviously another bug.
>
> Test 3. Moving three large files. Yes, that's where your patch helps.
>
> Test 4. Moving 9 symlinks and 3 large files. The count ends at 25%.
> Obviously one more bug.
>
> Test 5. Copying 9 symlinks and 3 large files. The count ends at 25%.
> The last bug is in the copy routine.
>
> Your patch eliminates counting symlinks, and the copy routine doesn't
> count them either. Not good. I believe that your patch does something
> wrong when symlinks are involved, but I cannot say it for sure before
> copying is fixed.
>
> With so many bugs around, I think it's better to fix the most basic of
> them rather than compensate for bugs and introduce new ones. I'll try to
> clean up what I can.
So ignore my patch and treat the original posting as bug report only.
I don't know the mc internals that well to cook some other patch.
> Regards,
> Pavel Roskin
Thanks,
Vita Samel
More information about the mc-devel
mailing list