public inbox for frysk-cvs@sourceware.org
help / color / mirror / Atom feed
From: cagney@sourceware.org
To: frysk-cvs@sourceware.org
Subject: [SCM]  master: Better match TestSymbol and funit-symbols.S.
Date: Fri, 20 Jun 2008 01:59:00 -0000	[thread overview]
Message-ID: <20080620015917.12866.qmail@sourceware.org> (raw)

The branch, master has been updated
       via  c739492c8f74c2ac21aee9dd17f7bad43cd413c5 (commit)
      from  847c1c52413c34619621fad761bc8e47c897c978 (commit)

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

- Log -----------------------------------------------------------------
commit c739492c8f74c2ac21aee9dd17f7bad43cd413c5
Author: Andrew Cagney <cagney@redhat.com>
Date:   Thu Jun 19 21:59:09 2008 -0400

    Better match TestSymbol and funit-symbols.S.
    
    frysk-core/frysk/symtab/ChangeLog
    2008-06-19  Andrew Cagney  <cagney@redhat.com>
    
    	* TestSymbol.java: Re-number arg param so that matching
    	funit-symbols.S is easier.

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

Summary of changes:
 frysk-core/frysk/symtab/ChangeLog       |    5 ++
 frysk-core/frysk/symtab/TestSymbol.java |   78 +++++++++++++++---------------
 2 files changed, 44 insertions(+), 39 deletions(-)

First 500 lines of diff:
diff --git a/frysk-core/frysk/symtab/ChangeLog b/frysk-core/frysk/symtab/ChangeLog
index ba96d4e..a7910a0 100644
--- a/frysk-core/frysk/symtab/ChangeLog
+++ b/frysk-core/frysk/symtab/ChangeLog
@@ -1,3 +1,8 @@
+2008-06-19  Andrew Cagney  <cagney@redhat.com>
+
+	* TestSymbol.java: Re-number arg param so that matching
+	funit-symbols.S is easier.
+
 2008-06-05  Andrew Cagney  <cagney@redhat.com>
 
 	* DwflSymbol.java: Use DwflDie.
diff --git a/frysk-core/frysk/symtab/TestSymbol.java b/frysk-core/frysk/symtab/TestSymbol.java
index 79bf39a..a94c2ae 100644
--- a/frysk-core/frysk/symtab/TestSymbol.java
+++ b/frysk-core/frysk/symtab/TestSymbol.java
@@ -46,13 +46,13 @@ import frysk.testbed.DaemonBlockedAtSignal;
 public class TestSymbol
     extends TestLib
 {
-    private void symbolTest (String command, int numberOfArgs,
-			     String name, boolean addressValid,
-			     boolean sizeValid) {
-	// Construct an argument list containing numberOfArgs dummy
-	// arguments.  The inferior program just looks at ARGC to
-	// determine what to do.
-	String[] fullCommand = new String[numberOfArgs + 1];
+    private void symbolTest(String command, int numberOfArgs,
+			    String name, boolean addressValid,
+			    boolean sizeValid) {
+	// Construct an argument list numberOfArgs long (including
+	// arg[0] the program).  The inferior program just looks at
+	// ARGC to determine what to do.
+	String[] fullCommand = new String[numberOfArgs];
 	fullCommand[0] =  getExecPath (command);
     	for (int i = 1; i < fullCommand.length; i++) {
 	    fullCommand[i] = Integer.toString(i);
@@ -80,128 +80,128 @@ public class TestSymbol
     private String unknown = SymbolFactory.UNKNOWN.getName ();
 
     public void testDebug () {
-	symbolTest("funit-symbols", 1, "global_st_size", true, true);
+	symbolTest("funit-symbols", 2, "global_st_size", true, true);
     }
   
     public void testNoDebug () {
-	symbolTest("funit-symbols-nodebug", 1, "global_st_size", true, true);
+	symbolTest("funit-symbols-nodebug", 2, "global_st_size", true, true);
     }
   
     public void testStripped () {
-	symbolTest("funit-symbols-stripped", 1, unknown, false, false);
+	symbolTest("funit-symbols-stripped", 2, unknown, false, false);
     }
   
     public void testStaticDebug () {
-	symbolTest("funit-symbols", 2, "local_st_size", true, true);
+	symbolTest("funit-symbols", 3, "local_st_size", true, true);
     }
   
     public void testStaticNoDebug () {
-	symbolTest("funit-symbols-nodebug", 2, "local_st_size", true, true);
+	symbolTest("funit-symbols-nodebug", 3, "local_st_size", true, true);
     }
   
     public void testStaticStripped () {
-	symbolTest("funit-symbols-stripped", 2, unknown, false, false);
+	symbolTest("funit-symbols-stripped", 3, unknown, false, false);
     }
   
     public void testNoSize() {
-	symbolTest("funit-symbols", 3, "global_st_size_0", true, false);
+	symbolTest("funit-symbols", 4, "global_st_size_0", true, false);
     }
   
     public void testNoDebugNoSize() {
-	symbolTest("funit-symbols-nodebug", 3, "global_st_size_0",
+	symbolTest("funit-symbols-nodebug", 4, "global_st_size_0",
 		   true, false);   
     }
   
     public void testStrippedNoSize() {
-	symbolTest("funit-symbols-stripped", 3, unknown, false, false);    
+	symbolTest("funit-symbols-stripped", 4, unknown, false, false);    
     }
   
     public void testStaticNoSize() {
-	symbolTest("funit-symbols", 4, "local_st_size_0", true, false);    
+	symbolTest("funit-symbols", 5, "local_st_size_0", true, false);    
     }
     public void testStaticNoDebugNoSize() {
-	symbolTest("funit-symbols-nodebug", 4, "local_st_size_0", true, false);   
+	symbolTest("funit-symbols-nodebug", 5, "local_st_size_0", true, false);   
     }
     public void testStaticStrippedNoSize() {
-	symbolTest("funit-symbols-stripped", 4, unknown, false, false);    
+	symbolTest("funit-symbols-stripped", 5, unknown, false, false);    
     }
 
     public void testGlobalInGlobal() {
-	symbolTest(5, "global_in_global", true, true);
+	symbolTest(6, "global_in_global", true, true);
     }
     public void testLocalInGlobal() {
         if (unresolved(5941))
             return;
-	symbolTest(6, "local_in_global", true, true);
+	symbolTest(7, "local_in_global", true, true);
     }
     public void testGlobalInLocal() {
-	symbolTest(7, "global_in_local", true, true);
+	symbolTest(8, "global_in_local", true, true);
     }
     public void testLocalInLocal() {
-	symbolTest(8, "local_in_local", true, true);
+	symbolTest(9, "local_in_local", true, true);
     }
 
     public void testGlobalAfterNested() {
-	symbolTest(9, "global_outer", true, true);
+	symbolTest(10, "global_outer", true, true);
     }
     public void testLocalAfterNested() {
-	symbolTest(10, "local_outer", true, true);
+	symbolTest(11, "local_outer", true, true);
     }
 
     public void testNoSymbolAfterGlobal() {
         if (unresolved(5941))
             return;
-	symbolTest(11, unknown, false, false);
+	symbolTest(12, unknown, false, false);
     }
     public void testNoSymbolAfterLocal() {
         if (unresolved(5941))
             return;
-	symbolTest(12, unknown, false, false);
+	symbolTest(13, unknown, false, false);
     }
 
     public void testGlobalSize0InGlobal() {
-	symbolTest(13, "global_after_0", true, true);
+	symbolTest(14, "global_after_0", true, true);
     }
     public void testLocalSize0InGlobal() {
-	symbolTest(14, "global_after_0", true, true);
+	symbolTest(15, "global_after_0", true, true);
     }
     public void testGlobalSize0InLocal() {
-	symbolTest(15, "local_after_0", true, true);
+	symbolTest(16, "local_after_0", true, true);
     }
     public void testLocalSize0InLocal() {
-	symbolTest(16, "local_after_0", true, true);
+	symbolTest(17, "local_after_0", true, true);
     }
 
     public void testGlobalAfterNestedSize0() {
-	symbolTest(17, "global_after_0", true, true);
+	symbolTest(18, "global_after_0", true, true);
     }
     public void testLocalAfterNestedSize0() {
-	symbolTest(18, "local_after_0", true, true);
+	symbolTest(19, "local_after_0", true, true);
     }
 
     public void testSmallGlobalAtLargeGlobal() {
-	symbolTest(19, "small_global_at_large_global", true, true);
+	symbolTest(20, "small_global_at_large_global", true, true);
     }
     public void testSmallLocalAtLargeGlobal() {
         if (unresolved(5941))
             return;
-	symbolTest(20, "small_local_at_large_global", true, true);
+	symbolTest(21, "small_local_at_large_global", true, true);
     }
     public void testSmallGlobalAtLargeLocal() {
-	symbolTest(21, "small_global_at_large_local", true, true);
+	symbolTest(22, "small_global_at_large_local", true, true);
     }
     public void testSmallLocalAtLargeLocal() {
-	symbolTest(22, "small_local_at_large_local", true, true);
+	symbolTest(23, "small_local_at_large_local", true, true);
     }
 
     public void testAfterGlobalContiningSize0() {
         if (unresolved(5941))
             return;
-	symbolTest(23, unknown, false, false);
+	symbolTest(24, unknown, false, false);
     }
     public void testAfterLocalContiningSize0() {
         if (unresolved(5941))
             return;
-	symbolTest(24, unknown, false, false);
+	symbolTest(25, unknown, false, false);
     }
 }


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


                 reply	other threads:[~2008-06-20  1:59 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=20080620015917.12866.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).