94e8052 wasn't right either: we need any overflow item appearing in the
range, so increment the record number before the test, but don't compare
against the stop recno in the test, do that afterward.  In other words,
if we're past "start", then we need to record, and if we're past stop,
we quit looking.
Copy the corrupted file before salvaging it, the dd command to
reproduce the corruption is too painful to build every time I
want to debug one of these.
One more try to get the range check for overflow records correct.
Decrement the RLE *before* setting the number of skip items to 0.
Forward declarations of enumerated types is a gcc extension, we don't
need to do it anyway.
clang complaint: bt_curprev.c:83:6: warning: variable 'ins' may be
uninitialized when used here [-Wconditional-uninitialized]
clang complaint: rec_split.c:1023:16: warning: variable 'parent' may be
uninitialized when used here [-Wconditional-uninitialized]
clang complaint: rec_write.c:2141:15: warning: variable 'result_len' may
be uninitialized when used here [-Wconditional-uninitialized]
clang complaints for unconditionalized variables; I don't believe either are
a real problem, flag them as compiler warnings.
Flag the change as a compiler warning, I don't think it's a real problem.
clang unconditionalized variable complaint
sort the switch options, fix the switch formatting.
Tables aren't "wt", they're "wt.wt", fix things up so we corrupt them
too, and test salvage.
Minor shuffling to turn on more clang warnings:

- a few more uninitialized variables
- make variables local to a single file static
- replace use of WT_UNUSED_RET with error checking of the call (the
	macro was using a GCC extension clang doesn't like)

With these changes, clang is mostly silent using the following options:

	-Weverything
	-Wno-cast-align
	-Wno-conversion
	-Wno-covered-switch-default
	-Wno-documentation-unknown-command
	-Wno-format-nonliteral
	-Wno-language-extension-token
	-Wno-missing-noreturn
	-Wno-packed
	-Wno-padded
	-Wno-switch-enum
	-Wno-zero-length-array

Also, minor KNF whitespace cleanups.
typo, get rid of "leaf leaf" in error message.
Rework 5169532; it's not enough to select a page size that's large
enough that we won't have to split, we also have to turn off split
processing.  Shuffle some code: if __rec_split_init() has to know
about salvage anyway, might as well put all the magic there.
Minor wording changes to clarify the LevelDB configuration options.
update
update