Hallo Michael,
Patch v3 applies cleanly, code compiles and make check is ok, but the
command is probably not tested anywhere, as already mentioned on other
threads.
The patch is missing a documentation update.
There are debatable changes of behavior:
if (errno == ENOENT) return / continue...
For instance, a file disappearing is ok online, but not so if offline. On
the other hand, the probability that a file suddenly disappears while the
server offline looks remote, so reporting such issues does not seem
useful.
However I'm more wary with other continues/skips added. ISTM that skipping
a block because of a read error, or because it is new, or some other
reasons, is not the same thing, so should be counted & reported
differently?
+ if (block_retry == false)
Why not trust boolean operations?
if (!block_retry)
--
Fabien.
