Uploaded image for project: 'Pivot'
  1. Pivot
  2. PIVOT-747

pivot & blocking edt

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 2.0
    • 2.0.1
    • wtk
    • None

    Description

      Greg, hello;

      in this thread:
      http://mail-archives.apache.org/mod_mbox/pivot-user/201001.mbox/%3C4B5E581D.2080604@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

      Attachments

        1. patch-001-exit.patch
          1 kB
          Andrei Pozolotin
        2. patch_2011-05-29_assert-runner.patch
          1 kB
          Andrei Pozolotin
        3. patch_2011-05-27_edt-name.patch
          0.7 kB
          Andrei Pozolotin
        4. Container.java
          1 kB
          Andrei Pozolotin

        Issue Links

          Activity

            People

              grandinj Noel Grandin
              andrei.pozolotin Andrei Pozolotin
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: