public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
* [SCM]  master: Use word size of machine to create long type, instead of hard coded size of 8 bytes.
@ 2007-11-20 15:27 tthomas
  0 siblings, 0 replies; only message in thread
From: tthomas @ 2007-11-20 15:27 UTC (permalink / raw)
  To: frysk-cvs

The branch, master has been updated
       via  3f179370e640e8f156df2f90a48855ed11446e34 (commit)
      from  254811a3ee56ca12371db0ada5fb03187fbe44cd (commit)

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

- Log -----------------------------------------------------------------
commit 3f179370e640e8f156df2f90a48855ed11446e34
Author: Teresa Thomas <tthomas@redhat.com>
Date:   Tue Nov 20 10:24:28 2007 -0500

    Use word size of machine to create long type, instead of hard coded size of 8 bytes.
    
    frysk-core/frysk/expr/ChangeLog
    2007-11-20  Teresa Thomas  <tthomas@redhat.com>
    
    	* CExprEvaluator.g: Use word size of machine to
    	create long type.
    	* TestbedSymTab.java (getWordSize): Return machine
    	word size.

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

Summary of changes:
 frysk-core/frysk/expr/CExprEvaluator.g   |    2 +-
 frysk-core/frysk/expr/ChangeLog          |    7 +++++++
 frysk-core/frysk/expr/TestbedSymTab.java |    6 +++++-
 3 files changed, 13 insertions(+), 2 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/expr/CExprEvaluator.g b/frysk-core/frysk/expr/CExprEvaluator.g
index 5de32c6..976e8a4 100644
--- a/frysk-core/frysk/expr/CExprEvaluator.g
+++ b/frysk-core/frysk/expr/CExprEvaluator.g
@@ -111,7 +111,7 @@ options {
         // FIXME: The ExprSymTab can provide types such as this.
         shortType = new SignedType("short", ByteOrder.LITTLE_ENDIAN, 2);
         intType = new SignedType("int", ByteOrder.LITTLE_ENDIAN, 4);
-        longType = new SignedType("long", ByteOrder.LITTLE_ENDIAN, 8);
+        longType = new SignedType("long", ByteOrder.LITTLE_ENDIAN, exprSymTab.getWordSize());
         floatType = new FloatingPointType("false", ByteOrder.LITTLE_ENDIAN, 4);
         doubleType = new FloatingPointType("double", ByteOrder.LITTLE_ENDIAN, 8);
     }
diff --git a/frysk-core/frysk/expr/ChangeLog b/frysk-core/frysk/expr/ChangeLog
index 08a71da..eb3fbc5 100644
--- a/frysk-core/frysk/expr/ChangeLog
+++ b/frysk-core/frysk/expr/ChangeLog
@@ -1,3 +1,10 @@
+2007-11-20  Teresa Thomas  <tthomas@redhat.com>
+	
+	* CExprEvaluator.g: Use word size of machine to
+	create long type.
+	* TestbedSymTab.java (getWordSize): Return machine
+	word size.	
+	
 2007-11-18  Andrew Cagney  <cagney@redhat.com>
 
 	* IncompleteMemberException.java: Update Type.complete renamed to
diff --git a/frysk-core/frysk/expr/TestbedSymTab.java b/frysk-core/frysk/expr/TestbedSymTab.java
index a39b2b1..9dab515 100644
--- a/frysk-core/frysk/expr/TestbedSymTab.java
+++ b/frysk-core/frysk/expr/TestbedSymTab.java
@@ -43,6 +43,8 @@ import java.util.Iterator;
 import inua.eio.ByteBuffer;
 import inua.eio.ByteOrder;
 import java.util.List;
+
+import frysk.Config;
 import frysk.value.ObjectDeclaration;
 import frysk.value.Type;
 import frysk.value.Value;
@@ -119,7 +121,9 @@ class TestbedSymTab implements ExprSymTab {
      * Return the wordsize.
      */
     public int getWordSize() {
-	throw new RuntimeException("no word size");
+        // Since no debugee word size available, return word size of 
+	// debugger. Required for some expression evaluation, test cases.
+	return Config.getWordSize ();
     }
 
     public void complete(String incomplete, List candidates) {


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


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2007-11-20 15:27 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-11-20 15:27 [SCM] master: Use word size of machine to create long type, instead of hard coded size of 8 bytes tthomas

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).