2010/2/7 Junio C Hamano <gitster@xxxxxxxxx>: > First, is this a condition that we want to change the behaviour to > "succeed" later? Yes, assuming it is possible to fix. > Imagine that the gap between abc and def block in your example is much > larger to exceed the number of pre-context lines of your second patch > (usually 3), and imagine you are the "git apply --whitespace=fix" program > you have updated to "fix" the preceived problem. You know you earlier > might have stripped some blank lines at the EOF, but there is nothing that > tells you if you had only 3 blank lines, or you had even more. How many > blank lines will you be adding back? My original idea was to add back exactly the number of lines needed so that the context lines would match. That can be calculated from the line numbers of the last line of the pre-image and the line number in the chunk and by scanning the chunk for blank context lines (both at the beginning and end of chunk). Since the blanks lines at the end will be stripped away anyway, I doesn't matter if I add back fewer lines than were there originally. Thinking a little more about it, if there is a chunk that starts beyond the end of file, I could add the number of blanks lines that is missing up to the beginning of the chunk plus the number of lines in the chunk itself. That will also take care of the case that the chunk deletes blanks lines. (That will probably add too many blanks at the end, but they will be stripped out again.) That is my plan. Of course, since I have not attempted to implement it yet, there could be fatal flaws in it. Do you see any fatal flaws that I don't see? -- Björn Gustavsson, Erlang/OTP, Ericsson AB -- 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