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/threads] Eliminate lin-lwp.c


Daniel Jacobowitz wrote:
On Thu, Sep 16, 2004 at 05:11:27PM -0400, Andrew Cagney wrote:

I do not think that leaving these functions where they are will
complicate the process of cleaning them up.  Could you explain
why you see this as a problem?

As with inf-child and inf-ptrace, an important objective here is to get all the inf-linux methods local to a file so that we can ensure that all accesses are strictly via the target vector. Leaving them where they are defeats this, a little.


So the reason you want them all in one file is so that you can make
them all static, and ensure they aren't accessed "on the side".  Is
that right?  That's a good technical reason for this patch, one which I
didn't think of and you didn't explain.  I'm just trying to get you to
share your wisdom with all us other maintainers.

With that explanation presented, I have no objection.  Let's make sure
Michael agrees.

I'm fine with it.




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