diff options
author | Matt Holgate <matt@emobix.co.uk> | 2014-06-26 15:31:16 +0100 |
---|---|---|
committer | Matt Holgate <matt@emobix.co.uk> | 2014-06-26 16:55:47 +0100 |
commit | c28e36bcc82da76274e07a2906dd994e24980180 (patch) | |
tree | 3ab39a1b757c6fe748ad06c6dfe2184d85e1e8ef | |
parent | ada560eb015e7b2208c492d47f28da8093733c4a (diff) | |
download | mupdf-c28e36bcc82da76274e07a2906dd994e24980180.tar.xz |
Fix some compiler warnings on Android.
Use intptr_t when casting between a jlong and a pointer to suppress errors
about different size words.
Add a 'u' suffix to unsigned values output by the cmap dump utility.
-rw-r--r-- | platform/android/jni/mupdf.c | 20 | ||||
-rw-r--r-- | scripts/cmapdump.c | 10 |
2 files changed, 15 insertions, 15 deletions
diff --git a/platform/android/jni/mupdf.c b/platform/android/jni/mupdf.c index 574a9d6f..927fef3c 100644 --- a/platform/android/jni/mupdf.c +++ b/platform/android/jni/mupdf.c @@ -252,7 +252,7 @@ static void alerts_fin(globals *glo) // Should only be called from the single background AsyncTask thread static globals *get_globals(JNIEnv *env, jobject thiz) { - globals *glo = (globals *)(void *)((*env)->GetLongField(env, thiz, global_fid)); + globals *glo = (globals *)(intptr_t)((*env)->GetLongField(env, thiz, global_fid)); if (glo != NULL) { glo->env = env; @@ -265,7 +265,7 @@ static globals *get_globals(JNIEnv *env, jobject thiz) // are not used. static globals *get_globals_any_thread(JNIEnv *env, jobject thiz) { - return (globals *)(void *)((*env)->GetLongField(env, thiz, global_fid)); + return (globals *)(intptr_t)((*env)->GetLongField(env, thiz, global_fid)); } JNIEXPORT jlong JNICALL @@ -340,7 +340,7 @@ JNI_FN(MuPDFCore_openFile)(JNIEnv * env, jobject thiz, jstring jfilename) (*env)->ReleaseStringUTFChars(env, jfilename, filename); - return (jlong)(void *)glo; + return (jlong)(intptr_t)glo; } typedef struct buffer_state_s @@ -482,7 +482,7 @@ JNI_FN(MuPDFCore_openBuffer)(JNIEnv * env, jobject thiz) glo = NULL; } - return (jlong)(void *)glo; + return (jlong)(intptr_t)glo; } JNIEXPORT int JNICALL @@ -651,7 +651,7 @@ JNI_FN(MuPDFCore_drawPage)(JNIEnv *env, jobject thiz, jobject bitmap, page_cache *pc = &glo->pages[glo->current]; int hq = (patchW < pageW || patchH < pageH); fz_matrix scale; - fz_cookie *cookie = (fz_cookie *)(unsigned int)cookiePtr; + fz_cookie *cookie = (fz_cookie *)(intptr_t)cookiePtr; if (pc->page == NULL) return 0; @@ -837,7 +837,7 @@ JNI_FN(MuPDFCore_updatePageInternal)(JNIEnv *env, jobject thiz, jobject bitmap, fz_document *doc = glo->doc; rect_node *crect; fz_matrix scale; - fz_cookie *cookie = (fz_cookie *)(unsigned int)cookiePtr; + fz_cookie *cookie = (fz_cookie *)(intptr_t)cookiePtr; for (i = 0; i < NUM_CACHE; i++) { @@ -853,7 +853,7 @@ JNI_FN(MuPDFCore_updatePageInternal)(JNIEnv *env, jobject thiz, jobject bitmap, /* Without a cached page object we cannot perform a partial update so render the entire bitmap instead */ JNI_FN(MuPDFCore_gotoPageInternal)(env, thiz, page); - return JNI_FN(MuPDFCore_drawPage)(env, thiz, bitmap, pageW, pageH, patchX, patchY, patchW, patchH, (jlong)(unsigned int)cookie); + return JNI_FN(MuPDFCore_drawPage)(env, thiz, bitmap, pageW, pageH, patchX, patchY, patchW, patchH, (jlong)(intptr_t)cookie); } idoc = pdf_specifics(doc); @@ -2607,13 +2607,13 @@ JNI_FN(MuPDFCore_createCookie)(JNIEnv * env, jobject thiz) return 0; fz_context *ctx = glo->ctx; - return (jlong) (unsigned int) fz_calloc_no_throw(ctx,1, sizeof(fz_cookie)); + return (jlong) (intptr_t) fz_calloc_no_throw(ctx,1, sizeof(fz_cookie)); } JNIEXPORT void JNICALL JNI_FN(MuPDFCore_destroyCookie)(JNIEnv * env, jobject thiz, jlong cookiePtr) { - fz_cookie *cookie = (fz_cookie *) (unsigned int) cookiePtr; + fz_cookie *cookie = (fz_cookie *) (intptr_t) cookiePtr; globals *glo = get_globals_any_thread(env, thiz); if (glo == NULL) return; @@ -2625,7 +2625,7 @@ JNI_FN(MuPDFCore_destroyCookie)(JNIEnv * env, jobject thiz, jlong cookiePtr) JNIEXPORT void JNICALL JNI_FN(MuPDFCore_abortCookie)(JNIEnv * env, jobject thiz, jlong cookiePtr) { - fz_cookie *cookie = (fz_cookie *) (unsigned int) cookiePtr; + fz_cookie *cookie = (fz_cookie *) (intptr_t) cookiePtr; if (cookie != NULL) cookie->abort = 1; } diff --git a/scripts/cmapdump.c b/scripts/cmapdump.c index 8d5cae2b..21052950 100644 --- a/scripts/cmapdump.c +++ b/scripts/cmapdump.c @@ -107,7 +107,7 @@ main(int argc, char **argv) { if (k % 4 == 0) fprintf(fo, "\n"); - fprintf(fo, "{%u,%u,%u},", cmap->ranges[k].low, cmap->ranges[k].high, cmap->ranges[k].out); + fprintf(fo, "{%uu,%uu,%uu},", cmap->ranges[k].low, cmap->ranges[k].high, cmap->ranges[k].out); } fprintf(fo, "\n};\n\n"); } @@ -119,7 +119,7 @@ main(int argc, char **argv) { if (k % 4 == 0) fprintf(fo, "\n"); - fprintf(fo, "{%u,%u,%u},", cmap->xranges[k].low, cmap->xranges[k].high, cmap->xranges[k].out); + fprintf(fo, "{%uu,%uu,%uu},", cmap->xranges[k].low, cmap->xranges[k].high, cmap->xranges[k].out); } fprintf(fo, "\n};\n\n"); } @@ -129,9 +129,9 @@ main(int argc, char **argv) fprintf(fo, "static const pdf_mrange cmap_%s_mranges[] = {", name); for (k = 0; k < cmap->mlen; k++) { - fprintf(fo, "\n{%u,%u,{", cmap->mranges[k].low, cmap->mranges[k].len); + fprintf(fo, "\n{%uu,%uu,{", cmap->mranges[k].low, cmap->mranges[k].len); for (m = 0; m < PDF_MRANGE_CAP; ++m) - fprintf(fo, "%u,", cmap->mranges[k].out[m]); + fprintf(fo, "%uu,", cmap->mranges[k].out[m]); fprintf(fo, "}},"); } fprintf(fo, "\n};\n\n"); @@ -149,7 +149,7 @@ main(int argc, char **argv) } for (k = 0; k < cmap->codespace_len; k++) { - fprintf(fo, "{%u,%u,%u},", cmap->codespace[k].n, cmap->codespace[k].low, cmap->codespace[k].high); + fprintf(fo, "{%u,%uu,%uu},", cmap->codespace[k].n, cmap->codespace[k].low, cmap->codespace[k].high); } fprintf(fo, " },\n"); |