public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: cagney@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM] master: Add DEFAULT and WARNING log levels.
Date: Tue, 11 Mar 2008 23:56:00 -0000 [thread overview]
Message-ID: <20080311235611.6541.qmail@sourceware.org> (raw)
The branch, master has been updated
via 4abb4a8829293f641e08ec74c12dde3183c5f376 (commit)
from 84bcf09e5a329252d81e853e49f0cf1449f937c2 (commit)
Those revisions listed above that are new to this repository have
not appeared on any other notification email.
- Log -----------------------------------------------------------------
commit 4abb4a8829293f641e08ec74c12dde3183c5f376
Author: Andrew Cagney <cagney@redhat.com>
Date: Tue Mar 11 19:53:42 2008 -0400
Add DEFAULT and WARNING log levels.
frysk-core/frysk/debuginfo/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* CompilerVersionFactory.java: Use LogFactory.
* TypeEntry.java: Ditto.
frysk-core/frysk/proc/dead/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* CorefileByteBuffer.java: Use LogFactory.
frysk-core/frysk/proc/live/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* LinuxPtraceProcState.java: Use LogFactory.
frysk-core/frysk/testbed/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* StatState.java: Use LogFactory.
frysk-core/frysk/value/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* Variable.java: Use LogFactory.
frysk-sys/frysk/expunit/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* Child.java: Use LogFinest.
frysk-sys/frysk/rsl/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* Log.java (finest(Class)): Delete.
* LogFactory.java (warning(String)): New.
(warning(Class)): New.
* Level.mkenum (WARNING, DEFAULT): Add.
* Node.java: Default to Level.DEFAULT.
frysk-sys/frysk/sys/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* Wait.java: Use LogFactory.
frysk-sys/lib/unwind/ChangeLog
2008-03-11 Andrew Cagney <cagney@redhat.com>
* Cursor.java: Use LogFactory.
-----------------------------------------------------------------------
Summary of changes:
frysk-core/frysk/debuginfo/ChangeLog | 5 +++++
.../frysk/debuginfo/CompilerVersionFactory.java | 7 ++++---
frysk-core/frysk/debuginfo/TypeEntry.java | 7 ++++---
frysk-core/frysk/proc/dead/ChangeLog | 2 ++
frysk-core/frysk/proc/dead/CorefileByteBuffer.java | 3 ++-
frysk-core/frysk/proc/live/ChangeLog | 2 ++
.../frysk/proc/live/LinuxPtraceProcState.java | 5 +++--
frysk-core/frysk/testbed/ChangeLog | 2 ++
frysk-core/frysk/testbed/StatState.java | 5 +++--
frysk-core/frysk/value/ChangeLog | 4 ++++
frysk-core/frysk/value/Variable.java | 7 ++++---
frysk-sys/frysk/expunit/ChangeLog | 4 ++++
frysk-sys/frysk/expunit/Child.java | 5 +++--
frysk-sys/frysk/rsl/ChangeLog | 8 ++++++++
frysk-sys/frysk/rsl/Level.mkenum | 7 ++++---
frysk-sys/frysk/rsl/Log.java | 9 ++-------
frysk-sys/frysk/rsl/LogFactory.java | 6 ++++++
frysk-sys/frysk/rsl/Node.java | 2 +-
frysk-sys/frysk/sys/ChangeLog | 4 ++++
frysk-sys/frysk/sys/Wait.java | 5 +++--
frysk-sys/lib/unwind/ChangeLog | 4 ++++
frysk-sys/lib/unwind/Cursor.java | 5 +++--
frysk-sys/lib/unwind/Unwind.java | 6 +++---
23 files changed, 80 insertions(+), 34 deletions(-)
First 500 lines of diff:
diff --git a/frysk-core/frysk/debuginfo/ChangeLog b/frysk-core/frysk/debuginfo/ChangeLog
index e28b011..1d3f91a 100644
--- a/frysk-core/frysk/debuginfo/ChangeLog
+++ b/frysk-core/frysk/debuginfo/ChangeLog
@@ -1,3 +1,8 @@
+2008-03-11 Andrew Cagney <cagney@redhat.com>
+
+ * CompilerVersionFactory.java: Use LogFactory.
+ * TypeEntry.java: Ditto.
+
2008-03-10 Sami Wagiaalla <swagiaal@redhat.com>
* DebugInfoStackFactory.java: Added DEFAULT options.
diff --git a/frysk-core/frysk/debuginfo/CompilerVersionFactory.java b/frysk-core/frysk/debuginfo/CompilerVersionFactory.java
index 1eaef01..da0317a 100644
--- a/frysk-core/frysk/debuginfo/CompilerVersionFactory.java
+++ b/frysk-core/frysk/debuginfo/CompilerVersionFactory.java
@@ -1,6 +1,6 @@
// This file is part of the program FRYSK.
//
-// Copyright 2007, Red Hat Inc.
+// Copyright 2007, 2008, Red Hat Inc.
//
// FRYSK is free software; you can redistribute it and/or modify it
// under the terms of the GNU General Public License as published by
@@ -41,11 +41,12 @@ package frysk.debuginfo;
import java.util.HashMap;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
public class CompilerVersionFactory {
private static HashMap compilerVersions = new HashMap();
- protected static Log fine = Log.fine(CompilerVersionFactory.class);
- protected static Log finest = Log.finest(CompilerVersionFactory.class);
+ protected static Log fine = LogFactory.fine(CompilerVersionFactory.class);
+ protected static Log finest = LogFactory.finest(CompilerVersionFactory.class);
public static CompilerVersion getCompilerVersion(String compiler) {
if (compilerVersions.containsKey(compiler))
diff --git a/frysk-core/frysk/debuginfo/TypeEntry.java b/frysk-core/frysk/debuginfo/TypeEntry.java
index 520ffa8..9ae3f03 100644
--- a/frysk-core/frysk/debuginfo/TypeEntry.java
+++ b/frysk-core/frysk/debuginfo/TypeEntry.java
@@ -1,6 +1,6 @@
// This file is part of the program FRYSK.
//
-// Copyright 2007, Red Hat Inc.
+// Copyright 2007, 2008, Red Hat Inc.
//
// FRYSK is free software; you can redistribute it and/or modify it
// under the terms of the GNU General Public License as published by
@@ -41,6 +41,7 @@ package frysk.debuginfo;
import frysk.isa.ISA;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
import frysk.scopes.SourceLocation;
import frysk.value.Access;
import frysk.value.ArrayType;
@@ -78,8 +79,8 @@ import lib.dwfl.DwTag;
import lib.dwfl.DwarfDie;
public class TypeEntry {
- static protected Log fine = Log.fine(TypeEntry.class);
- static protected Log finest = Log.finest(TypeEntry.class);
+ static protected Log fine = LogFactory.fine(TypeEntry.class);
+ static protected Log finest = LogFactory.finest(TypeEntry.class);
private final ByteOrder byteorder;
private final HashMap dieHash;
diff --git a/frysk-core/frysk/proc/dead/ChangeLog b/frysk-core/frysk/proc/dead/ChangeLog
index c452884..7d058c2 100644
--- a/frysk-core/frysk/proc/dead/ChangeLog
+++ b/frysk-core/frysk/proc/dead/ChangeLog
@@ -1,5 +1,7 @@
2008-03-11 Andrew Cagney <cagney@redhat.com>
+ * CorefileByteBuffer.java: Use LogFactory.
+
* LinuxExeProc.java (getMainTask()): Implement.
* LinuxCoreProc.java (getMainTask()): Implement.
* LinuxCoreInfo.java (constructTasks(LinuxCoreInfo)): Return the
diff --git a/frysk-core/frysk/proc/dead/CorefileByteBuffer.java b/frysk-core/frysk/proc/dead/CorefileByteBuffer.java
index 63dd0c2..0abed3b 100644
--- a/frysk-core/frysk/proc/dead/CorefileByteBuffer.java
+++ b/frysk-core/frysk/proc/dead/CorefileByteBuffer.java
@@ -49,6 +49,7 @@ import lib.dwfl.ElfException;
import lib.dwfl.ElfPHeader;
import inua.eio.ByteBuffer;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
public class CorefileByteBuffer extends ByteBuffer {
@@ -59,7 +60,7 @@ public class CorefileByteBuffer extends ByteBuffer {
StatelessFile coreFileRaw = null;
boolean fileOpen = false;
- private static final Log finest = Log.finest(CorefileByteBuffer.class);
+ private static final Log finest = LogFactory.finest(CorefileByteBuffer.class);
private CorefileByteBuffer(File file, long lowerExtreem,
long upperExtreem,
diff --git a/frysk-core/frysk/proc/live/ChangeLog b/frysk-core/frysk/proc/live/ChangeLog
index f747dea..db02f13 100644
--- a/frysk-core/frysk/proc/live/ChangeLog
+++ b/frysk-core/frysk/proc/live/ChangeLog
@@ -1,5 +1,7 @@
2008-03-11 Andrew Cagney <cagney@redhat.com>
+ * LinuxPtraceProcState.java: Use LogFactory.
+
* TestTaskObserverBlocked.java: Register offspring with
TearDownProcess.
diff --git a/frysk-core/frysk/proc/live/LinuxPtraceProcState.java b/frysk-core/frysk/proc/live/LinuxPtraceProcState.java
index 3dd8f3f..c0fb413 100644
--- a/frysk-core/frysk/proc/live/LinuxPtraceProcState.java
+++ b/frysk-core/frysk/proc/live/LinuxPtraceProcState.java
@@ -43,6 +43,7 @@ import java.util.Iterator;
import java.util.Collection;
import java.util.HashSet;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
import frysk.proc.Observation;
/**
@@ -51,8 +52,8 @@ import frysk.proc.Observation;
*/
abstract class LinuxPtraceProcState extends State {
- private static final Log fine = Log.fine(LinuxPtraceProcState.class);
- private static final Log finest = Log.finest(LinuxPtraceProcState.class);
+ private static final Log fine = LogFactory.fine(LinuxPtraceProcState.class);
+ private static final Log finest = LogFactory.finest(LinuxPtraceProcState.class);
LinuxPtraceProcState(String state) {
super (state);
diff --git a/frysk-core/frysk/testbed/ChangeLog b/frysk-core/frysk/testbed/ChangeLog
index 7e43be5..6acb136 100644
--- a/frysk-core/frysk/testbed/ChangeLog
+++ b/frysk-core/frysk/testbed/ChangeLog
@@ -1,5 +1,7 @@
2008-03-11 Andrew Cagney <cagney@redhat.com>
+ * StatState.java: Use LogFactory.
+
* TestLib.java (addToTearDown(Task)): New.
(addToTearDown(Proc)): New.
(setUp()): Check if Host.observableProcAddedXXX is null.
diff --git a/frysk-core/frysk/testbed/StatState.java b/frysk-core/frysk/testbed/StatState.java
index 1472884..adc5b22 100644
--- a/frysk-core/frysk/testbed/StatState.java
+++ b/frysk-core/frysk/testbed/StatState.java
@@ -43,6 +43,7 @@ import frysk.proc.Manager;
import frysk.junit.TestCase;
import frysk.sys.proc.Stat;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
import frysk.event.TimerEvent;
import frysk.proc.Task;
import frysk.sys.ProcessIdentifier;
@@ -53,8 +54,8 @@ import frysk.sys.ProcessIdentifierFactory;
*/
public class StatState {
- private final static Log fine = Log.fine(StatState.class);
- private final static Log finest = Log.finest(StatState.class);
+ private final static Log fine = LogFactory.fine(StatState.class);
+ private final static Log finest = LogFactory.finest(StatState.class);
private final char state;
private StatState(char state) {
diff --git a/frysk-core/frysk/value/ChangeLog b/frysk-core/frysk/value/ChangeLog
index dfee409..29b0467 100644
--- a/frysk-core/frysk/value/ChangeLog
+++ b/frysk-core/frysk/value/ChangeLog
@@ -1,3 +1,7 @@
+2008-03-11 Andrew Cagney <cagney@redhat.com>
+
+ * Variable.java: Use LogFactory.
+
2008-03-04 Andrew Cagney <cagney@redhat.com>
* TestPointer.java (testConstCharPointer()): New.
diff --git a/frysk-core/frysk/value/Variable.java b/frysk-core/frysk/value/Variable.java
index 0d80cca..89b3b18 100644
--- a/frysk-core/frysk/value/Variable.java
+++ b/frysk-core/frysk/value/Variable.java
@@ -1,6 +1,6 @@
// This file is part of the program FRYSK.
//
-// Copyright 2007, Red Hat Inc.
+// Copyright 2007, 2008, Red Hat Inc.
//
// FRYSK is free software; you can redistribute it and/or modify it
// under the terms of the GNU General Public License as published by
@@ -52,6 +52,7 @@ import frysk.debuginfo.ValueUnavailableException;
import frysk.debuginfo.VariableOptimizedOutException;
import frysk.isa.ISA;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
import frysk.scopes.SourceLocation;
/**
@@ -62,8 +63,8 @@ import frysk.scopes.SourceLocation;
public class Variable extends ObjectDeclaration{
- private static Log fine = Log.fine(Variable.class);
- private static Log finest = Log.finest(Variable.class);
+ private static Log fine = LogFactory.fine(Variable.class);
+ private static Log finest = LogFactory.finest(Variable.class);
private Type type;
private final DwarfDie variableDie;
diff --git a/frysk-sys/frysk/expunit/ChangeLog b/frysk-sys/frysk/expunit/ChangeLog
index cc9aa41..3434555 100644
--- a/frysk-sys/frysk/expunit/ChangeLog
+++ b/frysk-sys/frysk/expunit/ChangeLog
@@ -1,3 +1,7 @@
+2008-03-11 Andrew Cagney <cagney@redhat.com>
+
+ * Child.java: Use LogFinest.
+
2008-02-08 Mark Wielaard <mark@klomp.org>
* Child.java (expectMilliseconds): Pass miliseconds to
diff --git a/frysk-sys/frysk/expunit/Child.java b/frysk-sys/frysk/expunit/Child.java
index 3f051de..69281ef 100644
--- a/frysk-sys/frysk/expunit/Child.java
+++ b/frysk-sys/frysk/expunit/Child.java
@@ -44,14 +44,15 @@ import frysk.sys.ProcessIdentifier;
import frysk.sys.PseudoTerminal;
import frysk.sys.Signal;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
/**
* The child being waited on.
*/
class Child {
- static protected Log fine = Log.fine(Child.class);
- static private Log finest = Log.finest(Child.class);
+ static protected Log fine = LogFactory.fine(Child.class);
+ static private Log finest = LogFactory.finest(Child.class);
private final PseudoTerminal child;
private final ProcessIdentifier pid;
diff --git a/frysk-sys/frysk/rsl/ChangeLog b/frysk-sys/frysk/rsl/ChangeLog
index 937587a..318435f 100644
--- a/frysk-sys/frysk/rsl/ChangeLog
+++ b/frysk-sys/frysk/rsl/ChangeLog
@@ -1,3 +1,11 @@
+2008-03-11 Andrew Cagney <cagney@redhat.com>
+
+ * Log.java (finest(Class)): Delete.
+ * LogFactory.java (warning(String)): New.
+ (warning(Class)): New.
+ * Level.mkenum (WARNING, DEFAULT): Add.
+ * Node.java: Default to Level.DEFAULT.
+
2008-03-03 Andrew Cagney <cagney@redhat.com>
* Log.java (dump(Object)): Add missing else.
diff --git a/frysk-sys/frysk/rsl/Level.mkenum b/frysk-sys/frysk/rsl/Level.mkenum
index 513ac9d..12b539d 100644
--- a/frysk-sys/frysk/rsl/Level.mkenum
+++ b/frysk-sys/frysk/rsl/Level.mkenum
@@ -1,4 +1,5 @@
NONE 0
-FINE 1
-FINEST 2
-MAX 3
+WARNING|DEFAULT 1
+FINE 2
+FINEST 3
+MAX 4
diff --git a/frysk-sys/frysk/rsl/Log.java b/frysk-sys/frysk/rsl/Log.java
index 905c88a..0d2ec6c 100644
--- a/frysk-sys/frysk/rsl/Log.java
+++ b/frysk-sys/frysk/rsl/Log.java
@@ -103,17 +103,12 @@ public final class Log {
}
/**
- * For convenience, grab the FINE logger.
+ * For convenience, since this is the most common case, grab the
+ * FINE logger. For other loggers use LogFactory.
*/
public static Log fine(Class klass) {
return LogFactory.fine(klass);
}
- /**
- * For convenience, grab the FINEST logger.
- */
- public static Log finest(Class klass) {
- return LogFactory.finest(klass);
- }
// Static?
private static PrintWriter out = new PrintWriter(System.out);
diff --git a/frysk-sys/frysk/rsl/LogFactory.java b/frysk-sys/frysk/rsl/LogFactory.java
index 9b8385d..6bb02f4 100644
--- a/frysk-sys/frysk/rsl/LogFactory.java
+++ b/frysk-sys/frysk/rsl/LogFactory.java
@@ -57,6 +57,9 @@ public final class LogFactory {
public static Node get(String path) {
return get(root, path);
}
+ public static Log warning(String klass) {
+ return get(klass).get(Level.WARNING);
+ }
public static Log fine(String klass) {
return get(klass).get(Level.FINE);
}
@@ -81,6 +84,9 @@ public final class LogFactory {
public static Node get(Class klass) {
return get(root, klass);
}
+ public static Log warning(Class klass) {
+ return get(klass).get(Level.WARNING);
+ }
public static Log fine(Class klass) {
return get(klass).get(Level.FINE);
}
diff --git a/frysk-sys/frysk/rsl/Node.java b/frysk-sys/frysk/rsl/Node.java
index 81b9d4f..cd3534e 100644
--- a/frysk-sys/frysk/rsl/Node.java
+++ b/frysk-sys/frysk/rsl/Node.java
@@ -77,7 +77,7 @@ public final class Node {
* Package private; the root node.
*/
Node() {
- this(null, "<root>", new Setting(Level.NONE));
+ this(null, "<root>", new Setting(Level.DEFAULT));
}
public String toString() {
diff --git a/frysk-sys/frysk/sys/ChangeLog b/frysk-sys/frysk/sys/ChangeLog
index 302da57..5f7704f 100644
--- a/frysk-sys/frysk/sys/ChangeLog
+++ b/frysk-sys/frysk/sys/ChangeLog
@@ -1,3 +1,7 @@
+2008-03-11 Andrew Cagney <cagney@redhat.com>
+
+ * Wait.java: Use LogFactory.
+
2008-03-10 Andrew Cagney <cagney@redhat.com>
* Signal.java-sh (kill, tkill): Add String name parameter.
diff --git a/frysk-sys/frysk/sys/Wait.java b/frysk-sys/frysk/sys/Wait.java
index 4e19c28..a41038a 100644
--- a/frysk-sys/frysk/sys/Wait.java
+++ b/frysk-sys/frysk/sys/Wait.java
@@ -40,6 +40,7 @@
package frysk.sys;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
/**
* Wait for an event from either a process, task, or all processes and
@@ -55,7 +56,7 @@ public final class Wait {
// Seems that when calling a native static methods this isn't
// initialized, force it.
if (fine == null)
- fine = Log.fine(Wait.class);
+ fine = LogFactory.fine(Wait.class);
return fine;
}
private static Log fine;
@@ -67,7 +68,7 @@ public final class Wait {
// Seems that when calling a native static methods this isn't
// initialized, force it.
if (finest == null)
- finest = Log.finest(Wait.class);
+ finest = LogFactory.finest(Wait.class);
return finest;
}
private static Log finest;
diff --git a/frysk-sys/lib/unwind/ChangeLog b/frysk-sys/lib/unwind/ChangeLog
index 81397cc..d130ee4 100644
--- a/frysk-sys/lib/unwind/ChangeLog
+++ b/frysk-sys/lib/unwind/ChangeLog
@@ -1,3 +1,7 @@
+2008-03-11 Andrew Cagney <cagney@redhat.com>
+
+ * Cursor.java: Use LogFactory.
+
2008-02-11 Andrew Cagney <cagney@redhat.com>
* Unwind.java: Use frysk.rsl.
diff --git a/frysk-sys/lib/unwind/Cursor.java b/frysk-sys/lib/unwind/Cursor.java
index 970a7c3..6d0b68f 100644
--- a/frysk-sys/lib/unwind/Cursor.java
+++ b/frysk-sys/lib/unwind/Cursor.java
@@ -41,10 +41,11 @@ package lib.unwind;
import gnu.gcj.RawDataManaged;
import frysk.rsl.Log;
+import frysk.rsl.LogFactory;
public class Cursor {
- private static final Log fine = Log.fine(Cursor.class);
- private static final Log finest = Log.finest(Cursor.class);
+ private static final Log fine = LogFactory.fine(Cursor.class);
+ private static final Log finest = LogFactory.finest(Cursor.class);
final RawDataManaged cursor;
final Unwind unwinder;
diff --git a/frysk-sys/lib/unwind/Unwind.java b/frysk-sys/lib/unwind/Unwind.java
index 349fe1e..c400f66 100644
--- a/frysk-sys/lib/unwind/Unwind.java
+++ b/frysk-sys/lib/unwind/Unwind.java
@@ -40,13 +40,13 @@
package lib.unwind;
import frysk.rsl.Log;
-
+import frysk.rsl.LogFactory;
import gnu.gcj.RawDataManaged;
import gnu.gcj.RawData;
public abstract class Unwind {
- static final Log fine = Log.fine(Unwind.class);
- static final Log finest = Log.finest(Unwind.class);
+ static final Log fine = LogFactory.fine(Unwind.class);
+ static final Log finest = LogFactory.finest(Unwind.class);
abstract RawDataManaged initRemote(AddressSpace addressSpace);
hooks/post-receive
--
frysk system monitor/debugger
reply other threads:[~2008-03-11 23:56 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=20080311235611.6541.qmail@sourceware.org \
--to=cagney@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).