"Elijah Newren via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > @@ -578,8 +624,7 @@ void diffcore_rename(struct diff_options *options) > struct diff_filespec *one = rename_src[j].p->one; > struct diff_score this_src; > > - if (one->rename_used && !want_copies) > - continue; > + assert(!one->rename_used || want_copies || break_idx); Doesn't assert becomes a no-op in production builds? Shouldn't this be a BUG() instead?
