Eliminate an unnecessary check in the FSFS history code. Justification (copied from issue #2054): Let the path and revision we are considering be path@C, with a predecessor rev of X.  Let parent@B be the innermost copy destination applying to path@C.  From where we are in the code, we know that X < B, or the (copyroot_rev > commit_rev) test would have been false earlier on.  Let the copy history of parent@B be copysource@A.  Since path@B is an unmodified child of parent@B, it must be the same node-rev as a child of copysource@A, and therefore cannot have a crev which is greater than A.
Eliminate an unnecessary check in the FSFS history code.
Justification (copied from issue #2054): Let the path and revision we
are considering be path@C, with a predecessor rev of X.  Let parent@B
be the innermost copy destination applying to path@C.  From where we
are in the code, we know that X < B, or the (copyroot_rev >
commit_rev) test would have been false earlier on.  Let the copy
history of parent@B be copysource@A.  Since path@B is an unmodified
child of parent@B, it must be the same node-rev as a child of
copysource@A, and therefore cannot have a crev which is greater than
A.

(Definition: "the innermost copy destination applying to path@C" means
the most recent copy root of path@C and its parent directories,
preferring child copies to parent copies if multiple copy targets
along the path happened in the same rev.)

* subversion/libsvn_fs_fs/tree.c (history_prev): Don't check for
  src_rev >= commit_rev; it should never be false, and if it is false
  because something is wrong, missing the copy may be the more harmful
  choice.

jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion,jmckaskill/subversion