svn commit: r184780 - head/usr.sbin/cron/crontab

Alexey Dokuchaev danfe at FreeBSD.org
Sun Nov 9 00:48:17 PST 2008


On Sun, Nov 09, 2008 at 07:34:11AM +0000, Matteo Riondato wrote:
>  
> +void
> +static remove_tmp(int sig)
> +{
> +	if (tmp_path) {
> +		unlink(tmp_path);
> +	}
> +	exit(ERROR_EXIT);
> +}

This looks weird: `static' should be on same line as `void' as `static
void' (so ^remove_tmp would match).  It will also always exit with
ERROR_EXIT, which does not look right, does it?

./danfe


More information about the svn-src-head mailing list