On Mon, May 1, 2017 at 7:59 PM, Junio C Hamano <gitster@xxxxxxxxx> wrote: > Samuel Lijin <sxlijin@xxxxxxxxx> writes: > >> On Fri, Mar 31, 2017 at 10:52 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote: >> >>> It might not be a bad idea to teach "blame" not to pay attention to >>> any path that is marked as "-diff" (e.g. binary files) when trying >>> to see if remaining contents appeared by borrowing from them. We do >>> not have that heuristics (yet). >> >> Could you elaborate on this? Do you mean to tell diffcore-rename to >> ignore diff_filespec objects if they're binary? > > No and yes ;-). I do not think it is a good idea to unconditionally > ignore binary in diffcore-rename. > > But when we know that the rename detection is called from inside > blame.c, where by definition we would be digging line-oriented > contents, there is no point checking if the contents we are looking > for came from an existing binary file. A followup thought: perhaps it would make sense for blame.c rename detection to only consider files with the same extension?