Appearance streams: some floating point values were truncated

Bounding box X coordinates could be truncated, causing them to be off by a fraction of a point. This was most likely not visible, but it was still wrong.

Appearance streams: some floating point values were truncated
Bounding box X coordinates could be truncated, causing them to be off by a fraction of a point. This was most likely not visible, but it was still wrong.
Jay Berkenbilt
1 parent e3129ef4
Showing 10 changed files with 1670 additions and 1670 deletions
libqpdf/QPDFFormFieldObjectHelper.cc
qpdf/qtest/qpdf/appearances-1.pdf
qpdf/qtest/qpdf/appearances-11.pdf
qpdf/qtest/qpdf/appearances-12.pdf
qpdf/qtest/qpdf/appearances-2.pdf
qpdf/qtest/qpdf/appearances-a-more.pdf
qpdf/qtest/qpdf/appearances-a-more2.pdf
qpdf/qtest/qpdf/appearances-a.pdf
qpdf/qtest/qpdf/appearances-b.pdf
qpdf/qtest/qpdf/appearances-quack.pdf
@@ -676,10 +676,10 @@ ValueSetter::writeAppearance()
     if (highlight)
     {
         write("q\n0.85 0.85 0.85 rg\n" +
-              QUtil::int_to_string(bbox.llx) + " " +
+              QUtil::double_to_string(bbox.llx) + " " +
               QUtil::double_to_string(bbox.lly + dy -
                                       (tfh * (highlight_idx + 1))) + " " +
-              QUtil::int_to_string(bbox.urx - bbox.llx) + " " +
+              QUtil::double_to_string(bbox.urx - bbox.llx) + " " +
               QUtil::double_to_string(tfh) +
               " re f\nQ\n");
     }
@@ -693,7 +693,7 @@ ValueSetter::writeAppearance()
         // which doesn't seem really worth the effort.
         if (i == 0)
         {
-            write(QUtil::int_to_string(bbox.llx + dx) + " " +
+            write(QUtil::double_to_string(bbox.llx + dx) + " " +
                   QUtil::double_to_string(bbox.lly + dy) + " Td\n");
         }
         else