@@@ dvdrip-upload: change settings while i'm stealing someone else's internet

dvdrip-upload: Add additional burbling around check sweeps.

dvdrip-upload: Replace the signalling protocol with, err, signals.

No longer needs lockfiles.

dvdrip-upload: Add an option for setting the archive name.

dvd-sector-copy.c: Make constants local to the only function which uses them.

dvd-sector-copy.c: Add  nonlinear progress models.

The details are rather complicated, so they're explained in the
commentary.  It seems to work pretty well in practice, settling in
within 5% within a minute or two of starting to copy an OTP disc.

dvd-sector-copy.c: Add `-P' option to log progress state for later analysis.

This is intentionally undocumented.

dvd-sector-copy.c: Update progress data at event edge.

Don't just redisplay.  In particular, this forces output right at the
very beginning, before we read anything, which will be significant
soon.

dvd-sector-copy.c: Package up the moving-average machinery.

Now we can have more than one.

dvd-sector-copy.c: Add substantial commentary.

And light reformatting.

dvd-sector-copy.c: Change the moving-average denominator computation.

Rather than updating this using the geometric-series formula as before,
calculate it as (1 - β^t), incorporating the initial correction from
Wikipedia.  My testing suggests that this stabilizes rather more
quickly.

dvd-sector-copy.c (update_progress): Squish time delta calculation.

dvd-sector-copy.c: Sink `#undef' for logically local macros to very end.

dvd-sector-copy.c: Sink debugging spew to the end of the `switch' arm.

dvd-sector-copy.c (main): Use `last' to track final event for fixup.

The `start' variable was doing double-duty, resulting in a possibly
erroneous value if the last event is `EV_WRITE', and a possibly
erroneous count of sectors.  Use `last' instead, because nothing else is
using this.

dvd-sector-copy.c (main): Sink progress setup to just before opening device.

dvd-sector-copy.c: Reformat an overly-long line.

dvd-sector-copy.c (main): Add a flag to track retrying.

Previously we'd try to determine whether retry was intended from whether
there are (begin/stop) events in the queue.  This is bad because we
might have tried to read an empty retry file, in which case we'd still
have no events, but the intend would clearly be to do nothing rather
than re-read the entire input disc.

dvd-sector-copy.c: Don't adjust a variable which is never spoken of again.

dvd-sector-copy.c: Fold loop initialization into the `for' satatement.

dvd-sector-copy.c: Commute some operations.

I think it makes slightly better sense this way.

dvd-sector-copy.c: Delete a redundant `else'.

This will read better once the commentary is added.

dvd-sector-copy.c (emit): Update progress display after adding file progress.

This will show the new progress, at the start of the file, immediately,
rather than showing an update without the file data just before the
first file.

dvd-sector-copy.c (find_good_sector): Use `good' rather than `bad_hi'.

It should be the case that, if there's any difference at all, `good' is
earlier than `bad_hi', which will give us more good data.

dvd-sector-copy.c: Calculate the clear-area size consistently.

This should be determined as a (possibly fractional) multiple of the
distance from the start of the bad region to the current position.
Alas, the code for the bounding stage incorrectly used the distance from
the previous position to the new one.  The inconsistency between the
bounding and refinement stages of the recovery algorithm could lead to
failed invariants and a nonsense result.

dvd-sector-copy.c (find_good_sector): Update the progress display later.

In particular, after clearing the display for debugging rather than
before.

@@@ dvd-sector-copy.c: Rearrange into a more logical order.

dvd-sector-copy.c (find_good_sector): Use plain `read_sectors' for retries.

This saves engaging the recovery buffering machinery until it's actually
going to be useful.  If the retry read actually reads anything at all
then we've succeeded and don't need the recovery machinery; if it
doesn't, then there's nothing left in the buffer anyway.

dvd-sector-copy.c (run_length_wanted): Round the initial requirement up.

dvd-sector-copy.c: Fold `report_bad_blocks_progress' into `recovered'.

It's not very complicated; it's only used once; and it saves a call to
`store_filename'.

dvd-sector-copy.c (recovery_read_sectors): Cross-check POS and OFF.

dvd-sector-copy.c (read_sectors): Force `errno' to zero even if WANT = 0.

Otherwise we'd get confused at the end.  I don't think anything actually
tries to call `read_sectors' with WANT = 0, but it's better not to leave
a booby trap here.

dvd-sector-copy.c: Fix broken indentation.

lib.[ch], *.c: Have `open_dvd' return errors rather than bailing.

In particular, this means that `dvd-id' can report on all of the other
discs.

chkdvdimg.c: Factor out `set_status' subroutine.

lib.[ch], dvd-sector-copy.c: Publish `read_line'.

lib.[ch], dvd-sector-copy.c: Publish the `buf' machinery as inline functions.

Translate the macros into inline functions, and export them from
`lib.h', with some minor tweaking.

  * I've added `buf_putz', and fixed `buf_putc' to advance the output
    pointer as one would expect.

  * I've pulled the actual buffer extension into an out-of-line
    function.

lib.[ch]: Move the memory allocation out-of-line.

lib.h: Add missing documentation of FILE argument to `device_size'.

dvd-sector-copy.c: Sit for the bad-blocks delay if we get an actual hit.

lib.[ch], dvd-sector-copy.c: Publish a couple of generally handy utilities.

lib.h: Disable magic attributes if the compiler doesn't understand them.

lib.[ch]: Add documentation and commentary.

lib.h: Make `MAXIDSZ' clearer about where the value comes from.

The number 99 is just a bit too round to believe that it's actually
carefully calculated rather than chosen arbitrarily.

lib.[ch]: Rearrange the toplevel definitions into a more sensible order.

No code changes.

multiprogress.h: Use a macro to mark `printf'-like functions.

Rather than using `__attribute__' directly.

multiprogress.[ch]: Add commentary.

Also some light layout changes.

multiprogress.c (progress_shownotice): Remove pointless guard.

The `progress_put_sequence' function already does the right thing here.

multiprogress.c: Set the terminal stream to be fully buffered.

The `free_render_state' function correctly forces output when we're done.

multiprogress.c (clear_progress): Don't check for output errors.

Nobody's watching to see if we found any.  Indeed, we might as well not
return anything at all.  So, err, don't.

multiprogress.c (clear_progress): Don't check `tty->fp': the caller did it.

multiprogress.[ch]: Publish the terminal-sequence output functions.

Otherwise externally-defined renderers don't have a good way to do this.

multiprogress.c (clear_progress): Write the `n - 1' loops in a better way.

multiprogress.c: Don't copy the contents of the temporary buffer.

Nothing relies on the temporary buffer's contents being preserved.

multiprogress.c: Fix `split_string' to overestimate.

Doing the right thing in `advance_measure' is much harder if
`split_string' sometimes falls short of the requested bound because of
wide characters.  On the other hand, overestimating is a little tricky
because we must remember to include all of the following zero-
width (e.g., combining) characters or the prefix will just be wrong.
But it's better to do this than come up with some kludge to make
`advance_measure' sometimes advance the state even when we've not
actually reached the right position.

multiprogress.c: Remove pointless return value from `setup_render_state'.

It's being checked, and, indeed, passed on.  But it didn't actually do
very much other than prevent the terminal-width fallback logic from
being useful.  So delete it.

multiprogress.[ch]: Rearrange the toplevel definitions more sensibly.

multiprogress.c: Pair calls to `free' with zeroing the points.

multiprogress.[ch]: Save and restore `PC' in the render state.

multiprogress.h: Add the right number of zeros to `PROGRESS_TTYINFO_INIT'.

multiprogress.c: Get the `termcap' string after allocating the buffers.

It works better that way, except with GNU `termcap'.

multiprogress.h: Fix an erroneously indented line.

Merge branch 'master' of git.distorted.org.uk:~mdw/publish/public-git/dvdrip

* 'master' of git.distorted.org.uk:~mdw/publish/public-git/dvdrip:
  dvd-info.c: Trigger the easy lookup if explicit chapter bounds aren't given.
  dvd-info.c: Don't forget to dump the last titleset.
  dvd-info.c: Add missing `\' in output banner.

chkdvdimg.c: Open the image file with `open_dvd'.

This means that it will wait until we actually insert a disc.  Yay!

lib.c, and knock-on: Add MODE argument to `open_dvd'.

Everyone wants read-only for now.

dvd-info.c: Trigger the easy lookup if explicit chapter bounds aren't given.

dvd-info.c: Don't forget to dump the last titleset.

dvd-info.c: Add missing `\' in output banner.

Add a new program `dvd-info' to report on interesting things.

I'm getting deeper in here, I know.

checkimg: Don't complain about every mismatch.

We can tell from the summary.

dvd-sector-copy.c: Use `bad_hi' consistently for the upper bound when done.

It's just conceivable that `good' is less than `bad_hi' at this point,
which screws everything up.  It's essential to pick a consistent upper
bound here.

dvd-sector-copy.c, lib.[ch]: Improve and publish the number parsing functions.

Makefile: Factor out the `CFLAGS'.

Makes overriding from the command line less awful.

dvd-sector-copy.c: Show file ends in the dump.

dvd-sector-copy.c: Wrap long line properly.

dvdrip-upload: Fix self-call the case where there's no daemon running.

dvd-sector-copy.c: Truncate the event list before processing a lost `stop'.

dvd-sector-copy.c: Report final statistics correctly.

dvd-sector-copy.c: Dump the event queue before we start processing it.

dvd-sector-copy.c: Eliminate unused variable.

dvd-sector-copy.c: Fix botched debugging code.

chkdvdimg.c: Name the file when reporting too many trailing zeros.

dvd-sector-copy.c: Add missing handler for the `-i' option.

chkdvdimg.c: Retrieve device size using the new function.

... which works on block devices as well as ordinary files.

dvd-sector-copy.c: Export device-size machinery to the library.

dvdrip: Check for the device size apparently changing during the copy.

This indicates something going horribly wrong.

dvdrip: Fail if we can't read the `.IFO' hash for the disc id.

This may help with the kernel reporting stale medium sizes.

dvd-sector-copy.c: Add option to verify source id.

scripts: Call utilities via environment variables.

dvd-id.c, lib.[ch]: Improve DVD ID computation.

Move the code to the library.  Accept flags (to the library function and
the utility) to fail if portions of the ID can't be determined.

Also, improve `dvd-id' to print IDs for multiple disc images.

chkdvdimg: Add a new program for checking DVD images.

Primarily I'm interested in checking that the length is right, because
one of my DVD drives sometimes reports the wrong medium size
(specifically, I think, the size for the previous medium).  Also,
sometimes there are zero sectors stuck on the end, which complicates
finding the backup UDF anchor descriptor, so provide an option to trim
these.

lib.h: Publish a `MEMCMP' macro, like `STRCMP'.

lib.c, lib.h: Introduce `vmoan_syserr' and `moan_syserr'.

This cleans up this little family of functions.  There's a behaviour
change: `vmoan' now prints a newline which it didn't before, and callers
have been adjusted to compensate.

lib.c: Remove redundant format attributes.

Should use `PRINTF_LIKE' nowadays anyway.

dvdrip-retry-botched-vobs: Retry using the newly determined block list.

Fortunately, I didn't actually engage this case when I did this to my
collection.

dvdrip-retry-botched-vobs: Process discs in a useful order.

dvdrip-retry-botched-vobs: Eject discs when we're done with them.

dvdrip-retry-botched-vobs: Delete the retry file correctly.

dvdrip-retry-botched-vobs: Be more vocal about wrong discs.

New programs for dealing with VOBs ripped without being decrypted.

When my drives were configured with the wrong region, `libdvdcss' would
have to crack the keys the hard way, and it sometimes failed.  I mostly
shrugged my shoulders, but now I've fixed this problem, I thought I'd
track down the miscopied VOBs and make some tools for fixing the
problem.