Re: Recent change to file-chase-links has a bug

[kai.grossjohann@gmx.net (Kai Großjohann)]

Kenichi Handa <handa@m17n.org> writes:

> Now the code is as below:
>
> (defun file-chase-links (filename &optional limit)
>   "Chase links in FILENAME until a name that is not a link.
> Unlike `file-truename', this does not check whether a parent
> directory name is a symbolic link.
> If the optional argument LIMIT is a number,
> it means chase no more than that many links and then stop."
>   (let (tem (newname filename)
> 	    (count 0)
> 	    (max (max limit 100)))

Wild guess: Richard meant (or limit 100) instead of (max limit 100).

>     (while (and (or (null limit) (< count limit))
> 		(setq tem (file-symlink-p newname)))
>       (save-match-data
> 	(if (= count max)
> 	    (error "Apparent cycle of symbolic links for %s" filename))

-- 
A preposition is not a good thing to end a sentence with.