more clean up for job alert runner. a lot more methods and objects
are static. It is now obvious that the object is really just a wrapper
for a single boolean in spawn (whether alerts are enabled).
(the only two fields are the reference to spawn and the enabled boolean,
and the spawn reference is final. So, eg. the class could be turned into
an entirely static utility class if that boolean were held in spawn and
passed in to scan alerts, or more likely, just moved the single enclosing if
statement out of scan alerts)
also some other clean up and optimizations