On Fri, 27 Jan 2023 05:05:37 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> wrote:
> Should we just update the spec citing it's not necessary to wait for > `doInBackground()` to finish before `done()` is called? It is also an option. All the suggested approaches set the state to `DONE` before `done` is called now. Yet the Workflow states the following: “After the `doInBackground` method is finished the `done` method is executed. Then `SwingWorker` notifies any `PropertyChangeListeners` about the state property change to `StateValue.DONE`.” However, the last sentence may not have been guaranteed before either but it was likely satisfied as `doneEDT` was called before `setState(StateValue.DONE)`. So, `done` had likely been executed before `PropertyChangeListener` was notified. Hence, a small update to the spec may still be required whatever option we choose. ------------- PR: https://git.openjdk.org/jdk/pull/11940
