This is the mail archive of the gdb-patches@sourceware.org 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: [PATCH v2 1/2] compile: set debug compile: Display GCC driver filename


On Mon, 27 Apr 2015 19:19:14 +0200, Pedro Alves wrote:
> Both.  I was curious on the reason why this was added as a flag
> to a method,

It looks to me as a most simple and working change to the API.


> and on your thoughts,

I do not have any thoughts about GDB, I make minimal changes.
Even these design changes are still wrong because of the incorrect principle
of search for the GCC driver.  In LLDB this already works for years better,
properly designed and with compatible licensing.


> I think it should indeed be made a separate method.

OK, I will post a new API for approval so that I do not have to rework the
2 patch series for the 3rd times as this reviewing method is very expensive.


Thanks,
Jan


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