Re: [PATCH 3/3] Diff overhaul, adding the other half...
From: Junio C Hamano <hidden>
Date: 2016-06-15 22:41:58
quoted
quoted
quoted
quoted
"LT" == Linus Torvalds [off-list ref] writes:
LT> Hmm.. It's not working well. Not only does it take a lot of
LT> CPU time...
LT> torvalds@ppc970:~/v2.6/linux> wc -c drivers/media/dvb/bt8xx/dst_ca.h drivers/usb/misc/emi62_fw_m.h
LT> 1591 drivers/media/dvb/bt8xx/dst_ca.h
LT> 795679 drivers/usb/misc/emi62_fw_m.h
LT> 797270 total
Yup. diffcore-rename bases the similarity index on the average
of src and dst files. Maybe the similarity check should also
say they should be comparable size, like this upfront before
even going into delta:
abs(src-dst) < avg(src,dst) * (1 - minscore/maxscore)