Re #7424. A better way of setting variable to infinity.
Re #7424. Remove integer conversion compiler warning.
Re #7424. Remove now unused variable.
Re #7424. Remove no-longer needed warning suppression.
Re #7424. Address cppcheck warnings.

…while I'm touching the file anyway.
Confidence method added

Confidence method is in and checks for a .vtk extension and if the file isn't ascii

Refs #7420
LoadVTK will now be elected when using the generic loader

LoadVTK will do 3 simple checks to determine how confident it is:
if the file isn't ASCII and ends in .vtk it's very confident (80)
if it ends in .vtk it's fairly confident (60)
if it's not ASCII it'll attempt it only if all other non-ascii loaders reject it (15)
otherwise it'll reject it (0).

LoadVTKTest will now check to see if the generic loader returns a not-null object

Refs #7420
Reduced calls to FileDescriptor::isAscii()

FileDescriptor::isAscii() is now only called once, and the result is stored

Refs #7420
Remove doxygen warnings

re #7450
Re #7424. Put 32 bit member variables next to each other.

This saves 8 bytes per object padding with gcc (which seems to ignore
the packing pragma) and will allow me to turn off the packing for the
intel compiler (which gives an avalanche of warnings) without any
effect.
Re #7424. Turn of packing for the Intel compiler.

With the present set of member variables it makes no difference and,
for the most recent compiler version, leads to a warning every time the
header is included (which is a lot!).
N.B. The packing also has no effect with gcc but I've left it on there.
Re #7424. Tidy-ups. No functional changes.
Re #7424. Turn off packing for the Intel compiler.

With the present set of member variables it makes no difference and,
for the most recent compiler version, leads to a warning every time the
header is included (which is a lot!).
N.B. The packing also has no effect with gcc but I've left it on there.
Re #7424. Turn off packing for the Intel compiler.

It makes no difference and gives a warning. (N.B. there's no difference
with gcc either, only visual c++.)
Re #7424. Tidy-ups. No functional changes.
Remove version number from python warning message on startup.

Keep it general about what the version number will be.
Refs #7460
refs #7372. Add updated script files.
refs #7372. Move to new version of python API.
refs #7372. Further migration work to v2 of python API

Things appear to be running the same in this gui+script as they were in the old gui+script. I've done a little refactoring work, but the bulk of the refactoring work will have to wait, as there are many violations.
Fixed unit tests to check for no throws

Was my fist time altering a unit test so i didn't know quite how it worked.

I was told to use TS_ASSERT_THROWS_NOTHING on the last two lines instead

Refs #7420