ReplicatedPG::fill_in_copy_get: remove extraneous if statement

This should leave the behavior unchanged.

Signed-off-by: Samuel Just <sam.just@inktank.com>
ReplicatedPG::fill_in_copy_get: fix omap loop conditions

cursor.omap_offet indicates the most recently recovered key, we continue
filling in at the smallest key k | k > cursor.omap_offset.  If the loop
as written terminates due to !(left > 0), iter points at the next key to
copy, rather than the last key copied, resulting in the next copy
operation skipping that key.

Now, iter, if valid, must point to the last key copied once the loop has
completed since we check left <= 0 prior to advancing iter.  We can
therefore use it to fill in cursor.omap_offset.

Signed-off-by: Samuel Just <sam.just@inktank.com>
Merge pull request #1372 from ceph/wip-7607

wip 7607

Reviewed-by: Sage Weil <sage@inktank.com>