public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: rmoseley@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: Fix minor logging errors.
Date: Thu, 13 Mar 2008 15:47:00 -0000	[thread overview]
Message-ID: <20080313154756.26703.qmail@sourceware.org> (raw)

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


                 reply	other threads:[~2008-03-13 15:47 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=20080313154756.26703.qmail@sourceware.org \
    --to=rmoseley@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).