From: SourceForge.net <no...@so...> - 2012-12-28 17:53:24
|
Patches item #3588172, was opened at 2012-11-17 06:29 Message generated for change (Comment added) made by ezust You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300588&aid=3588172&group_id=588 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 5 Private: No Submitted By: Thomas Meyer (thomasmey) >Assigned to: Jarek Czekalski (jarekczek) Summary: Enhance Task with Runnable Initial Comment: Enhance Task with a Runnable which will be run on the EDT after the task finishes successfully and convert some users of org.gjt.sp.util.AwtRunnableQueue.runAfterIoTasks(Runnable) to use this mechanism instead. The idea is to get rid of the AwtRunnableQueue in the long term. And to couple the Runnable which should be run after a background task finishes to the background task itself. Feedback is welcome on this idea. ---------------------------------------------------------------------- >Comment By: Alan Ezust (ezust) Date: 2012-12-28 09:53 Message: I agree, the AwtRunnableQueue seems like something we really don't need in the API. Adding the setRunInDispatch(Runnable) for a Task seems much more elegant. Other devs should look at this too since we are trying to finalize an API, but I think it's ok. jarek? Kazutoshi, Jarek, Matthieu, Dale? ---------------------------------------------------------------------- Comment By: Thomas Meyer (thomasmey) Date: 2012-12-28 02:47 Message: Hi Alan, Mhh. I guess it would have been better to open this ticket as feature request or something. I just wanted to have the opinion of the other developers on this idea. To get rid of the AwtRunnableQueue in the IOTasks classes is easy. But there exists several other users of this "delay work till IO is done" mechanism, which are not so easy to convert. maybe the other users are not possible to convert. ---------------------------------------------------------------------- Comment By: Alan Ezust (ezust) Date: 2012-12-12 13:01 Message: get rid of AwtRunnableQueue in the long term? It was just introduced. You should mark it deprecated immediately if you want to remove it . Also XSearch will need to be fixed again. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=300588&aid=3588172&group_id=588 |