Ulrich Weigand wrote: > Eli Zaretskii wrote: > > >> Fine with me. Note that the original code had a local variable that >> made this clear; this whole discussion was caused by a patch that >> removed that variable. >> > > Actually, the original code had a *global* static variable that was > used only at this single location -- cleaning this up was really the > point of the patch, I guess. > > Bye, > Ulrich > > The function "handle_inferior_event"'s length is >1450 (!!!) lines of code and I think its a good idea to clean it up if possible. The patch removes an unnecessary variable from that function. To increase the readability I added a comment to the patch. Ok? -- Markus Deuling GNU Toolchain for Linux on Cell BE deuling@de.ibm.com