public inbox for frysk@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Andrew Cagney <cagney@redhat.com>
Cc: frysk <frysk@sourceware.org>
Subject: Re: frysk.proc.{ptrace,corefile} -> frysk.proc.{live,dead}
Date: Mon, 18 Jun 2007 19:06:00 -0000	[thread overview]
Message-ID: <1182192973.4534.90.camel@dijkstra.wildebeest.org> (raw)
In-Reply-To: <4676C8D3.90107@redhat.com>

On Mon, 2007-06-18 at 14:02 -0400, Andrew Cagney wrote:
> Reading between the lines I suspect your concern here isn't so much 
> about this specific change but about the rate of change in general?

No, more the rate of changes I don't really understand the why of. I
like to have more design discussion on the list so we all know why and
what changes are proposed and being worked on.

> Remember Frysk as a project is relatively free flowing when it comes to 
> its code and refactoring.  We don't spend days or weeks musing over 
> choices, or trying to design perfection.  Instead we design for todays 
> problem and let tomorrows developers re-design for their future.  This 
> gives us much flexibility while while ensuring that we are focused on 
> real rather than hypothetical needs.

That is good. But what I am missing is the actual problem statement.
What exactly isn't possible today and what is possible tomorrow after
such a change is made. My concern is that some of this is just
refactoring for refactorings sake. That is why I ask the why and what
questions.

> Here I implemented frysk.proc.ptrace, and contrary to what you assert, 
> it is a mess.  It forced the exposure of numerous public variables in 
> frysk.proc.Task.

I don't believe I assert that at all. The split of the proc interface
and the proc.ptrace and proc.core implementation refactoring is not
complete though if it exposes things in the interfaces that should be in
the implementation packages.

>   By instead re-aligning the sub-packages to focus on 
> "liveness" and refactor all "liveness" variables and code out of 
> frysk.proc and into frysk.proc.live.LiveTask (e.g., implementation of 
> requestAddObserver blah) I can move forward and fix this problem.
>
> Can we do that?

We can certainly refactor the proc interfaces and implementation
subpackages. But you are mixing two things in your plan. The renaming of
existing packages and classes (which I don't see any benefit of) and
exposing flaws in of the refactoring of the proc, proc.ptrace and
proc.core code because there is a clear separation between interface and
implementation.

Cheers,

Mark

  reply	other threads:[~2007-06-18 18:56 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-13 13:17 Andrew Cagney
2007-06-14 10:11 ` Mark Wielaard
2007-06-14 14:08   ` Andrew Cagney
2007-06-15  9:05     ` Mark Wielaard
2007-06-15 15:04       ` Andrew Cagney
2007-06-15 19:44         ` Mark Wielaard
2007-06-18 18:15           ` Andrew Cagney
2007-06-18 19:06             ` Mark Wielaard [this message]
2007-06-18 23:29               ` Andrew Cagney
2007-06-19  8:59                 ` Mark Wielaard
2007-06-19 22:30                   ` Andrew Cagney
2007-06-20  9:21                     ` Mark Wielaard
2007-06-19  8:32               ` Andrew Cagney

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=1182192973.4534.90.camel@dijkstra.wildebeest.org \
    --to=mark@klomp.org \
    --cc=cagney@redhat.com \
    --cc=frysk@sourceware.org \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).