PicoContainer
  1. PicoContainer
  2. PICO-83

findComponentAdapter(Object) throws ConcurrentModificationException [patch]

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.0-beta-4
    • Component/s: PicoContainer (Java)
    • Labels:
      None
    • Number of attachments :
      1

      Description

      findComponentAdapter(Object) throws ConcurrentModificationException when parent weak ref goes away, due to improper handling of collection/iterator.

        Activity

        Hide
        Rafal Krzewski added a comment -

        a trivial patch that fixes the bug

        Show
        Rafal Krzewski added a comment - a trivial patch that fixes the bug
        Rafal Krzewski made changes -
        Field Original Value New Value
        Attachment parentReferenceIterator.diff [ 11158 ]
        Aslak Hellesøy made changes -
        Assignee Paul Hammant [ paul ]
        Hide
        Aslak Hellesøy added a comment -

        Can you take this one? I'll probably be away from my laptop for a week...

        Show
        Aslak Hellesøy added a comment - Can you take this one? I'll probably be away from my laptop for a week...
        Jon Tirsen made changes -
        Assignee Paul Hammant [ paul ] Jon Tirsen [ tirsen ]
        Aslak Hellesøy made changes -
        Fix Version/s 1.0 [ 10145 ]
        Status Open [ 1 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]
        Aslak Hellesøy made changes -
        Assignee Jon Tirsen [ tirsen ] Aslak Hellesoy [ rinkrank ]
        Aslak Hellesøy made changes -
        Resolution Fixed [ 1 ]
        Status Closed [ 6 ] Reopened [ 4 ]
        Aslak Hellesøy made changes -
        Fix Version/s 1.0 [ 10145 ]
        Environment
        Fix Version/s 1.0-beta-4 [ 10412 ]
        Aslak Hellesøy made changes -
        Status Reopened [ 4 ] Closed [ 6 ]
        Resolution Fixed [ 1 ]

          People

          • Assignee:
            Aslak Hellesøy
            Reporter:
            Rafal Krzewski
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Time Tracking

              Estimated:
              Original Estimate - 5 minutes
              5m
              Remaining:
              Remaining Estimate - 5 minutes
              5m
              Logged:
              Time Spent - Not Specified
              Not Specified