[ https://issues.apache.org/jira/browse/PIVOT-747?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13039143#comment-13039143 ]
Noel Grandin commented on PIVOT-747: ------------------------------------ I'm in agreement with Greg, trying to fix Andrei's use-case is getting too complex. So yes, I think we should make it possible to turn it off with a system property. I think it's actually fairly robust in general - very few developers play the kind of EDT-switching tricks that Andrei does. For those developers not using such tricks, it provides early-on checking for a class of problems that is otherwise hard to debug. I also note from reading the bugs referenced by Andrei, that this particular problem is fixed in JDK7, so when that comes around, Andrei will not longer need to turn this off. > pivot & blocking edt > -------------------- > > Key: PIVOT-747 > URL: https://issues.apache.org/jira/browse/PIVOT-747 > Project: Pivot > Issue Type: Improvement > Components: wtk > Affects Versions: 2.0 > Reporter: Andrei Pozolotin > Assignee: Noel Grandin > Priority: Minor > Fix For: 2.0.1 > > Attachments: Container.java, patch-001-exit.patch > > > Greg, hello; > in this thread: > http://mail-archives.apache.org/mod_mbox/pivot-user/201001.mbox/%3c4b5e581d.2080...@hms.harvard.edu%3E > your final word is: > "Sorry, it is not possible" > but I know that you know that it is possible :-) > the reason I need this is same as Martin here: > http://netbeans.org/bugzilla/show_bug.cgi?id=90590 > namely: do some cleanup after shutdown was requested and confirmed: > org.apache.pivot.wtk.Application > public boolean shutdown(boolean optional) throws Exception; > since you call shutdown(boolean optional) from EDT I need to block it, using > this approach: > http://bugs.sun.com/view_bug.do?bug_id=6424157 > wich "almost works", except you have this check everywhere: > Container.assertEventDispatchThread(); > which fails, as described above: > http://bugs.sun.com/view_bug.do?bug_id=6424157 > due to EventQueue.isDispatchThread() failing on the "T1 vs T1*" distinction > my request is this: > do you think you could make > Container.assertEventDispatchThread(); > less pedantic, and allow both "current and past/next" EDT threads to pass > which are created during EventQueue push() / pop() > thanks! > Andrei -- This message is automatically generated by JIRA. For more information on JIRA, see: http://www.atlassian.com/software/jira