flamingo
  1. flamingo
  2. FLAMINGO-86

Memory "leak" in BasicCommandPopupMenuUI

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 5.0
    • Fix Version/s: milestone 1
    • Component/s: common components
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      86

      Description

      Hi,after some memory analysis on our (Exie's) Swing application I have found what appears to be a memory problem.

      The method org.pushingpixels.flamingo.internal.ui.common.popup.BasicCommandPopupMenuUI#uninstallListeners()

      line 503
      does: PopupPanelManager.defaultManager().addPopupListener(this.popupListener);

      instead of: PopupPanelManager.defaultManager().removePopupListener(this.popupListener);

      To me this seem to be incorrect, I admit that there might be aspects of the code I don't understand. After recompiling with
      this change. The detected memory leak in our system was gone. And no new problems were introduced to the application.

      Hope this isn't a duplicate I tried searching for similar problems couldn't find any.

      -Jørn

        Activity

        There are no comments yet on this issue.

          People

          • Assignee:
            kirillcool
            Reporter:
            heigum
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: