- 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.
isn't good, we'll drop core if they ever fire, not a good thing in a
sample program. Add boilerplate to the examples, if we don't have an
evironment specified by "make check", create a clean standard one and
use it.
8. uninit_use: Using uninitialized value "value".
Restructure so we drop core if there's a problem (initialize
key & value to NULL), and we don't read "value" unless it's
been initialized.
success.
Add additional wording that WT_CURSOR::reset should be called after any
of the WT_CURSOR search, search-near, update or remove methods, once
the cursor is no longer being used to iterate or retrieve values.
Closes#587.
