[patch #5893] integrated side-by-side textmode diff viewer

[Pavel Tsekov <INVALID NOREPLY gnu org>]

Follow-up Comment #6, patch #5893 (project mc):

Why do you define the color pairs used to highlight the differences in
ydiff.c instead of color.h ? Why do you reuse color pairs when building
against ncurses but not slang ? If the user has redefined some of those color
pairs it would end up with unexpected colors in the diff viewer. I think it
would be better to designate a number of color pairs for use by the diff view
(as the rest of the code does) - this way it could be made user configurable.


    _______________________________________________________

Reply to this item at:

  <http://savannah.gnu.org/patch/?5893>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.gnu.org/