public inbox for frysk-cvs@sourceware.org help / color / mirror / Atom feed
From: cagney@sourceware.org To: frysk-cvs@sourceware.org Subject: [SCM] master: Delete unused Proc.requestRefresh(). Date: Mon, 17 Mar 2008 20:35:00 -0000 [thread overview] Message-ID: <20080317203536.29552.qmail@sourceware.org> (raw) The branch, master has been updated via 5f46b1275db9112a0816092ca96a4e9816405e21 (commit) from 986b1328a40fd806e01e5f2aee29ee3a4fe0f0a9 (commit) Those revisions listed above that are new to this repository have not appeared on any other notification email. - Log ----------------------------------------------------------------- commit 5f46b1275db9112a0816092ca96a4e9816405e21 Author: Andrew Cagney <cagney@redhat.com> Date: Mon Mar 17 16:33:50 2008 -0400 Delete unused Proc.requestRefresh(). frysk-core/frysk/proc/ChangeLog 2008-03-17 Andrew Cagney <cagney@redhat.com> * Proc.java (requestRefresh()): Delete. (sendRefresh()): Make concrete. frysk-core/frysk/proc/dead/ChangeLog 2008-03-17 Andrew Cagney <cagney@redhat.com> * DeadProc.java (requestRefresh()): Delete. (sendRefresh()): Delete. frysk-core/frysk/proc/dummy/ChangeLog 2008-03-17 Andrew Cagney <cagney@redhat.com> * DummyProc.java (requestRefresh()): Delete. (sendRefresh()): Delete. frysk-core/frysk/proc/live/ChangeLog 2008-03-17 Andrew Cagney <cagney@redhat.com> * LinuxPtraceProcState.java (handleRefresh(LinuxPtraceProc)): Delete. * LinuxPtraceProc.java (requestRefresh()): Delete. ----------------------------------------------------------------------- Summary of changes: frysk-core/frysk/proc/ChangeLog | 3 +++ frysk-core/frysk/proc/Proc.java | 13 +++++-------- frysk-core/frysk/proc/dead/ChangeLog | 3 +++ frysk-core/frysk/proc/dead/DeadProc.java | 11 ----------- frysk-core/frysk/proc/dummy/ChangeLog | 3 +++ frysk-core/frysk/proc/dummy/DummyProc.java | 5 ----- frysk-core/frysk/proc/live/ChangeLog | 3 +++ frysk-core/frysk/proc/live/LinuxPtraceProc.java | 13 ------------- .../frysk/proc/live/LinuxPtraceProcState.java | 8 -------- 9 files changed, 17 insertions(+), 45 deletions(-) First 500 lines of diff: diff --git a/frysk-core/frysk/proc/ChangeLog b/frysk-core/frysk/proc/ChangeLog index 64a9d1d..4255d71 100644 --- a/frysk-core/frysk/proc/ChangeLog +++ b/frysk-core/frysk/proc/ChangeLog @@ -1,5 +1,8 @@ 2008-03-17 Andrew Cagney <cagney@redhat.com> + * Proc.java (requestRefresh()): Delete. + (sendRefresh()): Make concrete. + * ProcObserver.java (ProcAction): Move to ... * ProcBlockObserver.java: ... here; new interface. * StressTestAbandon.java: Update. diff --git a/frysk-core/frysk/proc/Proc.java b/frysk-core/frysk/proc/Proc.java index 5cd5455..4e66cb9 100644 --- a/frysk-core/frysk/proc/Proc.java +++ b/frysk-core/frysk/proc/Proc.java @@ -167,8 +167,11 @@ public abstract class Proc implements Comparable { fine.log(this, "new - create attached running proc"); } - /** XXX: Should not be public. */ - public abstract void sendRefresh(); + /** XXX: Should not be public; should be frysk.proc.live + * private. */ + public void sendRefresh() { + fine.log(this, "sendRefresh"); + } /** * Return the current state as a string. @@ -211,12 +214,6 @@ public abstract class Proc implements Comparable { }); } - /** - * Request that the Proc's task list be refreshed using system - * tables. - */ - public abstract void requestRefresh(); - protected abstract void performDetach(); /** diff --git a/frysk-core/frysk/proc/dead/ChangeLog b/frysk-core/frysk/proc/dead/ChangeLog index 2cf9a3b..8a6170a 100644 --- a/frysk-core/frysk/proc/dead/ChangeLog +++ b/frysk-core/frysk/proc/dead/ChangeLog @@ -1,5 +1,8 @@ 2008-03-17 Andrew Cagney <cagney@redhat.com> + * DeadProc.java (requestRefresh()): Delete. + (sendRefresh()): Delete. + * TestLinuxCore.java: Use ProcBlockAction. * TestLinuxCore.java (actionCoreProc()): New; replace diff --git a/frysk-core/frysk/proc/dead/DeadProc.java b/frysk-core/frysk/proc/dead/DeadProc.java index 727cd15..a96c5fc 100644 --- a/frysk-core/frysk/proc/dead/DeadProc.java +++ b/frysk-core/frysk/proc/dead/DeadProc.java @@ -63,17 +63,6 @@ public abstract class DeadProc extends Proc { return "dead"; } - /** - * Request that the Proc's task list be refreshed using system - * tables. - */ - public void requestRefresh() { - fine.log(this, "requestRefresh"); - } - public void sendRefresh() { - fine.log(this, "sendRefresh"); - } - protected void performDetach() { fine.log(this, "performDetach"); // XXX: Fake out for now. What kind of observers would you put diff --git a/frysk-core/frysk/proc/dummy/ChangeLog b/frysk-core/frysk/proc/dummy/ChangeLog index fe07764..810823f 100644 --- a/frysk-core/frysk/proc/dummy/ChangeLog +++ b/frysk-core/frysk/proc/dummy/ChangeLog @@ -1,5 +1,8 @@ 2008-03-17 Andrew Cagney <cagney@redhat.com> + * DummyProc.java (requestRefresh()): Delete. + (sendRefresh()): Delete. + * DummyHost.java: Update; using TaskAttachedObserverXXX. * DummyTask.java: Ditto. diff --git a/frysk-core/frysk/proc/dummy/DummyProc.java b/frysk-core/frysk/proc/dummy/DummyProc.java index 2096835..8dae118 100644 --- a/frysk-core/frysk/proc/dummy/DummyProc.java +++ b/frysk-core/frysk/proc/dummy/DummyProc.java @@ -71,8 +71,6 @@ public class DummyProc extends Proc { public String[] getCmdLine() { return null; } - public void sendRefresh() { - } public MemoryMap[] getMaps () { return null; } @@ -82,9 +80,6 @@ public class DummyProc extends Proc { protected String getStateFIXME() { return "<dummy>"; } - public void requestRefresh() { - throw new RuntimeException("oops!"); - } protected void performDetach() { throw new RuntimeException("oops!"); } diff --git a/frysk-core/frysk/proc/live/ChangeLog b/frysk-core/frysk/proc/live/ChangeLog index 8f4d670..39d79f9 100644 --- a/frysk-core/frysk/proc/live/ChangeLog +++ b/frysk-core/frysk/proc/live/ChangeLog @@ -1,5 +1,8 @@ 2008-03-17 Andrew Cagney <cagney@redhat.com> + * LinuxPtraceProcState.java (handleRefresh(LinuxPtraceProc)): Delete. + * LinuxPtraceProc.java (requestRefresh()): Delete. + * TestProcStopped.java: Use ProcBlockObserver. * LinuxPtraceHost.java: Update; using TaskAttachedObserverXXX. diff --git a/frysk-core/frysk/proc/live/LinuxPtraceProc.java b/frysk-core/frysk/proc/live/LinuxPtraceProc.java index b76792a..a49b232 100644 --- a/frysk-core/frysk/proc/live/LinuxPtraceProc.java +++ b/frysk-core/frysk/proc/live/LinuxPtraceProc.java @@ -339,19 +339,6 @@ public class LinuxPtraceProc extends LiveProc { } /** - * Request that the Proc's task list be refreshed using system - * tables. - */ - public void requestRefresh() { - fine.log(this, "requestRefresh"); - Manager.eventLoop.add(new ProcEvent() { - public void execute() { - newState = oldState().handleRefresh(LinuxPtraceProc.this); - } - }); - } - - /** * (Internal) Tell the process that is no longer listed in the * system table remove itself. * diff --git a/frysk-core/frysk/proc/live/LinuxPtraceProcState.java b/frysk-core/frysk/proc/live/LinuxPtraceProcState.java index 59211ad..15bbe02 100644 --- a/frysk-core/frysk/proc/live/LinuxPtraceProcState.java +++ b/frysk-core/frysk/proc/live/LinuxPtraceProcState.java @@ -61,9 +61,6 @@ abstract class LinuxPtraceProcState extends State { LinuxPtraceProcState handleRemoval(LinuxPtraceProc proc) { throw unhandled(proc, "handleRemoval"); } - LinuxPtraceProcState handleRefresh(LinuxPtraceProc proc) { - throw unhandled(proc, "handleRefresh"); - } LinuxPtraceProcState handleTaskAttachCompleted(LinuxPtraceProc proc, LinuxPtraceTask task) { throw unhandled(proc, "handleTaskAttachCompleted"); } @@ -104,11 +101,6 @@ abstract class LinuxPtraceProcState extends State { */ private static final LinuxPtraceProcState detached = new LinuxPtraceProcState ("detached") { - LinuxPtraceProcState handleRefresh(LinuxPtraceProc proc) { - fine.log("handleRefresh", proc); - proc.sendRefresh (); - return detached; - } LinuxPtraceProcState handleRemoval(LinuxPtraceProc proc) { fine.log("handleRemoval", proc); // XXX: What about a dieing proc's tasks, have a hooks/post-receive -- frysk system monitor/debugger
reply other threads:[~2008-03-17 20:35 UTC|newest] Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
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=20080317203536.29552.qmail@sourceware.org \ --to=cagney@sourceware.org \ --cc=frysk-cvs@sourceware.org \ --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: linkBe 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).