Long delta chains edge case revived

From fs_fs.c:choose_delta_base():

    svn_boolean_t maybe_shared_rep = FALSE;
    if (!props && base->data_rep && svn_fs_fs__id_rev(base->id) > base->data_rep->revision)
      maybe_shared_rep = TRUE;

The third conjunct was intended to answer the question "Is
base->data_rep a shared rep", but that answer is no longer complete: the
condition will false-negative in the presence of intra-revision shared
reps (i.e., two node-revs created in the same revision that share a rep
between them).
Received on 2013-06-10 14:01:37 CEST