This is the mail archive of the
mailing list for the GDB project.
Re: [lttng-dev] Request change name of function lookup_enum in libbabeltrace to make GDB use this lib
- From: Pedro Alves <palves at redhat dot com>
- To: Mathieu Desnoyers <mathieu dot desnoyers at efficios dot com>
- Cc: Hui Zhu <teawater at gmail dot com>, Julien Desfossez <jdesfossez at efficios dot com>, lttng-dev <lttng-dev at lists dot lttng dot org>, Tom Tromey <tromey at redhat dot com>, gdb at sourceware dot org
- Date: Thu, 06 Dec 2012 15:57:30 +0000
- Subject: Re: [lttng-dev] Request change name of function lookup_enum in libbabeltrace to make GDB use this lib
- References: <CANFwon2s-eJoCrhzJmxhoGx9G2NeOAFf-FDFk1SoxD1OhPxWsg@mail.gmail.com> <20121205120845.GA31696@Krystal>
On 12/05/2012 12:08 PM, Mathieu Desnoyers wrote:
> * Hui Zhu (email@example.com) wrote:
>> I am working on add CTF support to GDB. You can see my patch review threads in:
>> To make GDB support CTF read, I use libbabeltrace with GDB. You can
>> see the patch in
>> I have a issue is libbabeltrace have a function called lookup_enum
>> that is same with a GDB function.
>> I change the function name of GDB to handle this issue in my patch.
>> But Tom said let libbabeltrace to change function name is better.
>> So I send this mail to ask do you mind change the function name of
>> lookup_enum? If you can change the function name that will be really
>> helpful for us. Thanks a lot.
>> And I post a patch about change the function name in libbabeltrace.
> I'm CCing Julien Desfossez on this one. From what I see,
> include/babeltrace/types.h is not included into the system, so it should
> not be considered to be a public header of libbabeltrace.
I've just built and installed babeltrace 1.0.0 (where's the mainline repository,
BTW?), and indeed, I'm not seeing the types.h file anywhere in the
$ ~/src/babeltrace/install/include> find
The GDB patch is including types.h explicitly:
So indeed, Hui, you'll need to make sure your patch works against an
installed babeltrace, making sure it does not pick up headers
from babeltrace's source directory. If there's really no reason to
include that types.h header (since it seems you don't really need any
function declared in that file), maybe there's actually nothing for
babeltrace to do.
> Julien, is there an publically exposed babeltrace API that performs
> something similar to the internal lookup_enum() ?
> Hui, are you using other functions from include/babeltrace/types.h ?