Re: ansi-color.el and comint.el

[Kevin Rodgers]

Stephan Stahl wrote:
> Hi.
>
> ansi-color.el may overwrite comint-output-filter-functions if it is
> loaded before comint.el.  That is because ansi-color.el does a simple
>
> (add-hook 'comint-output-filter-functions
>      'ansi-color-process-output)
>
> Now comint-output-filter-functions is defined and when later comint.el
> is loaded its own
>
> (defvar comint-output-filter-functions 
'(comint-postoutput-scroll-to-bottom
> comint-watch-for-password-prompt)
>
> does nothing so those two functions may be missing.  I'm not sure how
> this is fixed best.  Either have comint.el do:
>
> (defvar comint-output-filter-functions nil)
>
> (add-hook 'comint-output-filter-functions
>      'comint-watch-for-password-prompt)
> (add-hook 'comint-output-filter-functions
>      'comint-postoutput-scroll-to-bottom)
>
> or have ansi-color.el require comint.  If anyone else thinks this
> should be fixed :)

Would it work if ansi-color-for-comint-mode-on added
ansi-color-process-output to comint-output-filter-functions?  If so,
should ansi-color-for-comint-mode-off remove it?

--
Kevin Rodgers