public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: swagiaal@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: swagiaal: Removed unused method getType() from ExprSymTab.
Date: Fri, 04 Apr 2008 15:40:00 -0000	[thread overview]
Message-ID: <20080404154025.25493.qmail@sourceware.org> (raw)

The branch, master has been updated
       via  246616427a876f8fff03660b0b59ee034e76d517 (commit)
      from  dfa543f7189c681764596fc958a6d640a9558d46 (commit)

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

- Log -----------------------------------------------------------------
commit 246616427a876f8fff03660b0b59ee034e76d517
Author: Sami Wagiaalla <swagiaal@redhat.com>
Date:   Fri Apr 4 11:39:23 2008 -0400

    swagiaal: Removed unused method getType() from ExprSymTab.
    
    frysk-core/frysk/debuginfo/ChangeLog
    +2008-04-04  Sami Wagiaalla  <swagiaal@redhat.com>
    +
    +	* ObjectDeclarationSearchEngine.java: Removed getType().
    +
    
    frysk-core/frysk/expr/ChangeLog
    +2008-04-04  Sami Wagiaalla  <swagiaal@redhat.com>
    +
    +	* ExprSymTab.java: Removed getType().
    +

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

Summary of changes:
 frysk-core/frysk/debuginfo/ChangeLog               |    4 +++
 .../debuginfo/ObjectDeclarationSearchEngine.java   |   24 ++++++++-----------
 frysk-core/frysk/expr/ChangeLog                    |    4 +++
 frysk-core/frysk/expr/ExprSymTab.java              |    9 ++-----
 4 files changed, 21 insertions(+), 20 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/debuginfo/ChangeLog b/frysk-core/frysk/debuginfo/ChangeLog
index 9779c5d..7383e48 100644
--- a/frysk-core/frysk/debuginfo/ChangeLog
+++ b/frysk-core/frysk/debuginfo/ChangeLog
@@ -1,3 +1,7 @@
+2008-04-04  Sami Wagiaalla  <swagiaal@redhat.com>
+
+	* ObjectDeclarationSearchEngine.java: Removed getType().
+
 2008-04-01  Andrew Cagney  <cagney@redhat.com>
 
 	* TestGccInterface.java: Move from frysk.vale.TestInterface.
diff --git a/frysk-core/frysk/debuginfo/ObjectDeclarationSearchEngine.java b/frysk-core/frysk/debuginfo/ObjectDeclarationSearchEngine.java
index e10c231..c47ecd1 100644
--- a/frysk-core/frysk/debuginfo/ObjectDeclarationSearchEngine.java
+++ b/frysk-core/frysk/debuginfo/ObjectDeclarationSearchEngine.java
@@ -39,26 +39,26 @@
 
 package frysk.debuginfo;
 
-import frysk.dwfl.DwflCache;
-import java.util.Iterator;
 import inua.eio.ByteBuffer;
 import inua.eio.ByteOrder;
-import lib.dwfl.Dwfl;
-import lib.dwfl.DwflDieBias;
-import lib.dwfl.DwarfDie;
+
+import java.util.Iterator;
 import java.util.LinkedList;
 import java.util.List;
+
+import lib.dwfl.DwarfDie;
+import lib.dwfl.Dwfl;
+import lib.dwfl.DwflDieBias;
+import frysk.dwfl.DwflCache;
 import frysk.expr.ExprSymTab;
-import frysk.isa.ISA;
 import frysk.isa.registers.Register;
 import frysk.isa.registers.Registers;
 import frysk.isa.registers.RegistersFactory;
 import frysk.proc.Task;
 import frysk.scopes.Scope;
+import frysk.scopes.Variable;
 import frysk.value.ObjectDeclaration;
-import frysk.value.Type;
 import frysk.value.Value;
-import frysk.scopes.Variable;
 
 /**
  * This engine implements the c++ scoping rules and uses when searching for
@@ -70,12 +70,12 @@ import frysk.scopes.Variable;
 public class ObjectDeclarationSearchEngine implements ExprSymTab{
 
     private final DebugInfoFrame frame;
-    private final ISA isa;
+//    private final ISA isa;
     private final Task task;
 
     public ObjectDeclarationSearchEngine(DebugInfoFrame frame) {
 	this.frame = frame;
-	this.isa = frame.getTask().getISA();
+//	this.isa = frame.getTask().getISA();
 	this.task = frame.getTask();
     }
     
@@ -95,10 +95,6 @@ public class ObjectDeclarationSearchEngine implements ExprSymTab{
 	throw new ObjectDeclarationNotFoundException(name);
     }
 
-    public Type getType(Variable variable) {
-	return variable.getType(isa);
-    }
-
     public Value getValue(String s) {
 	if (s.charAt(0) == '$') {
 	    Registers regs = RegistersFactory.getRegisters(frame.getTask()
diff --git a/frysk-core/frysk/expr/ChangeLog b/frysk-core/frysk/expr/ChangeLog
index da120d8..8f2608d 100644
--- a/frysk-core/frysk/expr/ChangeLog
+++ b/frysk-core/frysk/expr/ChangeLog
@@ -1,3 +1,7 @@
+2008-04-04  Sami Wagiaalla  <swagiaal@redhat.com>
+
+	* ExprSymTab.java: Removed getType().
+
 2008-04-01  Andrew Cagney  <cagney@redhat.com>
 
 	* ExprSymTab.java: Update; Variable moved.
diff --git a/frysk-core/frysk/expr/ExprSymTab.java b/frysk-core/frysk/expr/ExprSymTab.java
index 0af6cc9..9c1c9be 100644
--- a/frysk-core/frysk/expr/ExprSymTab.java
+++ b/frysk-core/frysk/expr/ExprSymTab.java
@@ -41,11 +41,12 @@ package frysk.expr;
 
 import inua.eio.ByteBuffer;
 import inua.eio.ByteOrder;
+
 import java.util.List;
+
+import frysk.scopes.Variable;
 import frysk.value.ObjectDeclaration;
-import frysk.value.Type;
 import frysk.value.Value;
-import frysk.scopes.Variable;
 
 public interface ExprSymTab
 {
@@ -70,10 +71,6 @@ public interface ExprSymTab
      */
     Value getValue(Variable v);
     /**
-     * Given a variable, return its type.
-     */
-    Type getType(Variable variable);
-    /**
      * Return the wordsize.
      */    
     int getWordSize();


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


                 reply	other threads:[~2008-04-04 15:40 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=20080404154025.25493.qmail@sourceware.org \
    --to=swagiaal@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).