From: Daniel Berlin <dan@cgsoftware.com>
To: Andrew Cagney <ac131313@cygnus.com>
Cc: Daniel Berlin <dan@cgsoftware.com>, gdb-patches@sources.redhat.com
Subject: Re: [RFA]: Symbol evaluation code
Date: Thu, 14 Jun 2001 13:15:00 -0000 [thread overview]
Message-ID: <87elsmhl7s.fsf@cgsoftware.com> (raw)
In-Reply-To: <3B277F6C.7040108@cygnus.com>
Andrew Cagney <ac131313@cygnus.com> writes:
>> I plan on making the push_opcode non-varargs before making anything
>> use it, but i really want to get this stuff out of my tree and into
>> sourceware, before I start having fun with cvs conflicts. Also, one
>> of my HD's is starting to go, and I didn't want to lose it.
>
>
> (Jim B actually suggested this)
>
> Just FYI, if people are creating a lot of expermental code then it
> might be a good idea to keep them on a CVS branch rather than in a
> local sandpit. At least that way it is being backed up and it is
> available for others to view.
Well, I plan on doing this for the type work soon.
Where soon is later today. I want to submit the dwarf2 rewrite so far
for comments, first.
However, the symbol evaluation code is, as I said somewhere,
completely self contained. Nothing uses it, it has no dependencies on
anything except itself, etc.
So what's the rule for things like that?
It doesn't seem to make sense to create a branch for it, it's not
affecting any other part of gdb, so you end up having to just do
merges for no good reason.
Moving to using it can be done completely incrementally, without any
hacks or disabling of anything.
It would seem, if we all agree this is the general direction to go in,
that things like that should just be added to the main branch.
I ask because, I actually made push_opcodes non-varargs. There are 3 push_operand
routines now, one for each type. They each take a cookie push_opcodes
returns, so we can do the same format checking and type checking
push_opcodes did.
So it's ready to be submitted, unless you want me to put it on a branch.
>
>
> I should note though, that there is a big difference between using a
> branch to run a few experements and using a branch to create jumbo
> patches. To give an example, I'm tinkering the MI code. Consequently
> I've the following working directories.
>
>
> MI/src
> I'm using this to figure out
> what needs changing and how
>
> GDB/* A reference directory
>
> WIP/* one of my merge directories
>
> I'm pulling changes out of the MI directory and them merging them into
> WIP before posting them. They then get back merged in to MI. Over
> time MI and WIP/GDB converge. I would never submit a patch taken
> directly from my experemental MI directory since it just isn't up to
> scratch.
>
> Andrew
>
>
>
--
"I like to skate on the other side of the ice.
"-Steven Wright
prev parent reply other threads:[~2001-06-14 13:15 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2001-05-29 10:38 Daniel Berlin
2001-06-13 7:57 ` Andrew Cagney
2001-06-14 13:15 ` Daniel Berlin [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87elsmhl7s.fsf@cgsoftware.com \
--to=dan@cgsoftware.com \
--cc=ac131313@cygnus.com \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox