[patch #5893] integrated side-by-side textmode diff viewer
Pavel Tsekov
INVALID.NOREPLY at gnu.org
Sat May 5 12:47:08 UTC 2007
Follow-up Comment #15, patch #5893 (project mc):
The fact that you feel that a certain API is useless doesn't make it go away.
MC uses glib heavily and that is a fact. If you feel that this is wrong -
start a new thread on the mailing list and share your thoughts. As I said
below - we don't want to discuss the usefullness of glib here - this
discussion is about your patch. For your patch to be consistent with the rest
of the code it has to be converted to use glib calls where appropriate.
I am not asking you to learn a whole new API - I am asking you to send a
proper patch. You obviously spent some time on MC code - I doubt it that you
missed the fact that it used glib.
The falsely reported off-by-one bug reported by me has nothing to do with
glib. It was my mistake - since I build and test on Solaris where diff is not
GNU diff, I quickly changed diff to gdiff without updating the constant which
represented the size of the format string.
_______________________________________________________
Reply to this item at:
<http://savannah.gnu.org/patch/?5893>
_______________________________________________
Message sent via/by Savannah
http://savannah.gnu.org/
More information about the mc-devel
mailing list