This is the mail archive of the
insight@sources.redhat.com
mailing list for the Insight project.
Re: [RFA] debug cleanup
- To: "Martin M. Hunt" <hunt at redhat dot com>
- Subject: Re: [RFA] debug cleanup
- From: Fernando Nasser <fnasser at cygnus dot com>
- Date: Tue, 06 Feb 2001 17:47:50 -0500
- CC: insight at sources dot redhat dot com
- Organization: Red Hat , Inc. - Toronto
- References: <Pine.SUN.4.33.0102061417470.5291-100000@rtl.cygnus.com>
"Martin M. Hunt" wrote:
>
> Here's a cleanup patch to eliminate some of the junk in the debug window.
>
> All debug functions print the functions name, so {debug "srcwin::start"}
> would print "srcwin::start srcwin::start"
>
Yes, I have been removing this one by one, when I work with the code, but I never had the time to go around chasing them.
I believe in some ancient time the "debug" facility did not print the function name.
> "debug" is just for informational/tracing use. Errors and warning should
> use "dbug" so the debug window can display and filter accordingly.
>
THis I know for sure: dbug was added later (I remember someone telling me that it was added). So the debug use for errors is probably mostly from before that was added.
> The debug window is now called "Insight Debug" instead of GDBTK.
>
Good catch.
> 2001-02-06 Martin Hunt <hunt@redhat.com>
>
> * library/debugwin.itb: Change window name to "Insight Debug"
>
> * library/embeddedwin.ith: Fix debug messages.
> * library/locals.tcl: Ditto.
> * library/watch.tcl (postMenu): Ditto.
> * library/variables.tcl: Ditto.
> * library/regwin.itb: Ditto.
> * library/gdbwin.ith: Ditto.
> * library/srcbar.itcl: Ditto.
>
I will check this in tomorrow.
Thanks for the cleanup patch.
--
Fernando Nasser
Red Hat - Toronto E-Mail: fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario M4P 2C9