Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Elena Zannoni <ezannoni@cygnus.com>
To: Kevin Buettner <kevinb@cygnus.com>
Cc: Daniel Berlin <dberlin@redhat.com>, gdb-patches@sources.redhat.com
Subject: Re: That dwarf2read patch i just submitted
Date: Tue, 24 Oct 2000 11:34:00 -0000	[thread overview]
Message-ID: <14837.54804.638957.639280@kwikemart.cygnus.com> (raw)
In-Reply-To: <1001023235955.ZM13394@ocotillo.lan>

The person has submitted a few other patches, always in that same
time period (spring of 99). I haven't find an assignment from the FSF.

Time to ask.

Elena


Kevin Buettner writes:
 > On Oct 23,  4:38pm, Daniel Berlin wrote:
 > 
 > > Sigh.
 > > I'll redo the work then.
 > > Is there any way i can give him credit, without having to get a
 > > copyright assignment from him?
 > 
 > In this case, you should first check to see if he has a copyright
 > assignment on file.  If he does, there's no problem.
 > 
 > If he doesn't, contact him and find out if he's willing to provide
 > one.  This can be the hard part because, although he may be perfectly
 > willing, his employer might not be.  (The employer also has to sign a
 > copyright disclaimer.) There's an added difficulty if he's changed
 > employer's since writing the patch.  (A former employer may be even
 > more unwilling than normal to sign a copyright disclaimer for work
 > done by an ex-employee.)
 > 
 > I know from first hand experience that it can be difficult to get the
 > employer to sign the copyright disclaimer.  Some are perfectly
 > willing, but their legal departments decide to hold things up.
 > 
 > Anyway... if you have trouble getting a disclaimer from either
 > the individual in question or their employer, I suppose that you
 > could try rewriting the patch yourself.  But it is probably better
 > describe in English (without using code) what needs to be done
 > to this file and get someone else who hasn't seen the patch to
 > do it.  I suppose you could then provide credit by saying something
 > like "Inspired by (but not based on) the work of ...".
 > 
 > Or, if you had an obviously different implementation, that too
 > would likely suffice.
 > 
 > If you do decide to reimplement Brian's patch, you should seek Andrew
 > Cagney's opinion first in order to make sure that your approach is
 > acceptable to the FSF.
 > 
 > Kevin
 > 
From msnyder@cygnus.com Tue Oct 24 11:37:00 2000
From: Michael Snyder <msnyder@cygnus.com>
To: gdb-patches@sourceware.cygnus.com
Subject: [PATCH]: commands.exp -- clean up long lines and indentation
Date: Tue, 24 Oct 2000 11:37:00 -0000
Message-id: <200010241837.LAA17209@cleaver.cygnus.com>
X-SW-Source: 2000-10/msg00172.html
Content-length: 26652

... because I hate long lines and bad indentation.

2000-10-24  Michael Snyder  <msnyder@cleaver.cygnus.com>

        * gdb.base/commands.exp: Break up long lines, and re-indent.

Index: commands.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.base/commands.exp,v
retrieving revision 1.5
diff -c -3 -p -r1.5 commands.exp
*** commands.exp	2000/06/07 04:14:02	1.5
--- commands.exp	2000/10/24 18:32:16
*************** proc gdbvar_simple_if_test {} {
*** 45,53 ****
  
      gdb_test "set \$foo = 0" "" "set foo in gdbvar_simple_if_test"
      # All this test should do is print 0xdeadbeef once.
!     gdb_test "if \$foo == 1\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" "\\\$\[0-9\]* = 0xdeadbeef" "gdbvar_simple_if_test #1"
      # All this test should do is print 0xfeedface once.
!     gdb_test "if \$foo == 0\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" "\\\$\[0-9\]* = 0xfeedface" "gdbvar_simple_if_test #2"
  }
  
  proc gdbvar_simple_while_test {} {
--- 45,55 ----
  
      gdb_test "set \$foo = 0" "" "set foo in gdbvar_simple_if_test"
      # All this test should do is print 0xdeadbeef once.
!     gdb_test "if \$foo == 1\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef" "gdbvar_simple_if_test #1"
      # All this test should do is print 0xfeedface once.
!     gdb_test "if \$foo == 0\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" \
! 	    "\\\$\[0-9\]* = 0xfeedface" "gdbvar_simple_if_test #2"
  }
  
  proc gdbvar_simple_while_test {} {
*************** proc gdbvar_simple_while_test {} {
*** 55,61 ****
  
      gdb_test "set \$foo = 5" "" "set foo in gdbvar_simple_while_test"
      # This test should print 0xfeedface five times.
!     gdb_test "while \$foo > 0\np/x 0xfeedface\nset \$foo -= 1\nend" "\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" "gdbvar_simple_while_test #1"
  }
  
  proc gdbvar_complex_if_while_test {} {
--- 57,65 ----
  
      gdb_test "set \$foo = 5" "" "set foo in gdbvar_simple_while_test"
      # This test should print 0xfeedface five times.
!     gdb_test "while \$foo > 0\np/x 0xfeedface\nset \$foo -= 1\nend" \
! 	    "\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" \
! 	    "gdbvar_simple_while_test #1"
  }
  
  proc gdbvar_complex_if_while_test {} {
*************** proc gdbvar_complex_if_while_test {} {
*** 63,69 ****
  
      gdb_test "set \$foo = 4" "" "set foo in gdbvar complex_if_while_test"
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while \$foo > 0\nset \$foo -= 1\nif \(\$foo % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend" "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" "gdbvar_complex_if_while_test #1"
  }
  
  proc progvar_simple_if_test {} {
--- 67,75 ----
  
      gdb_test "set \$foo = 4" "" "set foo in gdbvar complex_if_while_test"
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while \$foo > 0\nset \$foo -= 1\nif \(\$foo % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" \
! 	    "gdbvar_complex_if_while_test #1"
  }
  
  proc progvar_simple_if_test {} {
*************** proc progvar_simple_if_test {} {
*** 75,87 ****
      }
  
      if { ![runto factorial] } then { gdb_suppress_tests; }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      gdb_test "p value=5" "" "set value to 5 in progvar_simple_if_test #1"
      # All this test should do is print 0xdeadbeef once.
!     gdb_test "if value == 1\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" "\\\$\[0-9\]* = 0xdeadbeef" "progvar_simple_if_test #1"
      # All this test should do is print 0xfeedface once.
!     gdb_test "if value == 5\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" "\\\$\[0-9\]* = 0xfeedface" "progvar_simple_if_test #2"
      gdb_stop_suppressing_tests;
  }
  
--- 81,97 ----
      }
  
      if { ![runto factorial] } then { gdb_suppress_tests; }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      gdb_test "p value=5" "" "set value to 5 in progvar_simple_if_test #1"
      # All this test should do is print 0xdeadbeef once.
!     gdb_test "if value == 1\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef" \
! 	    "progvar_simple_if_test #1"
      # All this test should do is print 0xfeedface once.
!     gdb_test "if value == 5\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" \
! 	    "\\\$\[0-9\]* = 0xfeedface" \
! 	    "progvar_simple_if_test #2"
      gdb_stop_suppressing_tests;
  }
  
*************** proc progvar_simple_while_test {} {
*** 95,105 ****
  
      gdb_test "set args 5" "" "set args in progvar_simple_while_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      gdb_test "p value=5" "" "set value to 5 in progvar_simple_if_test #2"
      # This test should print 0xfeedface five times.
!     gdb_test "while value > 0\np/x 0xfeedface\nset value -= 1\nend" "\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" "progvar_simple_while_test #1"
      gdb_stop_suppressing_tests;
  }
  
--- 105,117 ----
  
      gdb_test "set args 5" "" "set args in progvar_simple_while_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      gdb_test "p value=5" "" "set value to 5 in progvar_simple_if_test #2"
      # This test should print 0xfeedface five times.
!     gdb_test "while value > 0\np/x 0xfeedface\nset value -= 1\nend" \
! 	    "\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" \
! 	    "progvar_simple_while_test #1"
      gdb_stop_suppressing_tests;
  }
  
*************** proc progvar_complex_if_while_test {} {
*** 113,123 ****
  
      gdb_test "set args 4" "" "set args in progvar_complex_if_while_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      gdb_test "p value=4" "" "set value to 4 in progvar_simple_if_test"
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while value > 0\nset value -= 1\nif \(value % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend" "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" "progvar_complex_if_while_test #1"
      gdb_stop_suppressing_tests;
  }
  
--- 125,137 ----
  
      gdb_test "set args 4" "" "set args in progvar_complex_if_while_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      gdb_test "p value=4" "" "set value to 4 in progvar_simple_if_test"
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while value > 0\nset value -= 1\nif \(value % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" \
! 	    "progvar_complex_if_while_test #1"
      gdb_stop_suppressing_tests;
  }
  
*************** proc if_while_breakpoint_command_test {}
*** 129,136 ****
  
      gdb_test "set args 5" "" "set args in if_while_breakpoint_command_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      gdb_test "p value=5" "" "set value to 5 in progvar_simple_if_test"
      delete_breakpoints
      gdb_test "break factorial" "Breakpoint.*at.*" "break factorial #1"
--- 143,150 ----
  
      gdb_test "set args 5" "" "set args in if_while_breakpoint_command_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      gdb_test "p value=5" "" "set value to 5 in progvar_simple_if_test"
      delete_breakpoints
      gdb_test "break factorial" "Breakpoint.*at.*" "break factorial #1"
*************** proc if_while_breakpoint_command_test {}
*** 145,153 ****
  	}
      }
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while value > 0\nset value -= 1\nif \(value % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend\nend" "" "commands part 2 in if_while_breakpoint_command_test"
!     gdb_test "continue" "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" "if_while_breakpoint_command_test #1"
!    gdb_test "info break" "while.*set.*if.*p/x.*else.*p/x.*end.*" "info break in if_while_breakpoint_command_test"
      gdb_stop_suppressing_tests;
  }
  
--- 159,173 ----
  	}
      }
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while value > 0\nset value -= 1\nif \(value % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend\nend" \
! 	    "" \
! 	    "commands part 2 in if_while_breakpoint_command_test"
!     gdb_test "continue" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" \
! 	    "if_while_breakpoint_command_test #1"
!    gdb_test "info break" \
! 	   "while.*set.*if.*p/x.*else.*p/x.*end.*" \
! 	   "info break in if_while_breakpoint_command_test"
      gdb_stop_suppressing_tests;
  }
  
*************** proc infrun_breakpoint_command_test {} {
*** 160,167 ****
  
      gdb_test "set args 6" "" "set args in progvar_simple_while_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      gdb_test "p value=6" "" "set value to 6 in progvar_simple_if_test #1"
      delete_breakpoints
      gdb_test "break factorial if value == 5" "Breakpoint.*at.*"
--- 180,187 ----
  
      gdb_test "set args 6" "" "set args in progvar_simple_while_test"
      if { ![runto factorial] } then { gdb_suppress_tests }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      gdb_test "p value=6" "" "set value to 6 in progvar_simple_if_test #1"
      delete_breakpoints
      gdb_test "break factorial if value == 5" "Breakpoint.*at.*"
*************** proc infrun_breakpoint_command_test {} {
*** 184,194 ****
  
      if { [istarget "hppa*-hp-hpux*"] } {
  	gdb_test "continue" \
! 	    "Continuing.*.*.*Breakpoint \[0-9\]*, factorial \\(value=5\\).*at.*\[0-9\]*\[      \]*if \\(value > 1\\) \{.*\[0-9\]*\[      \]*value \\*= factorial \\(value - 1\\);.*" \
! 	    "continue in infrun_breakpoint_command_test"
      } else {
  	gdb_test "continue" \
! 	    "Breakpoint \[0-9\]*, factorial \\(value=5\\).*at.*
  \[0-9\]*\[ 	\]*if \\(value > 1\\) \{.*
  \[0-9\]*\[ 	\]*value \\*= factorial \\(value - 1\\);.*
  factorial \\(value=4\\) at.*\[0-9\]*\[ 	\]*if \\(value > 1\\) \{.*
--- 204,214 ----
  
      if { [istarget "hppa*-hp-hpux*"] } {
  	gdb_test "continue" \
! 		"Continuing.*.*.*Breakpoint \[0-9\]*, factorial \\(value=5\\).*at.*\[0-9\]*\[      \]*if \\(value > 1\\) \{.*\[0-9\]*\[      \]*value \\*= factorial \\(value - 1\\);.*" \
! 		"continue in infrun_breakpoint_command_test"
      } else {
  	gdb_test "continue" \
! 		"Breakpoint \[0-9\]*, factorial \\(value=5\\).*at.*
  \[0-9\]*\[ 	\]*if \\(value > 1\\) \{.*
  \[0-9\]*\[ 	\]*value \\*= factorial \\(value - 1\\);.*
  factorial \\(value=4\\) at.*\[0-9\]*\[ 	\]*if \\(value > 1\\) \{.*
*************** factorial \\(value=3\\) at .*
*** 200,206 ****
  #2  \[0-9a-fx\]* in factorial \\(value=5\\).*
  #3  \[0-9a-fx\]* in factorial \\(value=6\\).*
  #4  \[0-9a-fx\]* in main \\(.*\\).*" \
! 	    "continue in infrun_breakpoint_command_test";
      }
  
      gdb_stop_suppressing_tests;
--- 220,226 ----
  #2  \[0-9a-fx\]* in factorial \\(value=5\\).*
  #3  \[0-9a-fx\]* in factorial \\(value=6\\).*
  #4  \[0-9a-fx\]* in main \\(.*\\).*" \
! 		"continue in infrun_breakpoint_command_test";
      }
  
      gdb_stop_suppressing_tests;
*************** proc breakpoint_command_test {} {
*** 214,227 ****
  
      gdb_test "set args 6" "" "set args in breakpoint_command_test"
      if { ![runto factorial] } then { gdb_suppress_tests; }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      gdb_test "p value=6" "" "set value to 6 in progvar_simple_if_test #2"
      delete_breakpoints
      gdb_test "break factorial" "Breakpoint.*at.*" "break factorial #2"
      gdb_test "commands\nprintf \"Now the value is %d\\n\", value\nend" \
  	"End with.*" "commands in breakpoint_command_test"
!     gdb_test "continue" "Breakpoint \[0-9\]*, factorial.*Now the value is 5" \
  	"continue in breakpoint_command_test"
      gdb_test "print value" " = 5" "print value in breakpoint_command_test"
      gdb_stop_suppressing_tests;
--- 234,248 ----
  
      gdb_test "set args 6" "" "set args in breakpoint_command_test"
      if { ![runto factorial] } then { gdb_suppress_tests; }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      gdb_test "p value=6" "" "set value to 6 in progvar_simple_if_test #2"
      delete_breakpoints
      gdb_test "break factorial" "Breakpoint.*at.*" "break factorial #2"
      gdb_test "commands\nprintf \"Now the value is %d\\n\", value\nend" \
  	"End with.*" "commands in breakpoint_command_test"
!     gdb_test "continue" \
! 	    "Breakpoint \[0-9\]*, factorial.*Now the value is 5" \
  	"continue in breakpoint_command_test"
      gdb_test "print value" " = 5" "print value in breakpoint_command_test"
      gdb_stop_suppressing_tests;
*************** proc user_defined_command_test {} {
*** 243,252 ****
  	}
      }
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while \$arg0 > 0\nset \$arg0 -= 1\nif \(\$arg0 % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend\nend" "" "enter commands in user_defined_command_test"
! 
!     gdb_test "mycommand \$foo" "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" "execute user defined command in user_defined_command_test"
!    gdb_test "show user mycommand" "while.*set.*if.*p/x.*else.*p/x.*end.*" "display user command in user_defined_command_test"
  }
  
  proc watchpoint_command_test {} {
--- 264,279 ----
  	}
      }
      # This test should alternate between 0xdeadbeef and 0xfeedface two times.
!     gdb_test "while \$arg0 > 0\nset \$arg0 -= 1\nif \(\$arg0 % 2\) == 1\np/x 0xdeadbeef\nelse\np/x 0xfeedface\nend\nend\nend" \
! 	    "" \
! 	    "enter commands in user_defined_command_test"
! 
!     gdb_test "mycommand \$foo" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface\[^\n\]*\n\\\$\[0-9\]* = 0xdeadbeef\[^\n\]*\n\\\$\[0-9\]* = 0xfeedface" \
! 	    "execute user defined command in user_defined_command_test"
!    gdb_test "show user mycommand" \
! 	   "while.*set.*if.*p/x.*else.*p/x.*end.*" \
! 	   "display user command in user_defined_command_test"
  }
  
  proc watchpoint_command_test {} {
*************** proc watchpoint_command_test {} {
*** 290,330 ****
  
      send_gdb "commands $wp_id\n"
      gdb_expect {
!       -re "Type commands for when breakpoint $wp_id is hit, one per line.*>"\
!               {pass "begin commands on watch"}
!       -re "$gdb_prompt $"\
!               {fail "begin commands on watch"}
!       timeout {fail "(timeout) begin commands on watch"}
      }
      send_gdb "print value\n"
      gdb_expect {
!       -re ">"\
!               {pass "add print command to watch"}
!       -re "$gdb_prompt $"\
!               {fail "add print command to watch"}
!       timeout {fail "(timeout) add print command to watch"}
      }
      send_gdb "continue\n"
      gdb_expect {
!       -re ">"\
!               {pass "add continue command to watch"}
!       -re "$gdb_prompt $"\
!               {fail "add continue command to watch"}
!       timeout {fail "(timeout) add continue command to watch"}
      }
      send_gdb "end\n"
      gdb_expect {
!       -re "$gdb_prompt $"\
!               {pass "begin commands on watch"}
!       timeout {fail "(timeout) begin commands on watch"}
      }
      send_gdb "continue\n"
      gdb_expect {
!       -re "Continuing.*\[Ww\]atchpoint $wp_id deleted because the program has left the block in.*which its expression is valid.*run.c:57.*$gdb_prompt $"\
!               {pass "continue with watch"}
!       -re "$gdb_prompt $"\
!               {fail "continue with watch"}
!       timeout {fail "(timeout) continue with watch"}
      }
  }
  
--- 317,352 ----
  
      send_gdb "commands $wp_id\n"
      gdb_expect {
!       -re "Type commands for when breakpoint $wp_id is hit, one per line.*>" {
! 	  pass "begin commands on watch"
!       }
!       -re "$gdb_prompt $" {fail "begin commands on watch"}
!       timeout             {fail "(timeout) begin commands on watch"}
      }
      send_gdb "print value\n"
      gdb_expect {
! 	-re ">"               {pass "add print command to watch"}
! 	-re "$gdb_prompt $"   {fail "add print command to watch"}
! 	timeout               {fail "(timeout) add print command to watch"}
      }
      send_gdb "continue\n"
      gdb_expect {
! 	-re ">"               {pass "add continue command to watch"}
! 	-re "$gdb_prompt $"   {fail "add continue command to watch"}
! 	timeout               {fail "(timeout) add continue command to watch"}
      }
      send_gdb "end\n"
      gdb_expect {
! 	-re "$gdb_prompt $"   {pass "begin commands on watch"}
! 	timeout               {fail "(timeout) begin commands on watch"}
      }
      send_gdb "continue\n"
      gdb_expect {
! 	-re "Continuing.*\[Ww\]atchpoint $wp_id deleted because the program has left the block in.*which its expression is valid.*run.c:57.*$gdb_prompt $" {
! 	    pass "continue with watch"
! 	}
! 	-re "$gdb_prompt $"   {fail "continue with watch"}
! 	timeout               {fail "(timeout) continue with watch"}
      }
  }
  
*************** proc test_command_prompt_position {} {
*** 337,375 ****
      }
  
      if { ![runto factorial] } then { gdb_suppress_tests; }
!     # Don't depend upon argument passing, since most simulators don't currently
!     # support it.  Bash value variable to be what we want.
      delete_breakpoints
      gdb_test "break factorial" "Breakpoint.*at.*" "break factorial #2"
      gdb_test "p value=5" "" "set value to 5 in test_command_prompt_position"
      # All this test should do is print 0xdeadbeef once.
!     gdb_test "if value == 1\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" "\\\$\[0-9\]* = 0xdeadbeef" "if test in test_command_prompt_position"
! 
! # Now let's test for the correct position of the '>' in gdb's prompt for commands.
! # It should be at the beginning of the line, and not after one space.
  
      send_gdb "commands\n"
      gdb_expect {
! 	    -re "Type commands.*End with.*\[\r\n\]>$" \
!                 { send_gdb "printf \"Now the value is %d\\n\", value\n"
! 	          gdb_expect {
!                           -re "^printf.*value\r\n>$" \
! 			       { send_gdb "end\n"
!                                  gdb_expect {
! 			               -re "^end\r\n$gdb_prompt $" { pass "> OK in test_command_prompt_position" }
! 				       -re ".*$gdb_prompt $" { fail "some other message in test_command_prompt_position" }
! 				       timeout  { fail "(timeout) 1 in test_command_prompt_position" }
! 				 }
! 			       }
! 			   -re "^ >$" { fail "> not OK in test_command_prompt_position" }
! 			   -re ".*$gdb_prompt $"   { fail "wrong message in test_command_prompt_position" }
! 			   timeout    { fail "(timeout) 2 in test_command_prompt_position " }
! 		       }
! 		 }
! 		 -re "Type commands.*End with.*\[\r\n\] >$" { fail "prompt not OK in test_command_prompt_position" }
! 	   -re ".*$gdb_prompt $" { fail "commands in test_command_prompt_position" }
! 	   timeout { fail "(timeout) 3 commands in test_command_prompt_position" }
  	}
  
      gdb_stop_suppressing_tests;
  }
--- 359,414 ----
      }
  
      if { ![runto factorial] } then { gdb_suppress_tests; }
!     # Don't depend upon argument passing, since most simulators don't
!     # currently support it.  Bash value variable to be what we want.
      delete_breakpoints
      gdb_test "break factorial" "Breakpoint.*at.*" "break factorial #2"
      gdb_test "p value=5" "" "set value to 5 in test_command_prompt_position"
      # All this test should do is print 0xdeadbeef once.
!     gdb_test "if value == 1\np/x 0xfeedface\nelse\np/x 0xdeadbeef\nend" \
! 	    "\\\$\[0-9\]* = 0xdeadbeef" \
! 	    "if test in test_command_prompt_position"
!     
!     # Now let's test for the correct position of the '>' in gdb's
!     # prompt for commands.  It should be at the beginning of the line,
!     # and not after one space.
  
      send_gdb "commands\n"
      gdb_expect {
! 	-re "Type commands.*End with.*\[\r\n\]>$" { 
! 	    send_gdb "printf \"Now the value is %d\\n\", value\n"
! 	    gdb_expect {
! 		-re "^printf.*value\r\n>$" {
! 		    send_gdb "end\n"
! 		    gdb_expect {
! 			-re "^end\r\n$gdb_prompt $" { 
! 			    pass "> OK in test_command_prompt_position" 
! 			}
! 			-re ".*$gdb_prompt $" { 
! 			    fail "some other message in test_command_prompt_position" 
! 			}
! 			timeout  { 
! 			    fail "(timeout) 1 in test_command_prompt_position"
! 			}
! 		    }
! 		}
! 		-re "^ >$" { fail "> not OK in test_command_prompt_position" }
! 		-re ".*$gdb_prompt $"   { 
! 		    fail "wrong message in test_command_prompt_position" 
! 		}
! 		timeout    { 
! 		    fail "(timeout) 2 in test_command_prompt_position " 
! 		}
! 	    }
! 	}
! 	-re "Type commands.*End with.*\[\r\n\] >$" { 
! 	    fail "prompt not OK in test_command_prompt_position" 
! 	}
! 	-re ".*$gdb_prompt $" { 
! 	    fail "commands in test_command_prompt_position" 
  	}
+ 	timeout { fail "(timeout) 3 commands in test_command_prompt_position" }
+     }
  
      gdb_stop_suppressing_tests;
  }
*************** proc test_command_prompt_position {} {
*** 377,403 ****
  
  
  proc deprecated_command_test {} {
-     
-     
      gdb_test "maintenance deprecate blah" "Can't find command.*" \
            "tried to deprecate non-existsing command"
  
      gdb_test "maintenance deprecate p \"new_p\"" ""
!     gdb_test "p 5" "Warning: 'p', an alias for the command 'print' is deprecated.*Use 'new_p'.*" "p deprecated warning, with replacement"
      gdb_test "p 5" ".\[0-9\]* = 5.*" "Deprecated warning goes away"
  
      gdb_test "maintenance deprecate p \"new_p\"" ""
      gdb_test "maintenance deprecate print \"new_print\"" ""
!     gdb_test "p 5" "Warning: command 'print' \\(p\\) is deprecated.*Use 'new_print'.*" "both alias and command are deprecated"
      gdb_test "p 5" ".\[0-9\]* = 5.*" "Deprecated warning goes away"
- 
-     gdb_test "maintenance deprecate set remote memory-read-packet-size \"srm\" " "" "deprecate long comamnd"
-     gdb_test "set remote memory-read-packet-size" "Warning: command 'set remote memory-read-packet-size' is deprecated.*Use 'srm'.*" "long command deprecated"
- 
-     gdb_test "maintenance deprecate set remote memory-read-packet-size" "" "deprecate long comamnd"
-     gdb_test "set remote memory-read-packet-size" "Warning: command 'set remote memory-read-packet-size' is deprecated.*No alternative known.*" "long command deprecated with no alternative."
  
!     gdb_test "maintenance deprecate" "\"maintenance deprecate\".*" "deprecate with no arguments"
  }
  
  
--- 416,454 ----
  
  
  proc deprecated_command_test {} {
      gdb_test "maintenance deprecate blah" "Can't find command.*" \
            "tried to deprecate non-existsing command"
  
      gdb_test "maintenance deprecate p \"new_p\"" ""
!     gdb_test "p 5" \
! 	    "Warning: 'p', an alias for the command 'print' is deprecated.*Use 'new_p'.*" \
! 	    "p deprecated warning, with replacement"
      gdb_test "p 5" ".\[0-9\]* = 5.*" "Deprecated warning goes away"
  
      gdb_test "maintenance deprecate p \"new_p\"" ""
      gdb_test "maintenance deprecate print \"new_print\"" ""
!     gdb_test "p 5" \
! 	    "Warning: command 'print' \\(p\\) is deprecated.*Use 'new_print'.*" \
! 	    "both alias and command are deprecated"
      gdb_test "p 5" ".\[0-9\]* = 5.*" "Deprecated warning goes away"
  
!     gdb_test "maintenance deprecate set remote memory-read-packet-size \"srm\" " \
! 	    "" \
! 	    "deprecate long comamnd"
!     gdb_test "set remote memory-read-packet-size" \
! 	    "Warning: command 'set remote memory-read-packet-size' is deprecated.*Use 'srm'.*" \
! 	    "long command deprecated"
! 
!     gdb_test "maintenance deprecate set remote memory-read-packet-size" \
! 	    "" \
! 	    "deprecate long comamnd"
!     gdb_test "set remote memory-read-packet-size" \
! 	    "Warning: command 'set remote memory-read-packet-size' is deprecated.*No alternative known.*" \
! 	    "long command deprecated with no alternative."
! 
!     gdb_test "maintenance deprecate" \
! 	    "\"maintenance deprecate\".*" \
! 	    "deprecate with no arguments"
  }
  
  
From ezannoni@cygnus.com Tue Oct 24 11:46:00 2000
From: Elena Zannoni <ezannoni@cygnus.com>
To: Daniel Berlin <dberlin@redhat.com>
Cc: Kevin Buettner <kevinb@cygnus.com>, gdb-patches@sources.redhat.com
Subject: Re: That dwarf2read patch i just submitted
Date: Tue, 24 Oct 2000 11:46:00 -0000
Message-id: <14837.55523.942503.523490@kwikemart.cygnus.com>
References: <m3zojvmz9j.fsf@dan2.cygnus.com> <1001023165317.ZM12542@ocotillo.lan> <m3aebvqyxy.fsf@dan2.cygnus.com> <1001023173236.ZM12624@ocotillo.lan> <m3d7grpa35.fsf@dan2.cygnus.com> <1001023235955.ZM13394@ocotillo.lan> <14837.54804.638957.639280@kwikemart.cygnus.com>
X-SW-Source: 2000-10/msg00173.html
Content-length: 338

Elena Zannoni writes:
 > 
 > 
 > The person has submitted a few other patches, always in that same
 > time period (spring of 99). I haven't find an assignment from the FSF.
 > 
 > Time to ask.
 > 
 > Elena
 > 


A mail to him bounced.  I am afraid we must think of plan B. Unless we
can get in touch with somebody at the company.

Elena


  parent reply	other threads:[~2000-10-24 11:34 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-10-23  7:02 Daniel Berlin
     [not found] ` <1001023165317.ZM12542@ocotillo.lan>
     [not found]   ` <m3aebvqyxy.fsf@dan2.cygnus.com>
2000-10-23 10:32     ` Kevin Buettner
     [not found]       ` <m3d7grpa35.fsf@dan2.cygnus.com>
     [not found]         ` <1001023235955.ZM13394@ocotillo.lan>
2000-10-24 11:34           ` Elena Zannoni [this message]
     [not found]       ` <39F60D7B.C901357F@apple.com>
     [not found]         ` <39F6D4BB.8431FC3C@cygnus.com>
2000-10-25  7:19           ` Daniel Berlin

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=14837.54804.638957.639280@kwikemart.cygnus.com \
    --to=ezannoni@cygnus.com \
    --cc=dberlin@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=kevinb@cygnus.com \
    /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