ICEpdf
  1. ICEpdf
  2. PDF-706

Page pagePaint listeners not being removed

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 5.0.4
          • Fix Version/s: 5.0.5
          • Component/s: Core/Parsing
          • Labels:
            None
          • Environment:
            any
          • Salesforce Case Reference:

            Description

            A supported customer came across this bug in our code when doing some tracing. The code reads as

                public void removePaintPageListener(PaintPageListener listener) {
                    // remove a listener if it is already registered
                    synchronized (paintPageListeners) {
                        if (paintPageListeners.contains(listener)) {
                            paintPageListeners.add(listener);
                        }

                    }
                }

            The typo/bug is patinPageListeners.add() instead of remove.

              Activity

              Ascending order - Click to sort in descending order
              Patrick Corless created issue -
              Hide
              Permalink
              Judy Guglielmin added a comment -

              from user:- no specific file to cause this jira but the following comment:-

              Unfortunately we don't have a specific pdf for this issue.
              The problem happened when pdf files were opened and closed
              fast.

              Show
              Judy Guglielmin added a comment - from user:- no specific file to cause this jira but the following comment:- Unfortunately we don't have a specific pdf for this issue. The problem happened when pdf files were opened and closed fast.
              Judy Guglielmin made changes -
              Field Original Value New Value
              Salesforce Case Reference 5007000000YZ0GMAA1
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #39519 Thu Jan 09 13:52:36 MST 2014 patrick.corless PDF-706 updated removePaintPageListener() to actually remove the listener from the list.
              Files Changed
              Commit graph MODIFY /icepdf/branches/icepdf-5.0.1/icepdf/core/src/org/icepdf/core/pobjects/Page.java
              Repository Revision Date User Message
              ICEsoft Public SVN Repository #39520 Thu Jan 09 14:06:24 MST 2014 patrick.corless PDF-706 updated removePaintPageListener() to actually remove the listener from the list.
              Files Changed
              Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/Page.java
              Hide
              Permalink
              Patrick Corless added a comment -

              Marking as fixed.

              Show
              Patrick Corless added a comment - Marking as fixed.
              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:
                  Patrick Corless
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  2 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: