On Mon, 2015-04-20 at 20:36 +0300, stepharo wrote:
> Jan
> 
> what is your fix? to show the end of line differently?

No, I changed MCMethodDefinition>>= to be more relaxed about
sources and treat two sources that differ only in leading/trailing
spaces as "same". This way, these methods do not occur in merge tool 
at all. 
This is just a quick fix - much better would be to compare AST's and
treat whitespace-changes specially (i.e., provide a filter to show/hide
whitespace-only-changes). 

> Do you think that we should integrate it?

Yes.

> Stef
> 
> Le 20/4/15 12:36, Jan Vrany a écrit :
> > Hi,
> >
> > I just wanted to merge some code in Monticello and the merge tool
> > marked all methods as conflict because their source differ in trailing
> > whitespace (newline). The diff panel on the right does not show any
> > difference.
> >
> > http://smalltalkhub.com/#!/~JanVrany/Misc/versions/Monticello-Fixes-JanVrany.1
> >
> > Here's a fix for this particular problem in case somebody runs into the
> > same problem.
> >
> > Best, Jan
> >
> >
> >
> 
> 



Reply via email to