From dc094186605e566bf063211cd2f23e921c88c5d4 Mon Sep 17 00:00:00 2001 From: Michael R Sweet Date: Thu, 25 Sep 2025 17:20:20 -0400 Subject: [PATCH] More clang warning fixes. --- pdfio-array.c | 4 ++-- pdfio-common.c | 12 ++++++------ pdfio-content.c | 2 +- pdfio-crypto.c | 6 ++++-- pdfio-file.c | 26 +++++++++++++------------- pdfio-object.c | 10 +++++----- pdfio-page.c | 2 +- pdfio-stream.c | 4 ++-- pdfio-string.c | 2 +- pdfio-token.c | 6 +++--- pdfio-value.c | 4 ++-- pdfio.xcodeproj/project.pbxproj | 4 ---- testpdfio.c | 7 ++++++- 13 files changed, 46 insertions(+), 43 deletions(-) diff --git a/pdfio-array.c b/pdfio-array.c index de2dd30..84f96e7 100644 --- a/pdfio-array.c +++ b/pdfio-array.c @@ -262,7 +262,7 @@ pdfioArrayCopy(pdfio_file_t *pdf, // I - PDF file vdst; // Current destination value - PDFIO_DEBUG("pdfioArrayCopy(pdf=%p, a=%p(%p))\n", pdf, a, a ? a->pdf : NULL); + PDFIO_DEBUG("pdfioArrayCopy(pdf=%p, a=%p(%p))\n", (void *)pdf, (void *)a, a ? (void *)a->pdf : NULL); // Create the new array... if ((na = pdfioArrayCreate(pdf)) == NULL) @@ -606,7 +606,7 @@ _pdfioArrayRead(pdfio_file_t *pdf, // I - PDF file _pdfio_value_t value; // Value - PDFIO_DEBUG("_pdfioArrayRead(pdf=%p, tb=%p)\n", pdf, tb); + PDFIO_DEBUG("_pdfioArrayRead(pdf=%p, tb=%p)\n", (void *)pdf, (void *)tb); // Create an array... if ((array = pdfioArrayCreate(pdf)) == NULL) diff --git a/pdfio-common.c b/pdfio-common.c index e88b081..c6889aa 100644 --- a/pdfio-common.c +++ b/pdfio-common.c @@ -27,7 +27,7 @@ bool // O - `true` on sucess, `false` on EOF _pdfioFileConsume(pdfio_file_t *pdf, // I - PDF file size_t bytes) // I - Bytes to consume { - PDFIO_DEBUG("_pdfioFileConsume(pdf=%p, bytes=%u)\n", pdf, (unsigned)bytes); + PDFIO_DEBUG("_pdfioFileConsume(pdf=%p, bytes=%u)\n", (void *)pdf, (unsigned)bytes); if ((size_t)(pdf->bufend - pdf->bufptr) > bytes) pdf->bufptr += bytes; @@ -91,7 +91,7 @@ _pdfioFileError(pdfio_file_t *pdf, // I - PDF file bool // O - `true` on success, `false` on failure _pdfioFileFlush(pdfio_file_t *pdf) // I - PDF file { - PDFIO_DEBUG("_pdfioFileFlush(pdf=%p)\n", pdf); + PDFIO_DEBUG("_pdfioFileFlush(pdf=%p)\n", (void *)pdf); if (pdf->bufptr > pdf->buffer) { @@ -142,7 +142,7 @@ _pdfioFileGets(pdfio_file_t *pdf, // I - PDF file *bufend = buffer + bufsize - 1; // Pointer to end of buffer - PDFIO_DEBUG("_pdfioFileGets(pdf=%p, buffer=%p, bufsize=%lu, discard=%s) bufpos=%ld, buffer=%p, bufptr=%p, bufend=%p, offset=%lu\n", pdf, buffer, (unsigned long)bufsize, discard ? "true" : "false", (long)pdf->bufpos, pdf->buffer, pdf->bufptr, pdf->bufend, (unsigned long)(pdf->bufpos + (pdf->bufptr - pdf->buffer))); + PDFIO_DEBUG("_pdfioFileGets(pdf=%p, buffer=%p, bufsize=%lu, discard=%s) bufpos=%ld, buffer=%p, bufptr=%p, bufend=%p, offset=%lu\n", (void *)pdf, (void *)buffer, (unsigned long)bufsize, discard ? "true" : "false", (long)pdf->bufpos, (void *)pdf->buffer, (void *)pdf->bufptr, (void *)pdf->bufend, (unsigned long)(pdf->bufpos + (pdf->bufptr - pdf->buffer))); while (!eol) { @@ -359,7 +359,7 @@ _pdfioFileSeek(pdfio_file_t *pdf, // I - PDF file off_t offset, // I - Offset int whence) // I - Offset base { - PDFIO_DEBUG("_pdfioFileSeek(pdf=%p, offset=%ld, whence=%d) pdf->bufpos=%lu\n", pdf, (long)offset, whence, (unsigned long)(pdf ? pdf->bufpos : 0)); + PDFIO_DEBUG("_pdfioFileSeek(pdf=%p, offset=%ld, whence=%d) pdf->bufpos=%lu\n", (void *)pdf, (long)offset, whence, (unsigned long)(pdf ? pdf->bufpos : 0)); // Adjust offset for relative seeks... if (whence == SEEK_CUR) @@ -376,7 +376,7 @@ _pdfioFileSeek(pdfio_file_t *pdf, // I - PDF file // Yes, seek within existing buffer... pdf->bufptr = pdf->buffer + (offset - pdf->bufpos); PDFIO_DEBUG("_pdfioFileSeek: Seek within buffer, bufpos=%ld.\n", (long)pdf->bufpos); - PDFIO_DEBUG("_pdfioFileSeek: buffer=%p, bufptr=%p(<%02X%02X...>), bufend=%p\n", pdf->buffer, pdf->bufptr, pdf->bufptr[0] & 255, pdf->bufptr[1] & 255, pdf->bufend); + PDFIO_DEBUG("_pdfioFileSeek: buffer=%p, bufptr=%p(<%02X%02X...>), bufend=%p\n", (void *)pdf->buffer, (void *)pdf->bufptr, pdf->bufptr[0] & 255, pdf->bufptr[1] & 255, (void *)pdf->bufend); return (offset); } @@ -411,7 +411,7 @@ _pdfioFileSeek(pdfio_file_t *pdf, // I - PDF file } PDFIO_DEBUG("_pdfioFileSeek: Reset bufpos=%ld, offset=%lu.\n", (long)pdf->bufpos, (unsigned long)offset); - PDFIO_DEBUG("_pdfioFileSeek: buffer=%p, bufptr=%p, bufend=%p\n", pdf->buffer, pdf->bufptr, pdf->bufend); + PDFIO_DEBUG("_pdfioFileSeek: buffer=%p, bufptr=%p, bufend=%p\n", (void *)pdf->buffer, (void *)pdf->bufptr, (void *)pdf->bufend); pdf->bufpos = offset; diff --git a/pdfio-content.c b/pdfio-content.c index d3c368a..0bcbd7c 100644 --- a/pdfio-content.c +++ b/pdfio-content.c @@ -355,7 +355,7 @@ pdfioArrayCreateColorFromPrimaries( double matrix[3][3]; // CIE XYZ transform matrix - PDFIO_DEBUG("pdfioFileCreateCalibratedColorFromPrimaries(pdf=%p, num_colors=%lu, gamma=%.6f, wx=%.6f, wy=%.6f, rx=%.6f, ry=%.6f, gx=%.6f, gy=%.6f, bx=%.6f, by=%.6f)\n", pdf, (unsigned long)num_colors, gamma, wx, wy, rx, ry, gx, gy, bx, by); + PDFIO_DEBUG("pdfioFileCreateCalibratedColorFromPrimaries(pdf=%p, num_colors=%lu, gamma=%.6f, wx=%.6f, wy=%.6f, rx=%.6f, ry=%.6f, gx=%.6f, gy=%.6f, bx=%.6f, by=%.6f)\n", (void *)pdf, (unsigned long)num_colors, gamma, wx, wy, rx, ry, gx, gy, bx, by); // Range check input... if (!pdf || (num_colors != 1 && num_colors != 3) || gamma <= 0.0 || ry == 0.0 || gy == 0.0 || by == 0.0) diff --git a/pdfio-crypto.c b/pdfio-crypto.c index 53859bb..3aca546 100644 --- a/pdfio-crypto.c +++ b/pdfio-crypto.c @@ -411,7 +411,7 @@ _pdfioCryptoMakeReader( uint8_t digest[16]; // MD5 digest value - PDFIO_DEBUG("_pdfioCryptoMakeReader(pdf=%p, obj=%p(%d), ctx=%p, iv=%p, ivlen=%p(%d))\n", pdf, obj, (int)obj->number, ctx, iv, ivlen, (int)*ivlen); + PDFIO_DEBUG("_pdfioCryptoMakeReader(pdf=%p, obj=%p(%d), ctx=%p, iv=%p, ivlen=%p(%d))\n", (void *)pdf, (void *)obj, (int)obj->number, (void *)ctx, (void *)iv, (void *)ivlen, (int)*ivlen); // Range check input... if (!pdf) @@ -454,6 +454,8 @@ _pdfioCryptoMakeReader( return (NULL); } + __attribute__((fallthrough)); + case PDFIO_ENCRYPTION_RC4_128 : // Copy the key data for the MD5 hash. memcpy(data, pdf->file_key, 16); @@ -504,7 +506,7 @@ _pdfioCryptoMakeWriter( uint8_t digest[16]; /* MD5 digest value */ - PDFIO_DEBUG("_pdfioCryptoMakeWriter(pdf=%p, obj=%p(%d), ctx=%p, iv=%p, ivlen=%p(%d))\n", pdf, obj, (int)obj->number, ctx, iv, ivlen, (int)*ivlen); + PDFIO_DEBUG("_pdfioCryptoMakeWriter(pdf=%p, obj=%p(%d), ctx=%p, iv=%p, ivlen=%p(%d))\n", (void *)pdf, (void *)obj, (int)obj->number, (void *)ctx, (void *)iv, (void *)ivlen, (int)*ivlen); // Range check input... if (!pdf) diff --git a/pdfio-file.c b/pdfio-file.c index c44ee8a..31fd396 100644 --- a/pdfio-file.c +++ b/pdfio-file.c @@ -720,7 +720,7 @@ pdfioFileFindObj( if (number == pdf->objs[current]->number) { // Fast match... - PDFIO_DEBUG("pdfioFileFindObj: Returning %lu (%p)\n", (unsigned long)current, pdf->objs[current]); + PDFIO_DEBUG("pdfioFileFindObj: Returning %lu (%p)\n", (unsigned long)current, (void *)pdf->objs[current]); return (pdf->objs[current]); } else if (number < pdf->objs[current]->number) @@ -748,12 +748,12 @@ pdfioFileFindObj( if (number == pdf->objs[left]->number) { - PDFIO_DEBUG("pdfioFileFindObj: Returning %lu (%p)\n", (unsigned long)left, pdf->objs[left]); + PDFIO_DEBUG("pdfioFileFindObj: Returning %lu (%p)\n", (unsigned long)left, (void *)pdf->objs[left]); return (pdf->objs[left]); } else if (number == pdf->objs[right]->number) { - PDFIO_DEBUG("pdfioFileFindObj: Returning %lu (%p)\n", (unsigned long)right, pdf->objs[right]); + PDFIO_DEBUG("pdfioFileFindObj: Returning %lu (%p)\n", (unsigned long)right, (void *)pdf->objs[right]); return (pdf->objs[right]); } else @@ -1115,7 +1115,7 @@ pdfioFileOpen( } else { - PDFIO_DEBUG("pdfioFileOpen: line=%p,ptr=%p(\"%s\")\n", line, ptr, ptr); + PDFIO_DEBUG("pdfioFileOpen: line=%p,ptr=%p(\"%s\")\n", (void *)line, (void *)ptr, ptr); xref_offset = (off_t)strtol(ptr + 9, NULL, 10); @@ -1326,7 +1326,7 @@ add_obj(pdfio_file_t *pdf, // I - PDF file obj->generation = generation; obj->offset = offset; - PDFIO_DEBUG("add_obj: obj=%p, ->pdf=%p, ->number=%lu, ->offset=%lu\n", obj, pdf, (unsigned long)obj->number, (unsigned long)offset); + PDFIO_DEBUG("add_obj: obj=%p, ->pdf=%p, ->number=%lu, ->offset=%lu\n", (void *)obj, (void *)pdf, (unsigned long)obj->number, (unsigned long)offset); // Insert object into array as needed... if (pdf->num_objs == 0 || obj->number > pdf->objs[pdf->num_objs - 1]->number) @@ -1596,7 +1596,7 @@ get_lconv(void) if ((loc = localeconv()) != NULL) { - PDFIO_DEBUG("get_lconv: loc=%p, loc->decimal_point=\"%s\"\n", loc, loc->decimal_point); + PDFIO_DEBUG("get_lconv: loc=%p, loc->decimal_point=\"%s\"\n", (void *)loc, loc->decimal_point); if (!loc->decimal_point || !strcmp(loc->decimal_point, ".")) loc = NULL; @@ -1633,7 +1633,7 @@ load_obj_stream(pdfio_obj_t *obj) // I - Object to load int count; // Count of objects - PDFIO_DEBUG("load_obj_stream(obj=%p(%d))\n", obj, (int)obj->number); + PDFIO_DEBUG("load_obj_stream(obj=%p(%d))\n", (void *)obj, (int)obj->number); // Open the object stream... if ((st = pdfioObjOpenStream(obj, true)) == NULL) @@ -1900,7 +1900,7 @@ load_xref( goto repair; } - PDFIO_DEBUG("load_xref: tb.bufptr=%p, tb.bufend=%p, tb.bufptr[0]=0x%02x, tb.bufptr[0]=0x%02x\n", tb.bufptr, tb.bufend, tb.bufptr[0], tb.bufptr[1]); + PDFIO_DEBUG("load_xref: tb.bufptr=%p, tb.bufend=%p, tb.bufptr[0]=0x%02x, tb.bufptr[0]=0x%02x\n", (void *)tb.bufptr, (void *)tb.bufend, tb.bufptr[0], tb.bufptr[1]); if (tb.bufptr && tb.bufptr < tb.bufend && (tb.bufptr[0] == 0x0d || tb.bufptr[0] == 0x0a)) tb.bufptr ++; // Skip trailing CR or LF after token @@ -2093,7 +2093,7 @@ load_xref( { if ((obj = pdfioFileFindObj(pdf, sobjs[i])) != NULL) { - PDFIO_DEBUG("load_xref: Loading compressed object stream %lu (pdf=%p, obj->pdf=%p).\n", (unsigned long)sobjs[i], pdf, obj->pdf); + PDFIO_DEBUG("load_xref: Loading compressed object stream %lu (pdf=%p, obj->pdf=%p).\n", (unsigned long)sobjs[i], (void *)pdf, (void *)obj->pdf); if (!load_obj_stream(obj)) return (false); @@ -2272,7 +2272,7 @@ load_xref( goto repair; } - PDFIO_DEBUG("load_xref: Root=%p(%lu)\n", pdf->root_obj, (unsigned long)pdf->root_obj->number); + PDFIO_DEBUG("load_xref: Root=%p(%lu)\n", (void *)pdf->root_obj, (unsigned long)pdf->root_obj->number); if ((pages_obj = pdfioDictGetObj(pdfioObjGetDict(pdf->root_obj), "Pages")) == NULL) { @@ -2280,7 +2280,7 @@ load_xref( goto repair; } - PDFIO_DEBUG("load_xref: Pages=%p(%lu)\n", pdf->root_obj, (unsigned long)pdf->root_obj->number); + PDFIO_DEBUG("load_xref: Pages=%p(%lu)\n", (void *)pages_obj, (unsigned long)pages_obj->number); return (load_pages(pdf, pages_obj, 0)); @@ -2494,7 +2494,7 @@ repair_xref( return (false); } - PDFIO_DEBUG("repair_xref: Root=%p(%lu)\n", pdf->root_obj, (unsigned long)pdf->root_obj->number); + PDFIO_DEBUG("repair_xref: Root=%p(%lu)\n", (void *)pdf->root_obj, (unsigned long)pdf->root_obj->number); if ((pages_obj = pdfioDictGetObj(pdfioObjGetDict(pdf->root_obj), "Pages")) == NULL) { @@ -2502,7 +2502,7 @@ repair_xref( return (false); } - PDFIO_DEBUG("repair_xref: Pages=%p(%lu)\n", pages_obj, (unsigned long)pages_obj->number); + PDFIO_DEBUG("repair_xref: Pages=%p(%lu)\n", (void *)pages_obj, (unsigned long)pages_obj->number); // Load pages... return (load_pages(pdf, pages_obj, 0)); diff --git a/pdfio-object.c b/pdfio-object.c index 8f13fbd..b480404 100644 --- a/pdfio-object.c +++ b/pdfio-object.c @@ -69,7 +69,7 @@ pdfioObjCopy(pdfio_file_t *pdf, // I - PDF file ssize_t bytes; // Bytes read - PDFIO_DEBUG("pdfioObjCopy(pdf=%p, srcobj=%p(%p))\n", pdf, srcobj, srcobj ? srcobj->pdf : NULL); + PDFIO_DEBUG("pdfioObjCopy(pdf=%p, srcobj=%p(%p))\n", (void *)pdf, (void *)srcobj, srcobj ? (void *)srcobj->pdf : NULL); // Range check input if (!pdf || !srcobj) @@ -308,7 +308,7 @@ pdfioObjGetLength(pdfio_obj_t *obj) // I - Object // Try getting the length, directly or indirectly if ((length = (size_t)pdfioDictGetNumber(obj->value.value.dict, "Length")) > 0) { - PDFIO_DEBUG("pdfioObjGetLength(obj=%p) returning %lu.\n", obj, (unsigned long)length); + PDFIO_DEBUG("pdfioObjGetLength(obj=%p) returning %lu.\n", (void *)obj, (unsigned long)length); return (length); } @@ -328,7 +328,7 @@ pdfioObjGetLength(pdfio_obj_t *obj) // I - Object return (0); } - PDFIO_DEBUG("pdfioObjGetLength(obj=%p) returning %lu.\n", obj, (unsigned long)lenobj->value.value.number); + PDFIO_DEBUG("pdfioObjGetLength(obj=%p) returning %lu.\n", (void *)obj, (unsigned long)lenobj->value.value.number); return ((size_t)lenobj->value.value.number); } @@ -440,7 +440,7 @@ _pdfioObjLoad(pdfio_obj_t *obj) // I - Object _pdfio_token_t tb; // Token buffer/stack - PDFIO_DEBUG("_pdfioObjLoad(obj=%p(%lu)), offset=%lu\n", obj, (unsigned long)obj->number, (unsigned long)obj->offset); + PDFIO_DEBUG("_pdfioObjLoad(obj=%p(%lu)), offset=%lu\n", (void *)obj, (unsigned long)obj->number, (unsigned long)obj->offset); // Seek to the start of the object and read its header... if (_pdfioFileSeek(obj->pdf, obj->offset, SEEK_SET) != obj->offset) @@ -502,7 +502,7 @@ _pdfioObjLoad(pdfio_obj_t *obj) // I - Object return (false); } - PDFIO_DEBUG("_pdfioObjLoad: tb.bufptr=%p, tb.bufend=%p, tb.bufptr[0]=0x%02x, tb.bufptr[1]=0x%02x\n", tb.bufptr, tb.bufend, tb.bufptr[0], tb.bufptr[1]); + PDFIO_DEBUG("_pdfioObjLoad: tb.bufptr=%p, tb.bufend=%p, tb.bufptr[0]=0x%02x, tb.bufptr[1]=0x%02x\n", (void *)tb.bufptr, (void *)tb.bufend, tb.bufptr[0], tb.bufptr[1]); _pdfioTokenFlush(&tb); diff --git a/pdfio-page.c b/pdfio-page.c index b350bd6..94e6ff8 100644 --- a/pdfio-page.c +++ b/pdfio-page.c @@ -28,7 +28,7 @@ pdfioPageCopy(pdfio_file_t *pdf, // I - PDF file pdfio_obj_t *dstpage; // Destination page object - PDFIO_DEBUG("pdfioPageCopy(pdf=%p, srcpage=%p(%p))\n", pdf, srcpage, srcpage ? srcpage->pdf : NULL); + PDFIO_DEBUG("pdfioPageCopy(pdf=%p, srcpage=%p(%p))\n", (void *)pdf, (void *)srcpage, srcpage ? (void *)srcpage->pdf : NULL); // Range check input if (!pdf || !srcpage || srcpage->value.type != PDFIO_VALTYPE_DICT) diff --git a/pdfio-stream.c b/pdfio-stream.c index 808e454..67a534c 100644 --- a/pdfio-stream.c +++ b/pdfio-stream.c @@ -427,7 +427,7 @@ _pdfioStreamOpen(pdfio_obj_t *obj, // I - Object const char *type; // Object type - PDFIO_DEBUG("_pdfioStreamOpen(obj=%p(%u), decode=%s)\n", obj, (unsigned)obj->number, decode ? "true" : "false"); + PDFIO_DEBUG("_pdfioStreamOpen(obj=%p(%u), decode=%s)\n", (void *)obj, (unsigned)obj->number, decode ? "true" : "false"); // Allocate a new stream object... if ((st = (pdfio_stream_t *)calloc(1, sizeof(pdfio_stream_t))) == NULL) @@ -848,7 +848,7 @@ pdfioStreamWrite( *pptr; // Previous raw buffer - PDFIO_DEBUG("pdfioStreamWrite(st=%p, buffer=%p, bytes=%lu)\n", st, buffer, (unsigned long)bytes); + PDFIO_DEBUG("pdfioStreamWrite(st=%p, buffer=%p, bytes=%lu)\n", (void *)st, (void *)buffer, (unsigned long)bytes); // Range check input... if (!st || st->pdf->mode != _PDFIO_MODE_WRITE || !buffer || !bytes) diff --git a/pdfio-string.c b/pdfio-string.c index d8033de..8790bc4 100644 --- a/pdfio-string.c +++ b/pdfio-string.c @@ -723,7 +723,7 @@ pdfioStringCreate( int diff; // Different - PDFIO_DEBUG("pdfioStringCreate(pdf=%p, s=\"%s\")\n", pdf, s); + PDFIO_DEBUG("pdfioStringCreate(pdf=%p, s=\"%s\")\n", (void *)pdf, s); // Range check input... if (!pdf || !s) diff --git a/pdfio-token.c b/pdfio-token.c index dd9cef6..c13adb8 100644 --- a/pdfio-token.c +++ b/pdfio-token.c @@ -54,7 +54,7 @@ static int get_char(_pdfio_token_t *tb); void _pdfioTokenClear(_pdfio_token_t *tb) // I - Token buffer/stack { - PDFIO_DEBUG("_pdfioTokenClear(tb=%p)\n", tb); + PDFIO_DEBUG("_pdfioTokenClear(tb=%p)\n", (void *)tb); while (tb->num_tokens > 0) { @@ -132,7 +132,7 @@ _pdfioTokenGet(_pdfio_token_t *tb, // I - Token buffer/stack if ((len = strlen(tb->tokens[tb->num_tokens])) > (bufsize - 1)) { // Value too large... - PDFIO_DEBUG("_pdfioTokenGet(tb=%p, buffer=%p, bufsize=%u): Token '%s' from stack too large.\n", tb, buffer, (unsigned)bufsize, tb->tokens[tb->num_tokens]); + PDFIO_DEBUG("_pdfioTokenGet(tb=%p, buffer=%p, bufsize=%u): Token '%s' from stack too large.\n", (void *)tb, (void *)buffer, (unsigned)bufsize, tb->tokens[tb->num_tokens]); *buffer = '\0'; return (false); } @@ -140,7 +140,7 @@ _pdfioTokenGet(_pdfio_token_t *tb, // I - Token buffer/stack memcpy(buffer, tb->tokens[tb->num_tokens], len); buffer[len] = '\0'; - PDFIO_DEBUG("_pdfioTokenGet(tb=%p, buffer=%p, bufsize=%u): Popping '%s' from stack.\n", tb, buffer, (unsigned)bufsize, buffer); + PDFIO_DEBUG("_pdfioTokenGet(tb=%p, buffer=%p, bufsize=%u): Popping '%s' from stack.\n", (void *)tb, (void *)buffer, (unsigned)bufsize, buffer); free(tb->tokens[tb->num_tokens]); tb->tokens[tb->num_tokens] = NULL; diff --git a/pdfio-value.c b/pdfio-value.c index f4051f1..46bb234 100644 --- a/pdfio-value.c +++ b/pdfio-value.c @@ -46,7 +46,7 @@ _pdfioValueCopy(pdfio_file_t *pdfdst, // I - Destination PDF file #endif // DEBUG - PDFIO_DEBUG("_pdfioValueCopy(pdfdst=%p, vdst=%p, pdfsrc=%p, vsrc=%p(%s))\n", pdfdst, vdst, pdfsrc, vsrc, types[vsrc->type]); + PDFIO_DEBUG("_pdfioValueCopy(pdfdst=%p, vdst=%p, pdfsrc=%p, vsrc=%p(%s))\n", (void *)pdfdst, (void *)vdst, (void *)pdfsrc, (void *)vsrc, types[vsrc->type]); if (pdfdst == pdfsrc && vsrc->type != PDFIO_VALTYPE_BINARY) { @@ -359,7 +359,7 @@ _pdfioValueRead(pdfio_file_t *pdf, // I - PDF file #endif // DEBUG - PDFIO_DEBUG("_pdfioValueRead(pdf=%p, obj=%p, v=%p)\n", pdf, obj, v); + PDFIO_DEBUG("_pdfioValueRead(pdf=%p, obj=%p, v=%p)\n", (void *)pdf, (void *)obj, (void *)v); if (!token) goto done; diff --git a/pdfio.xcodeproj/project.pbxproj b/pdfio.xcodeproj/project.pbxproj index 96ab66a..8105c08 100644 --- a/pdfio.xcodeproj/project.pbxproj +++ b/pdfio.xcodeproj/project.pbxproj @@ -503,7 +503,6 @@ GCC_TREAT_INCOMPATIBLE_POINTER_TYPE_WARNINGS_AS_ERRORS = YES; GCC_TREAT_WARNINGS_AS_ERRORS = YES; GCC_WARN_ABOUT_MISSING_NEWLINE = YES; - GCC_WARN_PEDANTIC = YES; PRODUCT_NAME = "$(TARGET_NAME)"; SKIP_INSTALL = YES; }; @@ -520,7 +519,6 @@ EXECUTABLE_PREFIX = lib; GCC_TREAT_INCOMPATIBLE_POINTER_TYPE_WARNINGS_AS_ERRORS = YES; GCC_WARN_ABOUT_MISSING_NEWLINE = YES; - GCC_WARN_PEDANTIC = YES; PRODUCT_NAME = "$(TARGET_NAME)"; SKIP_INSTALL = YES; }; @@ -540,7 +538,6 @@ GCC_OPTIMIZATION_LEVEL = 0; GCC_TREAT_INCOMPATIBLE_POINTER_TYPE_WARNINGS_AS_ERRORS = YES; GCC_TREAT_WARNINGS_AS_ERRORS = YES; - GCC_WARN_PEDANTIC = YES; GCC_WARN_UNKNOWN_PRAGMAS = YES; MACOSX_DEPLOYMENT_TARGET = 11.0; PRODUCT_BUNDLE_IDENTIFIER = org.msweet.testpdfio; @@ -560,7 +557,6 @@ DEVELOPMENT_TEAM = ""; ENABLE_HARDENED_RUNTIME = YES; GCC_TREAT_INCOMPATIBLE_POINTER_TYPE_WARNINGS_AS_ERRORS = YES; - GCC_WARN_PEDANTIC = YES; GCC_WARN_UNKNOWN_PRAGMAS = YES; MACOSX_DEPLOYMENT_TARGET = 11.0; PRODUCT_BUNDLE_IDENTIFIER = org.msweet.testpdfio; diff --git a/testpdfio.c b/testpdfio.c index 562b65e..b7770d3 100644 --- a/testpdfio.c +++ b/testpdfio.c @@ -485,7 +485,7 @@ do_test_file(const char *filename, // I - PDF filename { dict = pdfioObjGetDict(obj); - printf(" %u %u obj dict=%p(%lu pairs)\n", (unsigned)pdfioObjGetNumber(obj), (unsigned)pdfioObjGetGeneration(obj), dict, dict ? (unsigned long)dict->num_pairs : 0UL); + printf(" %u %u obj dict=%p(%lu pairs)\n", (unsigned)pdfioObjGetNumber(obj), (unsigned)pdfioObjGetGeneration(obj), (void *)dict, dict ? (unsigned long)dict->num_pairs : 0UL); fputs(" ", stdout); _pdfioValueDebug(&obj->value, stdout); putchar('\n'); @@ -3502,9 +3502,14 @@ write_text_test(pdfio_file_t *pdf, // I - PDF file fputs("pdfioStreamClose: ", stdout); if (pdfioStreamClose(st)) + { puts("PASS"); + } else + { + fclose(fp); return (1); + } } fclose(fp);