Don't wait until the end of routine to return error codes. Leads to somewhat cleaner code and eliminates if statements
Maximum debug level is 2 AFAIK
StIstRawHitMaker: Do not protect for possible double delete. This maker is solely responsible for the StIstCollection
No need to empty vectors in destructor as they will be destructed anyway
StIstHitMaker: No need to check for valid pointer to StIstCollection for each ladder as it has been already verified at the begining of this Make()
StIstHitMaker: No point to proceed with cluster-to-tracking-hit conversion if there are no cluster container for this ladder
StIstHitMaker: Arranged checks for required containers and objects in a more logical way. Added corresponding log messages
Updated debug and log messages, added doxygen comments. Also other minor whitespace and style changes