Don't call stop() just because shouldRun is false - run() might be called again which would be impossible if fully stopped

Don't set anything which has to be reused to null in cleanup. It might be used in other calls to run() afterwards

By using locking and indices no complex data types are needed.

By avoiding the check this logic saves one synchronized block.