Re: [patch] devd reporting funny line numbers on parsing failure

From: Maxim Konovalov <maxim_at_macomnet.ru>
Date: Fri, 14 Apr 2006 11:21:28 +0400 (MSD)
Hi Niki,

> This fixes devd reporting funny line numbers when
> there are more than one config files to be parsed.
> Basically "lineno" is being incremented on every new line in
> every config file, so if you have unparsable line in a
> config file then devd will report that the unparsable line
> number is the number of the total parsed lines from all config files
> so far and not the line number from the beginning of the file being
> parsed.
>
>
> - --- devd.cc.    Thu Apr 13 22:02:14 2006
> +++ devd.cc.fixed.      Thu Apr 13 22:01:29 2006
> _at__at_ -311,6 +311,7 _at__at_
> 	yyin = fopen(fn, "r");
> 	if (yyin == NULL)
> 		err(1, "Cannot open config file %s", fn);
> +	lineno = 1;
> 	if (yyparse() != 0)
> 		errx(1, "Cannot parse %s at line %d", fn, lineno);
> 	fclose(yyin);

Fixed in HEAD.  Thanks!

-- 
Maxim Konovalov
Received on Fri Apr 14 2006 - 05:21:32 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:38:54 UTC