From 4b9d69806c30775f55563e72b0842daa06483f99 Mon Sep 17 00:00:00 2001 From: Ryan Harrison Date: Fri, 26 Jan 2018 17:11:51 +0000 Subject: Clean up construction of CFX_Barcode Because certain enum values can cause the initialization of the class to fail there is a seperate init method from the constructor. This CL is converting the code to use a standard factory pattern for this, instead of the existing implementation. Change-Id: Ia2293ce94ad0db5862db9796aeb8a224fd2b45f9 Reviewed-on: https://pdfium-review.googlesource.com/24230 Reviewed-by: Henrique Nakashima Commit-Queue: Ryan Harrison --- testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc') diff --git a/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc b/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc index 3334e86bfc..aaeebf3ace 100644 --- a/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc +++ b/testing/libfuzzer/pdf_cfx_barcode_fuzzer.cc @@ -17,20 +17,20 @@ extern "C" int LLVMFuzzerTestOneInput(const uint8_t* data, size_t size) { data += sizeof(wchar_t); size -= sizeof(wchar_t); - CFX_Barcode barcode; - if (!barcode.Create(type)) + auto barcode = CFX_Barcode::Create(type); + if (!barcode) return 0; // TODO(tsepez): Setup more options from |data|. - barcode.SetModuleHeight(300); - barcode.SetModuleWidth(420); - barcode.SetHeight(298); - barcode.SetWidth(418); + barcode->SetModuleHeight(300); + barcode->SetModuleWidth(420); + barcode->SetHeight(298); + barcode->SetWidth(418); WideStringView content(reinterpret_cast(data), size / sizeof(wchar_t)); - if (!barcode.Encode(content)) + if (!barcode->Encode(content)) return 0; // TODO(tsepez): Output to device. -- cgit v1.2.3