Fop
  1. Fop
  2. FOP-1121

image rendering differs between 0.90 and 0.91

    Details

    • Type: Bug Bug
    • Status: Closed
    • Resolution: Fixed
    • Affects Version/s: 0.91
    • Fix Version/s: None
    • Component/s: images
    • Labels:
      None
    • Environment:
      Operating System: All
      Platform: All
    • External issue ID:
      38089

      Description

      While rendering a .bmp image in fop 0.90alpha1 the pdf renderer rendered the
      image with exact size, it is resized in 0.90beta.

      My bitmap EventReminderIntegration.bmp has a width of 13.56cm, and a height of
      5.95cm.

      My xml contains this code:

      <fo:block space-before.optimum="20pt">
      <fo:external-graphic content-width="13.56cm" content-height="5.95cm"
      src="url(EventReminderIntegration.bmp)"/>
      </fo:block>

      In 0.91beta1, the image is resized to 5.5cm*5.5cm

        Activity

        Hide
        Manuel Mall added a comment -

        There was a change between 0.90 and 0.91 with respect to the image sizing code.
        That change exposed a bug in the BMP image header decoder which incorrectly
        reported the image width as the image height as well. That problem wasn't
        caught before because in our test suite most images are square. Any volunteer
        out there prepared to change all the test images to some rectangular format and
        to adjust the testcases accordingly?

        Any way the problem should be fixed now: http://svn.apache.org/viewcvs?
        rev=365282&view=rev

        Show
        Manuel Mall added a comment - There was a change between 0.90 and 0.91 with respect to the image sizing code. That change exposed a bug in the BMP image header decoder which incorrectly reported the image width as the image height as well. That problem wasn't caught before because in our test suite most images are square. Any volunteer out there prepared to change all the test images to some rectangular format and to adjust the testcases accordingly? Any way the problem should be fixed now: http://svn.apache.org/viewcvs? rev=365282&view=rev
        Hide
        removed added a comment -

        Tested with fresh build from fop-trunk.
        It is indeed resolved.
        Thank you.

        Show
        removed added a comment - Tested with fresh build from fop-trunk. It is indeed resolved. Thank you.
        Hide
        Glenn Adams added a comment -

        batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed

        Show
        Glenn Adams added a comment - batch transition pre-FOP1.0 resolved+fixed bugs to closed+fixed

          People

          • Assignee:
            fop-dev
            Reporter:
            removed
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development