diff options
author | halcanary <halcanary@google.com> | 2016-07-25 13:59:30 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2016-07-25 13:59:30 -0700 |
commit | 56586b6f3dc824431958946ac8c27f0c0b564a3d (patch) | |
tree | a5b81819648880b2dbbb52d05cb2edf6bcd7b0e7 /tests/PDFPrimitivesTest.cpp | |
parent | 1ab9e59da057e02ece707d736032c756fb3ce9cf (diff) |
SkPDF: re-work SkPDFUtils::FloatToDecimal
* do a lot less floating-point math by converting to
an integer as early as possible [faster].
* round rather than truncate.
* use 8 significant digits rather than 9 when possible.
* remove trailing zeros in fractions.
before:
0.12 ! PDFScalar nonrendering
after:
0.07 ! PDFScalar nonrendering
Accuracy guaranteed by existing unit test.
Example diffs:
-/Shading <</Function <</C0 [.321568638 .333333343 .321568638]
+/Shading <</Function <</C0 [.32156864 .33333334 .32156864]
-/C1 [.258823543 .270588248 .258823543]
+/C1 [.25882354 .27058825 .25882354]
-1 0 0 -1 20 120.394500 Tm
+1 0 0 -1 20 120.394501 Tm
-1 0 0 -1 20 184.789001 Tm
+1 0 0 -1 20 184.789 Tm
-291.503997 0 l
+291.504 0 l
BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2146103004
Review-Url: https://codereview.chromium.org/2146103004
Diffstat (limited to 'tests/PDFPrimitivesTest.cpp')
-rw-r--r-- | tests/PDFPrimitivesTest.cpp | 158 |
1 files changed, 81 insertions, 77 deletions
diff --git a/tests/PDFPrimitivesTest.cpp b/tests/PDFPrimitivesTest.cpp index d816b60823..cef11503b3 100644 --- a/tests/PDFPrimitivesTest.cpp +++ b/tests/PDFPrimitivesTest.cpp @@ -27,6 +27,9 @@ #include "Test.h" #include "sk_tool_utils.h" +#include <cstdlib> +#include <cmath> + #define DUMMY_TEXT "DCT compessed stream." namespace { @@ -44,9 +47,8 @@ static SkString emit_to_string(T& obj, Catalog* catPtr = nullptr) { catPtr = &catalog; } obj.emitObject(&buffer, catPtr->numbers, catPtr->substitutes); - SkAutoTDelete<SkStreamAsset> asset(buffer.detachAsStream()); - SkString tmp(asset->getLength()); - asset->read(tmp.writable_str(), asset->getLength()); + SkString tmp(buffer.bytesWritten()); + buffer.copyTo(tmp.writable_str()); return tmp; } @@ -54,42 +56,41 @@ static bool eq(const SkString& str, const char* strPtr, size_t len) { return len == str.size() && 0 == memcmp(str.c_str(), strPtr, len); } -#define ASSERT_EQL(REPORTER, SKSTRING, STRING, LEN) \ - do { \ - const char* strptr = STRING; \ - const SkString& sks = SKSTRING; \ - if (!eq(sks, strptr, LEN)) { \ - REPORT_FAILURE( \ - REPORTER, \ - "", \ - SkStringPrintf("'%s' != '%s'", strptr, sks.c_str())); \ - } \ - } while (false) - -#define ASSERT_EQ(REPORTER, SKSTRING, STRING) \ - do { \ - const char* str = STRING; \ - ASSERT_EQL(REPORTER, SKSTRING, str, strlen(str)); \ - } while (false) +static void assert_eql(skiatest::Reporter* reporter, + const SkString& skString, + const char* str, + size_t len) { + if (!eq(skString, str, len)) { + REPORT_FAILURE(reporter, "", SkStringPrintf( + "'%*s' != '%s'", len, str, skString.c_str())); + } +} -#define ASSERT_EMIT_EQ(REPORTER, OBJECT, STRING) \ - do { \ - SkString result = emit_to_string(OBJECT); \ - ASSERT_EQ(REPORTER, result, STRING); \ - } while (false) +static void assert_eq(skiatest::Reporter* reporter, + const SkString& skString, + const char* str) { + assert_eql(reporter, skString, str, strlen(str)); +} +template <typename T> +static void assert_emit_eq(skiatest::Reporter* reporter, + T& object, + const char* string) { + SkString result = emit_to_string(object); + assert_eq(reporter, result, string); +} static void TestPDFStream(skiatest::Reporter* reporter) { char streamBytes[] = "Test\nFoo\tBar"; SkAutoTDelete<SkMemoryStream> streamData(new SkMemoryStream( streamBytes, strlen(streamBytes), true)); sk_sp<SkPDFStream> stream(new SkPDFStream(streamData.get())); - ASSERT_EMIT_EQ(reporter, + assert_emit_eq(reporter, *stream, "<</Length 12>> stream\nTest\nFoo\tBar\nendstream"); stream->insertInt("Attribute", 42); - ASSERT_EMIT_EQ(reporter, + assert_emit_eq(reporter, *stream, "<</Length 12\n/Attribute 42>> stream\n" "Test\nFoo\tBar\nendstream"); @@ -115,7 +116,7 @@ static void TestPDFStream(skiatest::Reporter* reporter) { expected.writeText("\nendstream"); SkAutoDataUnref expectedResultData2(expected.copyToData()); SkString result = emit_to_string(*stream); - ASSERT_EQL(reporter, + assert_eql(reporter, result, (const char*)expectedResultData2->data(), expectedResultData2->size()); @@ -154,7 +155,7 @@ static void TestObjectRef(skiatest::Reporter* reporter) { SkString result = emit_to_string(*a2, &catalog); // If appendObjRef misbehaves, then the result would // be [[]], not [1 0 R]. - ASSERT_EQ(reporter, result, "[1 0 R]"); + assert_eq(reporter, result, "[1 0 R]"); } static void TestSubstitute(skiatest::Reporter* reporter) { @@ -190,91 +191,94 @@ static void test_issue1083() { doc->close(); } +static void assert_emit_eq_number(skiatest::Reporter* reporter, float number) { + SkPDFUnion pdfUnion = SkPDFUnion::Scalar(number); + SkString result = emit_to_string(pdfUnion); + float value = static_cast<float>(std::atof(result.c_str())); + if (value != number) { + ERRORF(reporter, "%.9g != %s", number, result.c_str()); + } +} + + static void TestPDFUnion(skiatest::Reporter* reporter) { SkPDFUnion boolTrue = SkPDFUnion::Bool(true); - ASSERT_EMIT_EQ(reporter, boolTrue, "true"); + assert_emit_eq(reporter, boolTrue, "true"); SkPDFUnion boolFalse = SkPDFUnion::Bool(false); - ASSERT_EMIT_EQ(reporter, boolFalse, "false"); + assert_emit_eq(reporter, boolFalse, "false"); SkPDFUnion int42 = SkPDFUnion::Int(42); - ASSERT_EMIT_EQ(reporter, int42, "42"); - - SkPDFUnion realHalf = SkPDFUnion::Scalar(SK_ScalarHalf); - ASSERT_EMIT_EQ(reporter, realHalf, ".5"); - - SkPDFUnion bigScalar = SkPDFUnion::Scalar(110999.75f); - ASSERT_EMIT_EQ(reporter, bigScalar, "110999.75"); - - SkPDFUnion biggerScalar = SkPDFUnion::Scalar(50000000.1f); - ASSERT_EMIT_EQ(reporter, biggerScalar, "50000000"); + assert_emit_eq(reporter, int42, "42"); - SkPDFUnion smallestScalar = SkPDFUnion::Scalar(1.0f / 65536); - ASSERT_EMIT_EQ(reporter, smallestScalar, ".0000152587890"); + assert_emit_eq_number(reporter, SK_ScalarHalf); + assert_emit_eq_number(reporter, 110999.75f); // bigScalar + assert_emit_eq_number(reporter, 50000000.1f); // biggerScalar + assert_emit_eq_number(reporter, 1.0f / 65536); // smallScalar SkPDFUnion stringSimple = SkPDFUnion::String("test ) string ( foo"); - ASSERT_EMIT_EQ(reporter, stringSimple, "(test \\) string \\( foo)"); + assert_emit_eq(reporter, stringSimple, "(test \\) string \\( foo)"); SkString stringComplexInput("\ttest ) string ( foo"); SkPDFUnion stringComplex = SkPDFUnion::String(stringComplexInput); - ASSERT_EMIT_EQ(reporter, stringComplex, "(\\011test \\) string \\( foo)"); + assert_emit_eq(reporter, stringComplex, "(\\011test \\) string \\( foo)"); SkString binaryStringInput("\1\2\3\4\5\6\7\10\11\12\13\14\15\16\17\20"); SkPDFUnion binaryString = SkPDFUnion::String(binaryStringInput); - ASSERT_EMIT_EQ(reporter, binaryString, "<0102030405060708090A0B0C0D0E0F10>"); + assert_emit_eq(reporter, binaryString, "<0102030405060708090A0B0C0D0E0F10>"); SkString nameInput("Test name\twith#tab"); SkPDFUnion name = SkPDFUnion::Name(nameInput); - ASSERT_EMIT_EQ(reporter, name, "/Test#20name#09with#23tab"); + assert_emit_eq(reporter, name, "/Test#20name#09with#23tab"); SkString nameInput2("A#/%()<>[]{}B"); SkPDFUnion name2 = SkPDFUnion::Name(nameInput2); - ASSERT_EMIT_EQ(reporter, name2, "/A#23#2F#25#28#29#3C#3E#5B#5D#7B#7DB"); + assert_emit_eq(reporter, name2, "/A#23#2F#25#28#29#3C#3E#5B#5D#7B#7DB"); SkPDFUnion name3 = SkPDFUnion::Name("SimpleNameWithOnlyPrintableASCII"); - ASSERT_EMIT_EQ(reporter, name3, "/SimpleNameWithOnlyPrintableASCII"); + assert_emit_eq(reporter, name3, "/SimpleNameWithOnlyPrintableASCII"); // Test that we correctly handle characters with the high-bit set. SkString highBitString("\xDE\xAD" "be\xEF"); SkPDFUnion highBitName = SkPDFUnion::Name(highBitString); - ASSERT_EMIT_EQ(reporter, highBitName, "/#DE#ADbe#EF"); + assert_emit_eq(reporter, highBitName, "/#DE#ADbe#EF"); } static void TestPDFArray(skiatest::Reporter* reporter) { sk_sp<SkPDFArray> array(new SkPDFArray); - ASSERT_EMIT_EQ(reporter, *array, "[]"); + assert_emit_eq(reporter, *array, "[]"); array->appendInt(42); - ASSERT_EMIT_EQ(reporter, *array, "[42]"); + assert_emit_eq(reporter, *array, "[42]"); array->appendScalar(SK_ScalarHalf); - ASSERT_EMIT_EQ(reporter, *array, "[42 .5]"); + assert_emit_eq(reporter, *array, "[42 .5]"); array->appendInt(0); - ASSERT_EMIT_EQ(reporter, *array, "[42 .5 0]"); + assert_emit_eq(reporter, *array, "[42 .5 0]"); array->appendBool(true); - ASSERT_EMIT_EQ(reporter, *array, "[42 .5 0 true]"); + assert_emit_eq(reporter, *array, "[42 .5 0 true]"); array->appendName("ThisName"); - ASSERT_EMIT_EQ(reporter, *array, "[42 .5 0 true /ThisName]"); + assert_emit_eq(reporter, *array, "[42 .5 0 true /ThisName]"); array->appendName(SkString("AnotherName")); - ASSERT_EMIT_EQ(reporter, *array, "[42 .5 0 true /ThisName /AnotherName]"); + assert_emit_eq(reporter, *array, "[42 .5 0 true /ThisName /AnotherName]"); array->appendString("This String"); - ASSERT_EMIT_EQ(reporter, *array, + assert_emit_eq(reporter, *array, "[42 .5 0 true /ThisName /AnotherName (This String)]"); array->appendString(SkString("Another String")); - ASSERT_EMIT_EQ(reporter, *array, + assert_emit_eq(reporter, *array, "[42 .5 0 true /ThisName /AnotherName (This String) " "(Another String)]"); sk_sp<SkPDFArray> innerArray(new SkPDFArray); innerArray->appendInt(-1); array->appendObject(std::move(innerArray)); - ASSERT_EMIT_EQ(reporter, *array, + assert_emit_eq(reporter, *array, "[42 .5 0 true /ThisName /AnotherName (This String) " "(Another String) [-1]]"); @@ -286,23 +290,23 @@ static void TestPDFArray(skiatest::Reporter* reporter) { array->appendObjRef(std::move(referencedArray)); SkString result = emit_to_string(*array, &catalog); - ASSERT_EQ(reporter, result, + assert_eq(reporter, result, "[42 .5 0 true /ThisName /AnotherName (This String) " "(Another String) [-1] 1 0 R]"); } static void TestPDFDict(skiatest::Reporter* reporter) { sk_sp<SkPDFDict> dict(new SkPDFDict); - ASSERT_EMIT_EQ(reporter, *dict, "<<>>"); + assert_emit_eq(reporter, *dict, "<<>>"); dict->insertInt("n1", SkToSizeT(42)); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 42>>"); + assert_emit_eq(reporter, *dict, "<</n1 42>>"); dict.reset(new SkPDFDict); - ASSERT_EMIT_EQ(reporter, *dict, "<<>>"); + assert_emit_eq(reporter, *dict, "<<>>"); dict->insertInt("n1", 42); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 42>>"); + assert_emit_eq(reporter, *dict, "<</n1 42>>"); dict->insertScalar("n2", SK_ScalarHalf); @@ -310,37 +314,37 @@ static void TestPDFDict(skiatest::Reporter* reporter) { sk_sp<SkPDFArray> innerArray(new SkPDFArray); innerArray->appendInt(-100); dict->insertObject(n3, std::move(innerArray)); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 42\n/n2 .5\n/n3 [-100]>>"); + assert_emit_eq(reporter, *dict, "<</n1 42\n/n2 .5\n/n3 [-100]>>"); dict.reset(new SkPDFDict); - ASSERT_EMIT_EQ(reporter, *dict, "<<>>"); + assert_emit_eq(reporter, *dict, "<<>>"); dict->insertInt("n1", 24); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24>>"); + assert_emit_eq(reporter, *dict, "<</n1 24>>"); dict->insertInt("n2", SkToSizeT(99)); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24\n/n2 99>>"); + assert_emit_eq(reporter, *dict, "<</n1 24\n/n2 99>>"); dict->insertScalar("n3", SK_ScalarHalf); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5>>"); + assert_emit_eq(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5>>"); dict->insertName("n4", "AName"); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName>>"); + assert_emit_eq(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName>>"); dict->insertName("n5", SkString("AnotherName")); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName\n" + assert_emit_eq(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName\n" "/n5 /AnotherName>>"); dict->insertString("n6", "A String"); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName\n" + assert_emit_eq(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName\n" "/n5 /AnotherName\n/n6 (A String)>>"); dict->insertString("n7", SkString("Another String")); - ASSERT_EMIT_EQ(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName\n" + assert_emit_eq(reporter, *dict, "<</n1 24\n/n2 99\n/n3 .5\n/n4 /AName\n" "/n5 /AnotherName\n/n6 (A String)\n/n7 (Another String)>>"); dict.reset(new SkPDFDict("DType")); - ASSERT_EMIT_EQ(reporter, *dict, "<</Type /DType>>"); + assert_emit_eq(reporter, *dict, "<</Type /DType>>"); sk_sp<SkPDFArray> referencedArray(new SkPDFArray); Catalog catalog; @@ -349,7 +353,7 @@ static void TestPDFDict(skiatest::Reporter* reporter) { referencedArray.get()) == 1); dict->insertObjRef("n1", std::move(referencedArray)); SkString result = emit_to_string(*dict, &catalog); - ASSERT_EQ(reporter, result, "<</Type /DType\n/n1 1 0 R>>"); + assert_eq(reporter, result, "<</Type /DType\n/n1 1 0 R>>"); } DEF_TEST(PDFPrimitives, reporter) { @@ -465,7 +469,7 @@ static void check_pdf_scalar_serialization( ERRORF(reporter, "unscannable result: %s", floatString); return; } - if (isfinite(inputFloat) && roundTripFloat != inputFloat) { + if (std::isfinite(inputFloat) && roundTripFloat != inputFloat) { ERRORF(reporter, "roundTripFloat (%.9g) != inputFloat (%.9g)", roundTripFloat, inputFloat); } |