From: Andrew Cagney Date: Thu, 8 Mar 2001 15:13:00 +0000 (+0000) Subject: * TODO: Revise 5.1 list. Delete PARAMS task. Add coding standard X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=a50c34dcb969c7b2a5976ae02842c98b69acdf5d;p=binutils-gdb.git * TODO: Revise 5.1 list. Delete PARAMS task. Add coding standard review. Clarify GNU/LINUX/sparc. Move other tasks to 5.2 or general. --- diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 264be2b18c8..05eeb4d1b1a 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,9 @@ +2001-03-08 Andrew Cagney + + * TODO: Revise 5.1 list. Delete PARAMS task. Add coding standard + review. Clarify GNU/LINUX/sparc. Move other tasks to 5.2 or + general. + 2001-03-07 Andrew Cagney * TODO (GDB 5.1 Known Problems): Document problem of building GDB diff --git a/gdb/TODO b/gdb/TODO index 28bbbe986c5..1f9a77f09a2 100644 --- a/gdb/TODO +++ b/gdb/TODO @@ -10,13 +10,39 @@ find out whether anyone else is working on it. Below is a list of problems identified during the GDB 5.0 release cycle. People hope to have these problems fixed in 5.1. +-- 2001-03-08 + +Update GDB's coding standard documentation. Known topics: + + o alloca/malloc et.al. + + o typedef and structs + + o ISO-C + +and most likely also: + + o include conventions + -- -RFD: infrun.c: No bpstat_stop_status call after proceed over break? -http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00665.html +Wow, three bug reports for the same problem in one day! We should +probably make fixing this a real priority :-). -GDB misses watchpoint triggers after proceeding over a breakpoint on -x86 targets. +Anyway, thanks for reporting. + +The following patch will fix the problems with setting breakpoints in +dynamically loaded objects: + + http://sourceware.cygnus.com/ml/gdb-patches/2000-05/msg00230.html + +This patch isn't checked in yet (ping Michael/JimB), but I hope this +will be in the next GDB release. + +There should really be a test in the testsuite for this problem, since +it keeps coming up :-(. Any volunteers? + +Mark -- @@ -29,58 +55,30 @@ http://sourceware.cygnus.com/ml/gdb-patches/2000-05/msg00309.html Mark --- - -Can't build IRIX -> arm GDB. -http://sourceware.cygnus.com/ml/gdb-patches/2000-04/msg00356.html - -David Whedon writes: -> Now I'm building for an embedded arm target. If there is a way of turning -> remote-rdi off, I couldn't find it. It looks like it gets built by default -> in gdb/configure.tgt(line 58) Anyway, the build dies in -> gdb/rdi-share/unixcomm.c. SERPORT1 et. al. never get defined because we -> aren't one of the architectures supported. +[The test has been submitted for approval - cagney] -- -Problem with weak functions -http://sourceware.cygnus.com/ml/gdb/2000-05/msg00060.html +RFD: infrun.c: No bpstat_stop_status call after proceed over break? +http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00665.html -Dan Nicolaescu writes: -> It seems that gdb-4.95.1 does not display correctly the function when -> stoping in weak functions. -> -> It stops in a function that is defined as weak, not in the function -> that is actually run... +GDB misses watchpoint triggers after proceeding over a breakpoint on +x86 targets. -- GDB 5.0 doesn't work on Linux/SPARC --- - -Thread support. Right now, as soon as a thread finishes and exits, -you're hosed. This problem is reported once a week or so. - --- - -Wow, three bug reports for the same problem in one day! We should -probably make fixing this a real priority :-). - -Anyway, thanks for reporting. - -The following patch will fix the problems with setting breakpoints in -dynamically loaded objects: +There are two parts to this. - http://sourceware.cygnus.com/ml/gdb-patches/2000-05/msg00230.html + o GDB 5.0 doesn't work on GNU/Linux/SPARC32 -This patch isn't checked in yet (ping Michael/JimB), but I hope this -will be in the next GDB release. + o GDB 5.0 doesn't work on the new target + GNU/Linux/SPARC64 -There should really be a test in the testsuite for this problem, since -it keeps coming up :-(. Any volunteers? +GDB does build on both these targets. -Mark +The first problem is the one that should be fixed. -- @@ -93,6 +91,21 @@ The following new features should be included in 5.1. Enable MI by default. Old code can be deleted after 5.1 is out. +Issues: + + o syntax change where a list would + look like: + [ foo=a, foo=b, foo=c ] + instead of + { foo=a, foo=b, foo=c } + + o kill off the idea of a reverse + query. + + o review test cases + + o enable it + -- Pascal (Pierre Muller, David Taylor) @@ -106,6 +119,8 @@ http://sourceware.cygnus.com/ml/gdb/2000-q1/msg00521.html Indent -gnu ? http://sourceware.cygnus.com/ml/gdb/2000-q1/msg00496.html +[I think this has been merged, need to confirm - cagney] + -- Java (Anthony Green, David Taylor) @@ -123,24 +138,12 @@ http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00515.html Patch: handle N_MAIN stab http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00527.html --- +-- 2001-03-08 -[Comming...] +Add CRIS target. -Modify gdb to work correctly with Pascal. - --- - -Revised UDP support (was: Re: [Fwd: [patch] UDP transport support]) -http://sourceware.cygnus.com/ml/gdb-patches/2000-04/msg00000.html - -(Broken) support for GDB's remote protocol across UDP is to be -included in the follow-on release. - -It should be noted that UDP can only work when the [Gg] packet fits in -a single UDP packet. - -There is also much debate over the merit of this. +A predicate to this is the multi-arching of SOFTWARE_SINGLE_STEP(). A +patch has been submitted. -- @@ -149,6 +152,11 @@ There is also much debate over the merit of this. The following code cleanups will hopefully be applied to GDB 5.1. +-- 2001-03-08 + +Resolve the status of all broken targets as identified by the MAINTAINERS +file. + -- Change documentation to GFDL license. @@ -164,12 +172,6 @@ http://www.gnu.org/copyleft/'' -- -Delete macro TARGET_BYTE_ORDER_SELECTABLE. - -Patches in the database. - --- - Fix copyright notices. Turns out that ``1998-2000'' isn't considered valid :-( @@ -178,42 +180,8 @@ http://sourceware.cygnus.com/ml/gdb-patches/2000-05/msg00467.html -- -Purge PARAMS. - -Eliminate all uses of PARAMS in GDB's source code. - --- - -printcmd.c (print_address_numeric): - -NOTE: This assumes that the significant address information is kept in -the least significant bits of ADDR - the upper bits were either zero -or sign extended. Should ADDRESS_TO_POINTER() or some -ADDRESS_TO_PRINTABLE() be used to do the conversion? - --- - -Compiler warnings. - -Eliminate all warnings for at least one host/target for the flags: --Wimplicit -Wreturn-type -Wcomment -Wtrigraphs -Wformat -Wparentheses --Wpointer-arith -Wuninitialized - --- - -Follow through `make check' with --enable-shared. - -When the srcware tree is configured with --enable-shared, the `expect' -program won't run properly. Jim Wilson found out gdb has a local hack -to set LD_LIBRARY_PATH, but, AFAIK, no other project has been hacked -similarly. - -http://sourceware.cygnus.com/ml/gdb/2000-q1/msg00845.html - --- - - GDB 5.1 Known Problems - ====================== + GDB 5.1 - Known Problems + ======================== -- @@ -245,7 +213,8 @@ will still occure. sid/component/configure.in mis-configures -- -Fix at least one thread bug. +Thread support. Right now, as soon as a thread finishes and exits, +you're hosed. This problem is reported once a week or so. -- @@ -275,7 +244,11 @@ Makefile.in and configure.in. -- -Eliminate one more compiler warnings. +Compiler warnings. + +Eliminate all warnings for at least one host/target for the flags: +-Wimplicit -Wreturn-type -Wcomment -Wtrigraphs -Wformat -Wparentheses +-Wpointer-arith -Wuninitialized -- @@ -291,6 +264,11 @@ See also sub-directory configure below. The current convention is (kind of) to use $(
_h) in all dependency lists. It isn't done in a consistent way. +-- + + GDB 5.2 - Known Problems + ======================== + -- Code Cleanups: General @@ -301,6 +279,69 @@ to any specific release. -- +Revised UDP support (was: Re: [Fwd: [patch] UDP transport support]) +http://sourceware.cygnus.com/ml/gdb-patches/2000-04/msg00000.html + +(Broken) support for GDB's remote protocol across UDP is to be +included in the follow-on release. + +It should be noted that UDP can only work when the [Gg] packet fits in +a single UDP packet. + +There is also much debate over the merit of this. + +-- + +Can't build IRIX -> arm GDB. +http://sourceware.cygnus.com/ml/gdb-patches/2000-04/msg00356.html + +David Whedon writes: +> Now I'm building for an embedded arm target. If there is a way of turning +> remote-rdi off, I couldn't find it. It looks like it gets built by default +> in gdb/configure.tgt(line 58) Anyway, the build dies in +> gdb/rdi-share/unixcomm.c. SERPORT1 et. al. never get defined because we +> aren't one of the architectures supported. + +-- + +Problem with weak functions +http://sourceware.cygnus.com/ml/gdb/2000-05/msg00060.html + +Dan Nicolaescu writes: +> It seems that gdb-4.95.1 does not display correctly the function when +> stoping in weak functions. +> +> It stops in a function that is defined as weak, not in the function +> that is actually run... + +-- + +Follow through `make check' with --enable-shared. + +When the srcware tree is configured with --enable-shared, the `expect' +program won't run properly. Jim Wilson found out gdb has a local hack +to set LD_LIBRARY_PATH, but, AFAIK, no other project has been hacked +similarly. + +http://sourceware.cygnus.com/ml/gdb/2000-q1/msg00845.html + +-- + +Delete macro TARGET_BYTE_ORDER_SELECTABLE. + +Patches in the database. + +-- + +printcmd.c (print_address_numeric): + +NOTE: This assumes that the significant address information is kept in +the least significant bits of ADDR - the upper bits were either zero +or sign extended. Should ADDRESS_TO_POINTER() or some +ADDRESS_TO_PRINTABLE() be used to do the conversion? + +-- + The BFD directory requires bug-fixed AUTOMAKE et.al. AUTOMAKE 1.4 incorrectly set the TEXINPUTS environment variable. It