Re: RFC: UI_OUT activation time

[Kevin Buettner <kevinb at cygnus dot com>]

On Nov 16,  2:37pm, Fernando Nasser wrote:

> Currently, the old version of the code (using the ?printf_* family of
> functions) is still there, on a "ifdef", but it is still the default.
> One has to explicitly build with UI_OUT defined to use the new code.

Yes, these ifdefs are evil.  They make the code hard to maintain and
double the amount of testing required in order to be sure that a
change is correct.  (This is one of the reasons that I haven't
finished making the suggested/required changes to the
TARGET_ADJUST_BREAKPOINT_ADDRESS patch that I submitted a while back.)

[...]
> I propose (and I will submit a patch) to make the ui_out code the default
> effective immediately, and the complete elimination of the old style code
> from the sources before 5.1.

I won't comment on your proposed schedule, but I think the quicker we
can remove the old code, the better.

Kevin