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.

dvdrip-monitor: New program to keep track of what's going on.

dvdrip: Leave a note in the working directory about which device we're using.

dvd-sector-copy.c, lib.[ch]: Move the file hacking functions to the library.

Makefile: Only `dvd-sector-copy' needs `-lm'.

multiprogress.c: Leave the cursor at the start of the last line.

Things still go wrong when stray characters get echoed but it's less
bad.  I think.

multiprogress.c: Probe termcap/info for newline sequence.

Just to show willing.  Provide the obvious default, because it seems to
be necessary!

multiprogress.c: Provide the obvious default for `cr'.

dvd-cache-keys.c: Allow multiple devices and/or image files.

dvd-cache-keys.c: Move the existence check into `kick_vob'.

Much neater.

dvd-cache-keys.c: Pass index and titleness as unsigned.

dvd-sector-copy.c: Allow averaging `alpha' to be set as a parameter.

dvd-sector-copy.c: Reinterpret the `step' as being beyond the high bound.

dvd-sector-copy.c: Just a light reformatting.

dvd-sector-copy.c: Go back to showing just disc progress when continuing.

dvd-sector-copy.c: Remove the pointless `status' variable.

Relic of former days when `dvdrip' used the return code to infer that
there were bad blocks.  Nowadays it checks for the `badblocks' file.

dvd-sector-copy.c: Report the number of bad sectors in the bad-blocks file.

A big mess of changes all at once.

I'm usually quite good about factoring out all of the changes I make, but
in this case it doesn't seem worth the bother.

dvdrip: Squish some code a bit.

dvdrip: Abolish the affordances for using `dvdbackup'.

dvdrip: Add `-r' option to retry the failed blocks.

dvd-sector-copy.c (report_progress): Fiddle with the averaging parameter.

dvd-sector-copy.c (report_progress): Show a throbber to help gauge speed.

dvd-sector-copy.c (recovered): Don't print comment if outside a VOB file.

dvd-sector-copy.c, dvdrip: Cue prompt to check image from `badblocks' file.

The exit status from the final run isn't helpful, but the bad-blocks
file is.

dvdrip: Prepare the `dest' file early, so that it can be edited.

dvd-sector-copy.c: Use the correct character to match comments.

Oops.

sort-badblocks: Another handy testing utility.

dvd-sector-copy.c: Limit `recovery_read' return value to length requested.

Otherwise the caller gets properly confused.

dvd-sector-copy.c: Add more debugging about the recovery algorithm.