This is the mail archive of the
frysk-cvs@sources.redhat.com
mailing list for the frysk project.
[SCM] master: Fix minor logging errors.
- From: rmoseley at sourceware dot org
- To: frysk-cvs at sourceware dot org
- Date: 13 Mar 2008 15:47:56 -0000
- Subject: [SCM] master: Fix minor logging errors.
- Reply-to: frysk at sourceware dot org
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