d/freebsd-nq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed off-by-1 errors in option line numbers. yyline is 0-based, but

Browse Source 
was used as if it is 1-based.  This happened to give the correct result
for options without values because of a compensating error in newline
lexing.  Didn't fix the latter, so line numbers in yyerror() may still
be 1 too high in some cases.
This commit is contained in:
Bruce Evans 1998-07-12 02:18:41 +00:00
parent 95de6ecfe8
commit f59d2e0292
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
usr.sbin/config/config.y
View File

@ -414,6 +414,10 @@ Option:
op->op_name = $1;
op->op_next = opt;
op->op_value = 0;
/*
* op->op_line is 1-based; yyline is 0-based but is now 1
* larger than when `Save_id' was lexed.
*/
op->op_line = yyline;
opt = op;
if ((s = strchr(op->op_name, '='))) {
@ -429,7 +433,7 @@ Option:
op->op_name = $1;
op->op_next = opt;
op->op_value = $3;
op->op_line = yyline;
op->op_line = yyline + 1;
opt = op;
} ;
@ -464,7 +468,7 @@ Mkoption:
op->op_ownfile = 0; /* for now */
op->op_next = mkopt;
op->op_value = $3;
op->op_line = yyline;
op->op_line = yyline + 1;
mkopt = op;
} ;