ICEpdf
  1. ICEpdf
  2. PDF-1215

Memory leak in multi page view

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 6.0.2
          • Fix Version/s: 6.3
          • Component/s: None
          • Labels:
            None
          • Environment:
            Windows 10
          • Workaround Description:
            Hide
            Add this to the the dispose() method in TwoPageview.java

            if (currentPageChanger != null) {
            currentPageChanger.dispose();
            }
            Show
            Add this to the the dispose() method in TwoPageview.java if (currentPageChanger != null) { currentPageChanger.dispose(); }

            Description

            In the multi page view (two pages next to each other) there is a memory leak. If the user changes to this view mode and the image pool is not activated (-Dorg.icepdf.core.views.imagePoolEnabled=false) it doesn't take long till the memory is gone. That's because in the class TwoPageView the currentPageChanger is not disposed.

              Activity

              Ascending order - Click to sort in descending order
              Richard Schiepek created issue -
              Patrick Corless made changes -
              Field Original Value New Value
              Fix Version/s 6.3 [ 13093 ]
              Hide
              Permalink
              Patrick Corless added a comment -

              Thanks Richard, much appreciated.

              Show
              Patrick Corless added a comment - Thanks Richard, much appreciated.
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as fixed.

              Change was submitted in rev. 52082

              Show
              Patrick Corless added a comment - Marking as fixed. Change was submitted in rev. 52082
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Patrick Corless made changes -
              Status Resolved [ 5 ] Closed [ 6 ]

                People

                • Assignee:
                  Patrick Corless
                  Reporter:
                  Richard Schiepek
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: