From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26746 invoked by alias); 13 Mar 2008 15:47:57 -0000 Received: (qmail 26719 invoked by uid 9519); 13 Mar 2008 15:47:56 -0000 Date: Thu, 13 Mar 2008 15:47:00 -0000 Message-ID: <20080313154756.26703.qmail@sourceware.org> From: rmoseley@sourceware.org To: frysk-cvs@sourceware.org Subject: [SCM] master: Fix minor logging errors. X-Git-Refname: refs/heads/master X-Git-Reftype: branch X-Git-Oldrev: 8901497a27c497b0781b8431ac773b73ef6eeeec X-Git-Newrev: f7bc2b759574fce3842b3b60fe473a9bb8b10431 Mailing-List: contact frysk-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: frysk-cvs-owner@sourceware.org Reply-To: frysk@sourceware.org X-SW-Source: 2008-q1/txt/msg00351.txt.bz2 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 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 + + * CLI.java: Synchronize message adding when type specified. + * LoadCommand.java: Use message logger instead of direct print. + 2008-03-10 Mark Wielaard * 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