* write.c (relax_segment <rs_space>): Calculate growth using
[deliverable/binutils-gdb.git] / gdb / TODO
index 5fb43cc5d7b176cc794cd8161e3bf1caefb26d17..6c83f79083c0e0550bb1dffcc3013d9f82811d88 100644 (file)
--- a/gdb/TODO
+++ b/gdb/TODO
@@ -26,6 +26,10 @@ and most likely also:
 
 --
 
+Obsolete ser-ocd.c.
+
+--
+
 Wow, three bug reports for the same problem in one day!  We should
 probably make fixing this a real priority :-).
 
@@ -152,7 +156,7 @@ patch has been submitted.
 
 The following code cleanups will hopefully be applied to GDB 5.1.
 
--- 2001-03-14
+-- 2001-03-26
 
 Resolve the build status of all broken targets as identified by the
 MAINTAINERS file.
@@ -160,62 +164,6 @@ MAINTAINERS file.
        o       arm-* vs NetBSD's lack of ``unix''
        o       arm-* vs IRIX (see below)
 
-       o       delete m88k?
-
--- 2001-03-15
-
-                       Obsolete some targets.
-
-Possible selection criteria are:
-
-       o       uses a deprecated feature
-
-       o       doesn't build
-
-       o       doesn't have a maintainer
-
-Steps:            
-         
-       o       post proposals to gdb@ (DONE)
-
-       o       post announcement to gdb-announce@
-               crossed with gdb@ reply-to to gdb@
-               (DONE)
-
-                       ns32k-*-mach3*
-                       ns32k-umax-*
-                       ns32k-utek-sysv*
-                       tic80-*
-                       m68*-isi-*
-                       m68*-sony-*
-                       a29k-ultra3
-                       powerpcle-*-solaris*
-                       powerpcle-*-cygwin*
-                       w65-*-*
-                       i[3456]86-*-sunos*
-
-       o       clobber the files:
-
-                       configure.{in,host,tgt}
-                       Makefile.in
-                       *-tdep.c *-nat.c *-xdep.c
-                       configure/*/*
-
-       o       update NEWS
-
---
-
-Change documentation to GFDL license.
-
-``It is time to make an effort to start using the GFDL more
-thoroughly.  Would all GNU maintainers please change the license to
-the GFDL, for all manuals and other major documentation files?
-
-The GFDL and some instructions for using it can be found in
-http://www.gnu.org/copyleft/''
-
-       RMS
-
 --
 
 Fix copyright notices.
@@ -238,6 +186,12 @@ was occuring in the opcodes directory.
 
 --
 
+m88k
+
+The m88k has suffered bit rot and is known to not build.
+
+--
+
 Solaris 8 x86 CURSES_H problem
 http://sources.redhat.com/ml/gdb/2000-07/msg00038.html
 
@@ -292,9 +246,84 @@ Makefile.in and configure.in.
 
 Compiler warnings.
 
-Eliminate all warnings for at least one host/target for the flags:
--Wimplicit -Wreturn-type -Wcomment -Wtrigraphs -Wformat -Wparentheses
--Wpointer-arith -Wuninitialized
+Eliminate warnings for all targets on at least one host for one of the
+-W flags.  Flags up for debate include: -Wswitch -Wcomment -trigraphs
+-Wtrigraphs -Wunused-function -Wunused-label -Wunused-variable
+-Wunused-value -Wchar-subscripts -Wtraditional -Wshadow -Wcast-qual
+-Wcast-align -Wwrite-strings -Wconversion -Wstrict-prototypes
+-Wmissing-prototypes -Wmissing-declarations -Wredundant-decls
+-Woverloaded-virtual -Winline
+
+--
+
+Deprecate, if not delete, the following:
+
+        register[]
+        register_valid[]
+        register_buffer()
+       REGISTER_BYTE()
+                Replaced by, on the target side
+                  supply_register()
+                and on core-gdb side:
+                  {read,write}_register_gen()
+               Remote.c will need to use something
+               other than REGISTER_BYTE() and
+               REGISTER_RAW_SIZE() when unpacking
+               [gG] packets.
+
+        STORE_PSEUDO_REGISTER
+        FETCH_PSEUDO_REGISTER
+                Now handed by the methods
+                  gdbarch_{read,write}_register()
+                which sits between core GDB and
+                the register cache.
+
+        REGISTER_CONVERTIBLE
+        REGISTER_CONVERT_TO_RAW
+        REGISTER_CONVERT_TO_VIRTUAL
+                I think these three are redundant.
+                gdbarch_register_{read,write} can
+                do any conversion it likes.
+
+        REGISTER_VIRTUAL_SIZE
+        MAX_REGISTER_VIRTUAL_SIZE
+        REGISTER_VIRTUAL_TYPE
+                I think these can be replaced by
+               the pair:
+                  FRAME_REGISTER_TYPE(frame, regnum)
+                  REGISTER_TYPE(regnum)
+
+       DO_REGISTERS_INFO
+               Replace with
+                FRAME_REGISTER_INFO (frame, ...)
+
+       REGISTER_SIM_REGNO()
+               If nothing else rename this so that
+               how it relates to rawreg and the
+               regnum is clear.
+
+       REGISTER_BYTES
+               The size of the cache can be computed
+               on the fly.
+
+       IS_TRAPPED_INTERNALVAR
+               The pseudo registers should eventually make
+               this redundant.
+
+--
+
+Obsolete the targets.
+
+arm*-wince-pe
+mips*-*-pe
+sh*-*-pe
+
+Obsolete the protocols:
+
+RDB
+
+``As of version 5.3, WindRiver has removed the RDB server (RDB
+protocol support is built into gdb).''  -- Till.
 
 --
 
@@ -325,16 +354,13 @@ 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.
+Investigate changing --target=a29k-amd-udi to a29k-*-coff* and
+rationalize *.mt files.  The got-ya is in remote-eb.c - it has its own
+custom tty manipulation - it should be using the serial object.
 
-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.
+Rename read_register{,_pid}() to read_unsigned_register{,_pid}().
 
 --
 
@@ -609,6 +635,11 @@ coverage analysis).
 Replace the file gdb/CONTRIBUTE with a file that is generated from the
 gdb/doc/*.texinfo directory.
 
+--
+
+Rewrite/break up sparcl-tdep.c so that it uses ser*.c as the mechanism
+for accessing either the serial or UDP port.
+
 --
 
                        New Features and Fixes
@@ -722,7 +753,106 @@ limited number of hardwired actions.
 
 --
 
-Get the TUI working on all platforms.
+Fix TUI
+
+     o  readline/*.h bitrot
+
+        The TUI isn't up-to-date with
+        respect to the readline currently
+        bundled with GDB.  Importing a
+        new readline is on the 5.1 wish
+        list so this can only get worse.
+
+        Grep for things like term_cursor_move.
+
+        (To be honest, I don't see anyone
+        importing a new readline before 5.1 is
+        out)
+
+     o  tui.c:va_catch_errors() bitrot
+
+        This nasty piece of work used knowledge
+        of the internals of GDBs error functions :-(
+        Ever since those internals were cleaned
+        up this code has been broken. :-(
+
+     o  tuiWin.c:c_makeVisibleWithNewHeight() broken
+        tuiLayout.c:_extractDisplayStartAddr() broken
+
+        Both these function call find_line_pc()
+        incorrectly (wrong args, wrong return value).
+
+        I suspect this bug has always been there!
+        It had been hidden because those files
+        didn't include the necessary header files
+        from gdb proper :-(
+
+     o  tuiRegs() host dependant
+
+        Not suprisingly, this isn't a very portable
+        section of code.  However, I'm sure people
+        could live with no regs in the short to
+        medium term.
+
+     o  defs.h: #include "tui.h" et.al.
+
+        I'm not sure where this came from.
+        It was a really bad idea.
+
+        To get things to compile I did a nasty
+        hack (Just declare what was needed and
+        replace any expressions like xx->y.z()
+        in GDB proper with function calls).  I
+        could commit it slightly cleaned up if
+        you like.
+
+        Medium Term. the #ifdef TUI and TuiDo()
+        should be changed to hooks (like GDBTK).
+        The gdb-events.[hc] is there for that
+        purpose (1)
+
+     o  tui.c:_tuiReset() host dependant
+
+        tui.c contains a lump of termio[s]
+        I suspect an equivalent block of
+        code can be lifted from readline.
+        An equivalent readline function may
+        even be available.
+
+     o  curses.h vs ncurses.h.
+
+        Simple portability problem.
+
+     o  subsetCompare()
+
+        This function is a mystery - where is it?
+
+     o  tui-file.[hc] cleanup
+
+        This can be significantly simplified.
+
+     o  The code should be pacified. (-Werror -W...)
+
+        There are plenty of #includes,
+        duplicate #includes, missing function decls
+        and the like.
+
+        Some of the problems I found were through
+        fixing a few of the warnings.
+
+     o  The code should be GNUtified.
+
+        It would be very nice to have this code
+        look like the rest of GDB.  That way people
+        would be more accepting of it as a true
+        gdb component.
+
+        Until it is GNUtified it is going to stick
+        out like a sore thumb to the programmer.
+
+     o  The code should be clearly copyrighted
+
+        (FSF, with due credit to HP)
 
 --
 
@@ -819,19 +949,6 @@ Robert Lipe writes:
 > practical matter, the current thread support is somewhat more annoying
 > than when GDB was thread-unaware.
 
---
-
-Migrate qfThreadInfo packet -> qThreadInfo. (Andrew Cagney)
-
-Add support for packet enable/disable commands with these thread
-packets.  General cleanup.
-
-[PATCH] Document the ThreadInfo remote protocol queries
-http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00832.html
-
-[PATCH] "info threads" queries for remote.c
-http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00831.html
-
 --
 
                        Language Support
@@ -873,6 +990,32 @@ Get DEC/Compaq to contribute their Modula-3 support.
 
 --
 
+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.
+
+--
+
+Migrate qfThreadInfo packet -> qThreadInfo. (Andrew Cagney)
+
+Add support for packet enable/disable commands with these thread
+packets.  General cleanup.
+
+[PATCH] Document the ThreadInfo remote protocol queries
+http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00832.html
+
+[PATCH] "info threads" queries for remote.c
+http://sourceware.cygnus.com/ml/gdb-patches/2000-q1/msg00831.html
+
+--
+
 Remote protocol doco feedback.
 
 Too much feedback to mention needs to be merged in (901660).  Search
@@ -899,6 +1042,32 @@ fixed so that it builds on 16 bit machines.
 
 --
 
+Fix the ``!'' packet.
+
+JT reported that the existing targets do, in fact return ``OK'' so it
+is possible to merge remote and extended-remote targets.
+
+--
+
+Drop ``<address>'' from the [SsCc] packets.
+
+I don't think that GDB generates them so having it in the protocol is
+silly.
+
+--
+
+Fix doco on the ``q'' packet.
+
+It has evolved into a generic RPC.  The notes should reflect this and,
+perhaps, the ``Q'' packet can be deprecated.
+
+The doco should mention that ``OK'' is a valid packet response.
+
+The doco should explain why ``OK'' needs to be a valid packet
+response.
+
+--
+
 Add the cycle step command.
 
 http://sourceware.cygnus.com/ml/gdb/2000-q1/msg00237.html
@@ -915,9 +1084,13 @@ on the fly - should it be returned in the ``T'' packet?
 Underlying problem is that the register file is target endian.  If the
 target endianess changes gdb doesn't know.
 
+Suggest encoding registers as NN!VALUE.
+
 --
 
-Rename read_register{,_pid}() to read_unsigned_register{,_pid}().
+GDB should allow incomming packets to be larger than outgoing ones.  A
+fully loaded T packet (containing all registers) can be very large -
+definitly larger than a corresponding Gg packet.
 
 --
 
@@ -1198,6 +1371,9 @@ The name is wrong for starters.  ``target_signal'' should probably be
 After that it needs to be multi-arched and made independent of any
 host signal numbering.
 
+Once this is done, the signal enum can probably be moved to
+include/gdb so that it is available to embedded stubs.
+
 --
 
 Update ALPHA so that it uses ``struct frame_extra_info'' instead of
This page took 0.02752 seconds and 4 git commands to generate.