This is the mail archive of the
systemtap@sourceware.org
mailing list for the systemtap project.
Re: [PATCH] Add function name and file to semantic error messages for not found local variables.
> In this case we cannot always just print the diename of the scope we are
> looking at since that might be NULL (see just above in the function). In
> that case we are just looking for the scope by address. And while we are
> improving the error message, lets also add the same for the other error
> case just above it. So I think we want something like the attached. Does
> that work for you?
Yes, I didn't noticed NULL problem and unconsciously omitted second
error message in this function, so your patch is definitely better
(and works, of course).
Here it was a rather silly mistake (because of the precedent code, as
you mentioned), but yet I must say that dataflow in systemtap is
(currently) somewhat cryptic to me (I didn't spend enough time looking
into it and sorry for this). I mean e.g. it's not obvious (to me)
where scope_die in tapsets.cxx is not NULL for sure. However, that is
my problem, no systemtap's.
Regards.
--
Przemysław Pawełczyk