Griffon
  1. Griffon
  2. GRIFFON-464

Rename threading methods to spell out which thread they affect

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.5-rc2
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      It's pretty easy to get confused with which method must be called to invoke code inside/outside the UI thread.
      Proposed changes

      in griffon.core.ThreadingHandler

       execOutside -> execOutsideUI
       execSync -> execInsideUISync
       execAsync -> execInsideUIAsync
      

      in griffon.core.GriffonApplication

       eventOutside -> eventOutsideUI
      

      in griffon.util.EventPublisher

       publishEventOutside -> publishEventOutsideUI
      

      in org.codehaus.griffon.runtime.core.EventRouter

       publishOutside -> publishOutsideUI
      

        Activity

        Show
        Andres Almiray added a comment - Fixed with https://github.com/griffon/griffon/commit/68540adba9fcecd6724b20b3cc20d8f1d5dcec07

          People

          • Assignee:
            Andres Almiray
            Reporter:
            Andres Almiray
          • Votes:
            1 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: