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] convert blocks to dictionaries, phase 1, main part


Andrew Cagney <ac131313@ges.redhat.com> writes:
> > That's not the point.  That's why I suggested a branch which does
> > require approval, precisely so that we wouldn't get into that problem.
> > But you don't seem to like that idea, so it's dead.
> 
> Me not liking an idea doesn't kill it.  It is the symtab maintainers,
> and not me that would do the review and hence, they and not me would
> need to be ok with it.

I don't know how Elena feels, but I generally defer to Andrew's
judgement in these sorts of things.

I'd suggest the following process:

- Get things working on a branch.  That is, produce a branch on which
  namespaces actually work --- the block -> dictionary conversion
  isn't enough.  This branch will never be merged directly; it's dead
  code.

- You'll almost certainly find some small messes which can be cleaned
  up indepenently of any other work.  These can be submitted for
  inclusion in the main sources as you go.

- You'll also probably learn some things that you'll wish you had
  known when you started.  But the design backtracking won't happen in
  the public sources --- it'll happen on your dead branch.

  Backtracking is hard enough in pristine code.  If you're
  simultaneously trying to break the change into incremental steps,
  I imagine the backtracking will be even more difficult.

- Once you've got it working, the way it really should, then you can
  decide how to submit it.  At that point, you'll have better ideas
  about how to approach this.  If some of the patches are large, so be
  it.

The downside about this process is that it looks like you do the work
twice: on the branch, and then again in the public sources.  But I
think that is partly an illusion.  What actually takes time in these
situations isn't the typing: it's the research (figuring out how the
damned thing works), and the backtracking (oh, this can't support
`using namespace' directives, we've got to do it another way).  The
"second time", when you're working from tested, running code, goes
much faster.


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