Application#shutdown() is currently defined to return true if it is OK to shut down, or false if it is not. I'm wondering if this might be confusing. It is different than how consumable events are handled, where the return value means "yes, consume the event" or "no, do not consume the event".

It might be clearer for shutdown()'s return value to mean "yes, cancel shutdown" (true) or "no, do not cancel shutdown" (false). Also, by default, Eclipse provides a stub method that returns false. An unsuspecting developer could easily be tripped up by this and create an application that will never shut down.

What do others think?


Reply via email to