Retain orig. filename as suffix for tmp. filename

Adam Byrtek / alpha alpha at student.uci.agh.edu.pl
Mon Feb 24 10:27:53 UTC 2003


On Mon, Feb 24, 2003 at 02:26:13AM +0100, Oswald Buddenhagen wrote:
> > +    if (basename && *basename==PATH_SEP)
> > +	basename++;
> this is totally bogus.

No, this is not 'totally bogus'. Filename var always contains at least
one PATH_SEP:

cdbackup-0.6.3.tar.gz#utar/cdbackup-0.6.3/CHANGES
/#ftp:portal at aaaa.pl/backup

So this 'if' is just to protect from the impossible. As all
programmers know impossible happens about once a year to (and I want
to be protected just in case some code changes etc.), and we surely
dont want to increment NULL of increment beyond the string. Result is
not so important in impossible case :)

Of course your code would be also good, with one exception:

  basename = filename;

because filename includes all the vfs stuff, which we don't want IMO.

Regards

-- 

  _.|._ |_  _.   :  Adam Byrtek /alpha/
 (_|||_)| |(_|   :  email  alpha@(irc.pl|debian.org)
     |           :  jabber alpha.pl(at)jabber.org, pgp 0xB25952C0



More information about the mc-devel mailing list