This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [PATCH] Multiple locations for tracepoints
- From: Joel Brobecker <brobecker at adacore dot com>
- To: Stan Shebs <stan at codesourcery dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Thu, 4 Mar 2010 09:27:37 +0400
- Subject: Re: [PATCH] Multiple locations for tracepoints
- References: <4B8F020D.9020506@codesourcery.com>
> Committed to the trunk. I'm not sure if this should go to 7.1 also;
> it has no effect outside tracepoint-land, but don't want to churn
> the sources if it's close to release.
I think we still have a couple of weeks before we decide to release.
I haven't started calling for known problems that people might think
is blocking for the release. I'll probably do that early next week.
For your patch, my initial reaction was that it was best to leave it
out of 7.1. But I'm not 100% sure because, not knowing tracepoints
all that well, I'm not sure whether you're fixing a bug or just
enhancing support for multi-location tracepoints. My guideline for
merging a change to the branch is: Is this a bad bug or regression?;
and is the fix relatively safe? Your patch is not all that small or
obvious, so unless it's a crippling issue, I'd suggest we leave it out
(everyone is allowed to disagree ;-), as always).
--
Joel