Zachary Griggs wrote:Also, it's bad practice to catch Exception, and worse practice to let it silently fail (do nothing in the catch).
You're
still ignoring exceptions. So you're calling the stop() method, which is known to throw exceptions if something goes wrong. And it appears that something may be going wrong. If that's the case then your
empty catch block is executed, telling you exactly nothing. So stop ignoring the exceptions. At the very least print out the stack trace: