testgit: bring up to PEP 8 standards.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
testversion: bring spacing up to PEP 8 standards.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
testversion: refactor to eliminate duplicate code.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
testdatabase: bring up to PEP 8 standards.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
testfilemanip: bring up to PEP 8 standards.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
Makefile: add pep8 checking to the test target.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
Remove unused imports.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
main: add missing import of sys.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
database: properly qualify function name.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
database: don't hard-code class name.

It's much better to use a class method than a static method and calling
the class name directly, so that any subclasses can override the
relevant methods if need be.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
filemanip: make QDDB code produce CorruptFileError.

The use of ValueError was inconsistent with the rest of the code, so
make the QDDB code use CorruptFileError as well.  This also has the nice
side effect of making the error messages consistent and well-formatted.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>
filemanip: explicitly initialize fieldno.

We know, due to the way QDDB files are structured and the way the code
is written, that fieldno will always be initialized appropriately.
However, pyflakes3 does not, so help it out by initializing the
variable.

Signed-off-by: brian m. carlson <sandals@crustytoothpaste.net>