Johannes Schindelin <Johannes.Schindelin@xxxxxx> writes: > Hi Max, > > On Sun, 29 Jun 2014, Max Kirillov wrote: > >> diff --git a/xdiff/xmerge.c b/xdiff/xmerge.c >> index 9e13b25..625198e 100644 >> --- a/xdiff/xmerge.c >> +++ b/xdiff/xmerge.c >> @@ -245,11 +245,11 @@ static int xdl_fill_merge_buffer(xdfenv_t *xe1, const char *name1, >> dest ? dest + size : NULL); >> /* Postimage from side #1 */ >> if (m->mode & 1) >> - size += xdl_recs_copy(xe1, m->i1, m->chg1, 1, >> + size += xdl_recs_copy(xe1, m->i1, m->chg1, (m->mode & 2), >> dest ? dest + size : NULL); >> /* Postimage from side #2 */ >> if (m->mode & 2) >> - size += xdl_recs_copy(xe2, m->i2, m->chg2, 1, >> + size += xdl_recs_copy(xe2, m->i2, m->chg2, 0, >> dest ? dest + size : NULL); >> } else >> continue; > > Makes sense to me, especially with the nice explanation in the commit > message. > Having said that, here is my ACK for the current revision of the patch > series ... Thanks, both. Queued. -- 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