This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: RFA: Some gengtype clean-ups
- From: kaih at khms dot westfalen dot de (Kai Henningsen)
- To: gcc-patches at gcc dot gnu dot org
- Date: 04 Feb 2006 10:59:00 +0200
- Subject: Re: RFA: Some gengtype clean-ups
- References: <87vewehbcw.fsf@talisman.home> <9mhV0YCXw-B@khms.westfalen.de> <9mhV0YCXw-B@khms.westfalen.de> <873bj4dbj2.fsf@talisman.home>
richard@codesourcery.com (Richard Sandiford) wrote on 31.01.06 in <873bj4dbj2.fsf@talisman.home>:
> kaih@khms.westfalen.de (Kai Henningsen) writes:
> > richard@codesourcery.com (Richard Sandiford) wrote on 20.01.06 in
> > <87vewehbcw.fsf@talisman.home>:
> >
> >> +/* Create a fake field with the given type and name. NEXT is the next
> >> + field in the chain. */
> >> +
> >> +static pair_p
> >> +create_field (pair_p next, type_p type, const char *name)
> >> +{
> >> + pair_p field;
> >> +
> >> + field = XNEW (struct pair);
> >> + field->next = next;
> >> + field->type = type;
> >> + field->name = name;
> >> + field->opt = NULL;
> >> + field->line.file = __FILE__;
> >> + field->line.line = __LINE__;
> >> + return field;
> >> +}
> >
> > Shouldn't this use the *caller's* __FILE__/__LINE__?
>
> Why?
Because that's what it was before?
MfG Kai