public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: cagney@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: Move Task.pendingObservations to LinuxPtraceTask.
Date: Wed, 16 Jan 2008 04:53:00 -0000	[thread overview]
Message-ID: <20080116045324.30049.qmail@sourceware.org> (raw)

The branch, master has been updated
       via  26aa2cd53bd6f4f9dbdf542d2969686776489fcc (commit)
       via  cab397955289fbe0c0811ad633ef694ed47f9f30 (commit)
      from  e9c42b96ada5c88bd7d1bf05c784f71e5a8dfa59 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email.

- Log -----------------------------------------------------------------
commit 26aa2cd53bd6f4f9dbdf542d2969686776489fcc
Author: Andrew Cagney <cagney@redhat.com>
Date:   Tue Jan 15 23:51:08 2008 -0500

    Move Task.pendingObservations to LinuxPtraceTask.
    
    frysk-core/frysk/proc/ChangeLog
    2008-01-15  Andrew Cagney  <cagney@redhat.com>
    
    	* Task.java (pendingObservations): Move to LinuxPtraceTask.
    
    frysk-core/frysk/proc/live/ChangeLog
    2008-01-15  Andrew Cagney  <cagney@redhat.com>
    
    	* LinuxPtraceTask.java (pendingObservations): Move here from
    	frysk.proc.Task.

commit cab397955289fbe0c0811ad633ef694ed47f9f30
Author: Andrew Cagney <cagney@redhat.com>
Date:   Tue Jan 15 23:40:40 2008 -0500

    Delete unused Task.TaskStateObservable.
    
    frysk-core/frysk/proc/ChangeLog
    2008-01-15  Andrew Cagney  <cagney@redhat.com>
    
    	* Task.java (TaskStateObservable): Delete.

-----------------------------------------------------------------------

Summary of changes:
 frysk-core/frysk/proc/ChangeLog                 |    4 +++
 frysk-core/frysk/proc/Task.java                 |   24 -----------------------
 frysk-core/frysk/proc/live/ChangeLog            |    3 ++
 frysk-core/frysk/proc/live/LinuxPtraceTask.java |    8 +++++++
 4 files changed, 15 insertions(+), 24 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/proc/ChangeLog b/frysk-core/frysk/proc/ChangeLog
index d9269ea..61ccb8a 100644
--- a/frysk-core/frysk/proc/ChangeLog
+++ b/frysk-core/frysk/proc/ChangeLog
@@ -1,5 +1,9 @@
 2008-01-15  Andrew Cagney  <cagney@redhat.com>
 
+	* Task.java (pendingObservations): Move to LinuxPtraceTask.
+
+	* Task.java (TaskStateObservable): Delete.
+
 	* TestIsa.java (DetachedObserver): Delete.
 	(testIsa()): Use StatState.
 	(testAttachDetachAttachAgainDetachAgainAttachAgainAgain()): Ditto.
diff --git a/frysk-core/frysk/proc/Task.java b/frysk-core/frysk/proc/Task.java
index 710c8c0..dcd37e0 100644
--- a/frysk-core/frysk/proc/Task.java
+++ b/frysk-core/frysk/proc/Task.java
@@ -42,13 +42,11 @@ package frysk.proc;
 import frysk.proc.TaskObserver.Terminating;
 import frysk.syscall.SyscallTable;
 import frysk.syscall.SyscallTableFactory;
-import java.util.LinkedList;
 import inua.eio.ByteBuffer;
 import java.util.Set;
 import java.util.HashSet;
 import java.util.logging.Level;
 import java.util.logging.Logger;
-import java.util.Observable;
 import frysk.isa.Register;
 import frysk.isa.ISA;
 import frysk.bank.RegisterBanks;
@@ -301,16 +299,6 @@ public abstract class Task {
     public abstract void requestDeleteInstructionObserver(TaskObserver.Instruction o);
 
     /**
-     * List containing the TaskObservations that are pending addition
-     * or deletion (in order that they were requested). Will be dealt
-     * with as soon as a stop event is received during one of the
-     * running states.
-     *
-     * XXX: Should not be public.
-     */
-    public LinkedList pendingObservations = new LinkedList();
-
-    /**
      * Return the address of the instruction that this task will
      * execute next.
      */
@@ -372,16 +360,4 @@ public abstract class Task {
 	syscallTable = null;
 	currentISA = null;
     }
-  
-    /**
-     * XXX: Temporary until .observable's are converted to
-     * .requestAddObserver.
-     */
-    public static class TaskStateObservable extends Observable {
-	public void notify(Object o) {
-	    logger.log(Level.FINE, "{0} notify -- all observers\n", o); 
-	    setChanged();
-	    notifyObservers(o);
-	}
-    }
 }
diff --git a/frysk-core/frysk/proc/live/ChangeLog b/frysk-core/frysk/proc/live/ChangeLog
index b8e3cab..5a67393 100644
--- a/frysk-core/frysk/proc/live/ChangeLog
+++ b/frysk-core/frysk/proc/live/ChangeLog
@@ -1,5 +1,8 @@
 2008-01-15  Andrew Cagney  <cagney@redhat.com>
 
+	* LinuxPtraceTask.java (pendingObservations): Move here from
+	frysk.proc.Task.
+
 	* LinuxPtraceTaskState.java: Don't notify Task.taskStateDetached.
 	* LinuxPtraceTask.java (sendDetach(int)): Call clearIsa().
 	
diff --git a/frysk-core/frysk/proc/live/LinuxPtraceTask.java b/frysk-core/frysk/proc/live/LinuxPtraceTask.java
index 20f4953..9f95ee9 100644
--- a/frysk-core/frysk/proc/live/LinuxPtraceTask.java
+++ b/frysk-core/frysk/proc/live/LinuxPtraceTask.java
@@ -39,6 +39,7 @@
 
 package frysk.proc.live;
 
+import java.util.LinkedList;
 import frysk.isa.Register;
 import frysk.isa.RegistersFactory;
 import java.util.Iterator;
@@ -903,6 +904,13 @@ public class LinuxPtraceTask extends LiveTask {
 	((LinuxPtraceProc)getProc()).requestDeleteInstructionObserver(this, instructionObservers, o);
     }
   
+    /**
+     * List containing the TaskObservations that are pending addition
+     * or deletion (in order that they were requested). Will be dealt
+     * with as soon as a stop event is received during one of the
+     * running states.
+     */
+    final LinkedList pendingObservations = new LinkedList();
 
     /**
      * (Internal) Request that all observers from this task be


hooks/post-receive
--
frysk system monitor/debugger


                 reply	other threads:[~2008-01-16  4:53 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=20080116045324.30049.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: 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).