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

              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
              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

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: