This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] Remove calls to inside_entry_file


Andrew,

On Fri, Mar 28, 2003 at 07:28:36PM -0500, Andrew Cagney wrote:

>Index: blockframe.c
>===================================================================
>RCS file: /cvs/src/src/gdb/blockframe.c,v


For "blockframe.c", please leave it as is. I'm already in enough trouble for breaking old targets so I'd prefer to leave that part untouched. It would only affect out-of-date targets anyway. The up-to-date targets don't rely on that function.


I've checked in the frame.c patch but still, I don't understand this
decision.  So called out-of-date targets can easily add the
inside_entry_file() call to their frame_chain_valid() implementation
so removing this call from blockframe.c does not necessarily break
them.  Keeping this call in blockframe.c on the other hand breaks
some targets for which this call is plainly wrong.  So the logic would
imply to remove the call in favour of *all* targets able to run correctly.

I've checked this patch (including the patch to i386_frame_chain_valid)
on four targets, xstormy16-elf, i686-pc-cygwin, i686-pc-linux and arm-elf.
The first two are running fine then, the latter two are totally
unaffected.

You want to run arm and i386 changes past their respective maintainers.


Andrew

PS: Patch?



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]