ICEpdf
  1. ICEpdf
  2. PDF-781

PageViewComponent buffer is larger then needed

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 5.0.6_P01
    • Fix Version/s: 5.0.7
    • Component/s: Viewer RI
    • Labels:
      None
    • Environment:
      any, Viewer RI

      Description

      While working though an issue related to an empty page buffer being painted I noticed there was a bug in how our page buffers where created. Basically at various zoom levels the buffers width or height could be many times larger then needed and as a result we are using significantly more memory then we have too.

      The extra size was being clipped the but the larger then needed BufferedImage was still being created. This should drastically reduce the amount of memory being used by the RI especially when using a column page view.

        Activity

        Hide
        Patrick Corless added a comment -

        Buffer tests are complete and after further audits the buffer is being created at the minimal size required. I've also reword the math around the system properties

        org.icepdf.core.views.buffersize.vertical and org.icepdf.core.views.buffersize.horizontal. Previouisly the default values were some strange 1.012 double value. The new default is 1.25 which means the buffer will be 25% larger then the default viewport but no larger then the bounding page dimensions.

        Show
        Patrick Corless added a comment - Buffer tests are complete and after further audits the buffer is being created at the minimal size required. I've also reword the math around the system properties org.icepdf.core.views.buffersize.vertical and org.icepdf.core.views.buffersize.horizontal. Previouisly the default values were some strange 1.012 double value. The new default is 1.25 which means the buffer will be 25% larger then the default viewport but no larger then the bounding page dimensions.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: