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

Daniel Borca INVALID.NOREPLY at gnu.org
Sat May 5 11:43:26 UTC 2007


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


--- Pavel Tsekov <INVALID.NOREPLY at gnu.org> wrote:

> 
> Follow-up Comment #13, patch #5893 (project mc):
> 
> Regarding the off-by-one issue - indeed you are right.
> In my local copy I've changed "diff" to "gdiff" without
> changing 14 to 15.

Which only proves my point.  In an earlier comment I wrote regarding to glib:
"There are probably more catchy things that right now that I am afraid of
(like how trailing-null is counted or not in some string routines etc)." 
q.e.d.  I coded long enough now to feel and fear these things a priori.

> Regarding glib - It struck me that your patch was the most
> complete patch for MC I have seen in quite a while.

I can imagine.

> Still you have the same attitude as most of the other
> patch authors - that is unwillingness to abide to certain
> simple rules. Yes I could alter your patch as I like but

You're asking me to learn a whole new hairy API (that, among others, I
personally find useless) just because I chose to share some piece of code
with the community?  Perhaps my code is not perfect, but glibbing it won't
make it any more perfect.

> after all this is your patch. There are many projects out
> there that have much stricter rules for accepting patches
> than those of MC.

I don't care about "many projects out there".  Nobody cares about all and
every project out there. 

> The fact that you do not want to do your homework and

This is a tendentious statement.  I said only that I'm not
the guy to do glib modifications, that's all.  But I *will* and *do* support
my patch.  For example, I just fixed the colorscheme (new set of patches
attached).  I will find some time soon to replace open/read/close/etc with
their mc_ counterparts.  I have to give them a bit of thinking first.

> support your patch doesn't really makes me more willing
> to review it. I am certainly busy as you are but I try
> to do my best to review the patches that are posted here.

You're not doing it for me, you know.  You're doing it for the
users.  You alone chose this task.  Which, btw, I think it is a praiseworthy
fact; and I appreciate your work.

> That said I really like the functionality that your patch
> provides and I'd really like to have it in MC.

Good.  Let's then end this and concentrate on what's important.  Later, we
can code a quick and dirty sed 's/malloc/g_try_malloc/g' script.  It is
really a secondary issue as it shouldn't change the semantics (and hence the
correctness) of the submitted code.

Thank you for your work and for understanding.

(file #12701, file #12702)
    _______________________________________________________

Additional Item Attachment:

File name: mc-4.6.1.ydiff.patch.gz        Size:16 KB
File name: mc-4.6.1_20070309.ydiff.patch.gz Size:16 KB


    _______________________________________________________

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