On Wed, 5 Feb 2003, Garance A Drosihn wrote:

> At 11:43 PM -0800 2/4/03, Doug Barton wrote:
> >On Tue, 4 Feb 2003, Garance A Drosihn wrote:
> >
> >  > At 10:36 PM -0800 2/3/03, Doug Barton wrote:
> >  > There's a section of mergemaster that starts out with the comment
> >>  "Do an absolute diff first to see if the files are actually different".
> >
> >That's an option that users can enable, it's not the default behavior.
> >
> >>  What I was saying is we should add the same -I argument to that
> >>  diff command,
> >
> >That's not going to happen... the whole point of strict comparison
> >mode is that ALL differences are displayed, no matter what. Since
> >users choose this option, it's reasonable not to change its behavior.
>
> Oh, I didn't realize that was an optional behavior, I thought
> that was the default code path.  That request is my mistake then,
> for skimming through the script a little too quickly.

Yeah, I had the feeling that you thought it was the default... I meant to
say something to that effect. I confess that the style isn't 100% readable
to people who haven't spent as much time as I have working on it... my
only defense is that when that script started, it was a whole lot simpler
than it is now. :)  Of course, there are some aspects of the style that I
actually like, so I guess I'm not totally blameless here.

Doug

-- 

    If it's moving, encrypt it. If it's not moving, encrypt
      it till it moves, then encrypt it some more.

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message

Reply via email to