public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
* [SCM]  master: Fix minor logging errors.
@ 2008-03-13 15:47 rmoseley
  0 siblings, 0 replies; only message in thread
From: rmoseley @ 2008-03-13 15:47 UTC (permalink / raw)
  To: frysk-cvs

The branch, master has been updated
       via  f7bc2b759574fce3842b3b60fe473a9bb8b10431 (commit)
      from  8901497a27c497b0781b8431ac773b73ef6eeeec (commit)

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

- Log -----------------------------------------------------------------
commit f7bc2b759574fce3842b3b60fe473a9bb8b10431
Author: Rick Moseley <rmoseley@localhost.localdomain>
Date:   Thu Mar 13 10:47:17 2008 -0500

    Fix minor logging errors.
    
    * CLI.java: Synchronize message adding when type specified.
    * LoadCommand.java: Use message logger instead of direct print.

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

Summary of changes:
 frysk-core/frysk/hpd/CLI.java         |    4 +++-
 frysk-core/frysk/hpd/ChangeLog        |    5 +++++
 frysk-core/frysk/hpd/LoadCommand.java |    2 +-
 3 files changed, 9 insertions(+), 2 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/hpd/CLI.java b/frysk-core/frysk/hpd/CLI.java
index dc85569..a95bcdf 100644
--- a/frysk-core/frysk/hpd/CLI.java
+++ b/frysk-core/frysk/hpd/CLI.java
@@ -332,7 +332,9 @@ public class CLI {
     }
 
     void addMessage(String msg, int type) {
-        addMessage(new Message(msg, type));
+	synchronized (messages) {
+	    addMessage(new Message(msg, type));
+	}
     }
 
     private void flushMessages() {
diff --git a/frysk-core/frysk/hpd/ChangeLog b/frysk-core/frysk/hpd/ChangeLog
index be519b5..d4a47a1 100644
--- a/frysk-core/frysk/hpd/ChangeLog
+++ b/frysk-core/frysk/hpd/ChangeLog
@@ -1,3 +1,8 @@
+2008-03-13  Rick Moseley  <rmoseley@redhat.com>
+
+	* CLI.java: Synchronize message adding when type specified.
+	* LoadCommand.java: Use message logger instead of direct print.
+
 2008-03-10  Mark Wielaard  <mwielaard@redhat.com>
 
 	* TestActionsCommand: Use tighter expect regular expressions.
diff --git a/frysk-core/frysk/hpd/LoadCommand.java b/frysk-core/frysk/hpd/LoadCommand.java
index ffc34e6..73fe7f4 100644
--- a/frysk-core/frysk/hpd/LoadCommand.java
+++ b/frysk-core/frysk/hpd/LoadCommand.java
@@ -144,7 +144,7 @@ public class LoadCommand extends ParameterizedCommand {
 	    cli.getLoadedProcs().put(exeProc, new Integer(procID));
 	}
 
-	cli.outWriter.println("Loaded executable file: " + exeProc.getExe());
+	cli.addMessage("Loaded executable file: " + exeProc.getExe(), Message.TYPE_NORMAL);
     }
     
     int completer(CLI cli, Input input, int cursor, List completions) {


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


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

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

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-13 15:47 [SCM] master: Fix minor logging errors rmoseley

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