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

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:


  Message sent via/by Savannah

[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]