On Fri, Apr 15, 2016 at 4:05 PM, Jacob Keller <jacob.keller@xxxxxxxxx> wrote: > There's a few places that will need cleaning up (comments and such) > that mention empty line still, but that's not surprising. I am going > to test this for a bit on my local repos, and see if it makes any > difference to the old heuristic as well. > > Thanks, > Jake > I ran this heuristic on git.git and it produces tons of false positive transforms which are much lease readable (to me at least), far more than those produced by the newline/blank link heuristic did. I think we should stick with the empty line heuristic instead of this version, even if it's easier to implement this version. We still would need to figure out how to handle CRLF properly but it's worth resolving that than this heuristic is. Thanks, Jake -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html