This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] [python] Prompt substitution


Pedro Alves <pedro@codesourcery.com> writes:


> I believe this to be a bit broken in the annotations (prefix and
> suffix != NULL) case.  If the observer changes the prompt, we won't print
> either the prefix or the suffix, but will print directly what the
> observer on the prompt itself only (get_prompt).

Hmm, interesting.  It passed the annotations tests :(

> I believe it was not intended for the hook to be able
> to override the prefix and the suffix as well.  Right?
> I'm fixing this along with the get-rid-of-prompt-stack
> change.

Nope, not intended to do anything at all with prefix/suffix. AFAIK
prefix/suffix is only used in one particular case of annotations that
matter (basically the control characters that are printed in the
prefix/suffix when annotations > 1).

Anyway, thanks for taking care of this.  And thanks for getting rid of
the crufty prompt-stack too!

Cheers

Phil


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]