Eliminate warnings when compiled with -Wall.
Eliminate warnings when compiling searchhandler.cpp with -Wall
Reformat closehandler.cpp so I don't combine a reformat with the changes required to eliminate all warnings with -Wall.
Make some changes intended to make it easier to mock a session class instance.
Fully parenthesize the MAX macro.
Make the string parameters passed to lock and unlock be const parameters which came out of the effort to mock it for unit tests.
Reformat before possibly making more substantive changes.
Reformat before possibly making substantive changes.