Details

    • Type: Sub-task Sub-task
    • Status: Open Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.0-JSR-3
    • Fix Version/s: None
    • Component/s: groovy-runtime
    • Labels:
      None
    • Number of attachments :
      0

      Description

      John Wilson:
      I have been looking at the code for ProxyMetaClass and I'm a little
      concerned by "void use(Closure closure)". This changes the metaclass
      returned by the registry for the duration of the execution of the
      closure.
      [..]
      2/ this is not thread safe: the code changes the metaclass for all
      the threads in the JVM.
      [..]
      Is 2/ the intended behaviour? If not we need to do something like
      the thread local storage trick done by Sam's "use" implementation.

      Dierk:
      2/ Yes, I raised the issue myself when commenting the commit.
      Using a TreadLocal is fine for me.

      We should investigate on the need for additional
      'synchronized' modifiers...

        Activity

        There are no comments yet on this issue.

          People

          • Assignee:
            Unassigned
            Reporter:
            Dierk König
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: