This is the mail archive of the
gdb-patches@sources.redhat.com
mailing list for the GDB project.
[maint] Doco known TUI todo items
- To: GDB Patches <gdb-patches at sourceware dot cygnus dot com>
- Subject: [maint] Doco known TUI todo items
- From: Andrew Cagney <ac131313 at cygnus dot com>
- Date: Mon, 19 Mar 2001 16:48:05 -0500
Just FYI,
Andrew
2001-03-19 Andrew Cagney <ac131313@redhat.com>
* TODO: List known problems with TUI.
Index: TODO
===================================================================
RCS file: /cvs/src/src/gdb/TODO,v
retrieving revision 1.66
diff -p -r1.66 TODO
*** TODO 2001/03/15 17:06:00 1.66
--- TODO 2001/03/19 21:44:24
*************** limited number of hardwired actions.
*** 722,728 ****
--
! Get the TUI working on all platforms.
--
--- 722,827 ----
--
! 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)
--