redundant code in ftpfs?

Pavel Tsekov ptsekov at gmx.net
Mon Mar 6 15:34:36 UTC 2006


On Sun, 5 Feb 2006, Arpi wrote:

>         p = ftpfs_translate_path (me, super, remote_path);
> 	if (ftpfs_chdir_internal (me, super, p) != COMPLETE) {
> 		...
> 	}
> 	g_free (p);
>
> the ftpfs_translate_path() call and the 'p' pointer are redundant here,
> as ftpfs_translate_path is also called from inside ftpfs_chdir_internal()
> on the p parameter. so now it's done twice, which is redundant, and
> may be a problem too...

I have removed the extra call to ftpfs_translate_path() and the related
code from the CVS version of ftpfs.c.

Thanks!



More information about the mc-devel mailing list