public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
* [SCM]  master: Don't double-parse fexe <exe>.
@ 2008-03-31 20:47 cagney
  0 siblings, 0 replies; only message in thread
From: cagney @ 2008-03-31 20:47 UTC (permalink / raw)
  To: frysk-cvs

The branch, master has been updated
       via  ef004caee59a22972827bc8ee5fb2b7a105bb7cb (commit)
      from  b12abe2ed615acb2de7c967882c29b27b3bfbd66 (commit)

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

- Log -----------------------------------------------------------------
commit ef004caee59a22972827bc8ee5fb2b7a105bb7cb
Author: Andrew Cagney <cagney@redhat.com>
Date:   Mon Mar 31 16:46:53 2008 -0400

    Don't double-parse fexe <exe>.
    
    frysk-core/frysk/bindir/ChangeLog
    2008-03-31  Andrew Cagney  <cagney@redhat.com>
    
    	* ftrace.java (commandAndArguments): Change type to Proc.
    	(run(String[])): Don't un-parse the commandAndArguments; pass to
    	Ftrace.trace(Proc).
    
    frysk-core/frysk/ftrace/ChangeLog
    2008-03-31  Andrew Cagney  <cagney@redhat.com>
    
    	* Ftrace.java (trace(Proc)): Replace trace(String[]).

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

Summary of changes:
 frysk-core/frysk/bindir/ChangeLog   |    4 ++++
 frysk-core/frysk/bindir/ftrace.java |   14 +++++---------
 frysk-core/frysk/ftrace/ChangeLog   |    4 ++++
 frysk-core/frysk/ftrace/Ftrace.java |    2 +-
 4 files changed, 14 insertions(+), 10 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/bindir/ChangeLog b/frysk-core/frysk/bindir/ChangeLog
index 86abbed..74e1fbf 100644
--- a/frysk-core/frysk/bindir/ChangeLog
+++ b/frysk-core/frysk/bindir/ChangeLog
@@ -1,5 +1,9 @@
 2008-03-31  Andrew Cagney  <cagney@redhat.com>
 
+	* ftrace.java (commandAndArguments): Change type to Proc.
+	(run(String[])): Don't un-parse the commandAndArguments; pass to
+	Ftrace.trace(Proc).
+
 	* fstack.xml: Use abs_root_srcdir when specifying paths.
 	
 	* fstack.xml: Use stack-options.xml, standard-options.xml, and
diff --git a/frysk-core/frysk/bindir/ftrace.java b/frysk-core/frysk/bindir/ftrace.java
index 2efb84d..6774501 100644
--- a/frysk-core/frysk/bindir/ftrace.java
+++ b/frysk-core/frysk/bindir/ftrace.java
@@ -72,7 +72,7 @@ class ftrace {
     // True if a PID was requested.
     private boolean requestedPid;
     // Command and arguments to exec.
-    private ArrayList commandAndArguments;
+    private Proc commandAndArguments;
 
     // For configuration of overall working set.  We need to load and
     // apply rules separately, to get all log messages, that's the
@@ -314,10 +314,7 @@ class ftrace {
 
 		//@Override
 		public void parseCommand(Proc command) {
-		    String[] line = command.getCmdLine();
-		    commandAndArguments = new ArrayList();
-		    for (int i = 0; i < line.length; i++)
-			commandAndArguments.add(line[i]);
+		    commandAndArguments = command;
 		}
 
             //@Override
@@ -384,11 +381,10 @@ class ftrace {
 	}
 
         if (commandAndArguments != null) {
-            String[] cmd = (String[]) commandAndArguments.toArray(new String[0]);
-            tracer.trace(cmd);
-        }
-        else
+            tracer.trace(commandAndArguments);
+        } else {
             tracer.trace();
+	}
     }
 
     public ftrace()
diff --git a/frysk-core/frysk/ftrace/ChangeLog b/frysk-core/frysk/ftrace/ChangeLog
index e41545e..5f964f1 100644
--- a/frysk-core/frysk/ftrace/ChangeLog
+++ b/frysk-core/frysk/ftrace/ChangeLog
@@ -1,3 +1,7 @@
+2008-03-31  Andrew Cagney  <cagney@redhat.com>
+
+	* Ftrace.java (trace(Proc)): Replace trace(String[]).
+
 2008-03-20  Andrew Cagney  <cagney@redhat.com>
 
 	* Ftrace.java: Use ProcTasksAction and ProcTasksObserver.
diff --git a/frysk-core/frysk/ftrace/Ftrace.java b/frysk-core/frysk/ftrace/Ftrace.java
index d66e470..87776b0 100644
--- a/frysk-core/frysk/ftrace/Ftrace.java
+++ b/frysk-core/frysk/ftrace/Ftrace.java
@@ -189,7 +189,7 @@ public class Ftrace {
 	new ProcTasksAction(proc, tasksObserver);
     }
 
-    public void trace (String[] command) {
+    public void trace(Proc command) {
 	init();
 	Manager.host.requestCreateAttachedProc(command, attachedObserver);
 	Manager.eventLoop.run();


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


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2008-03-31 20:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-31 20:47 [SCM] master: Don't double-parse fexe <exe> cagney

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).