Re: [PATCH] mergetool merge/skip/abort
- From
Caleb Cushing <xenoterracide@gmail.com>
- Date
- Jan 24, 2009, 18:36 UTC
- Message-ID
- <81bfc67a0901241036v6ca30c24q54487e118fd67c1c@mail.gmail.com>
- In-Reply-To
- <7vwscmue5z.fsf@gitster.siamese.dyndns.org>
> I do not use mergetool myself so I generally do not pay attention to > patches on this tool, but I would want to pick up ones that people > involved in mergetool discussion can agree to be good patches.
yeah I can see that.
> There are a few mergetool updates in flight from various authors. How > does your submission compare with others' in both form/presentation and > clarity of logic (remember, I am not keeping track)?
to be honest, a quick search of the past 2 months of patches didn't show me any patches that do the same thing as mine, so I'm not sure that comparing one feature to a different feature is good. I did try to remain consistent and even improve consistency with existing UI, and use the same/similar logic to existing. I'm not keeping track either, just fixing my own problem.
-- Caleb Cushing http://xenoterracide.blogspot.com