dany/libwebp
1
0
Fork 0
mirror of https://github.com/webmproject/libwebp.git synced 2025-10-30 18:05:36 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

Merge changes I466c377f,Ib761ebd3,I694857fc into 0.3.0

Browse Source 
* changes:
  gif2webp: only write error messages to stderr
  gif2webp: fix crash on open failure with libgif5
  gif2webp: silence a unused param warning
This commit is contained in:
James Zern
2013-03-18 16:52:28 -07:00
committed by Gerrit Code Review
parent bb85b43727 723847d5d7
commit 8a89c6ed28
1 changed files with 26 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

43
examples/gif2webp.c
View File

@@ -132,21 +132,24 @@ static int GetColorFromIndex(const ColorMapObject* const color_map, GifWord idx,
} }
} }
static void DisplayGifError(const GifFileType* const gif) { static void DisplayGifError(const GifFileType* const gif, int gif_error) {
// GIFLIB_MAJOR is only defined in libgif >= 4.2.0. // GIFLIB_MAJOR is only defined in libgif >= 4.2.0.
// libgif 4.2.0 has retired PrintGifError() and added GifErrorString(). // libgif 4.2.0 has retired PrintGifError() and added GifErrorString().
#if defined(GIFLIB_MAJOR) && defined(GIFLIB_MINOR) && \ #if defined(GIFLIB_MAJOR) && defined(GIFLIB_MINOR) && \
((GIFLIB_MAJOR == 4 && GIFLIB_MINOR >= 2) || GIFLIB_MAJOR > 4) ((GIFLIB_MAJOR == 4 && GIFLIB_MINOR >= 2) || GIFLIB_MAJOR > 4)
#if GIFLIB_MAJOR >= 5 #if GIFLIB_MAJOR >= 5
// Static string actually, hence the const char* cast. // Static string actually, hence the const char* cast.
const char* error_str = (const char*)GifErrorString(gif->Error); const char* error_str = (const char*)GifErrorString(
(gif == NULL) ? gif_error : gif->Error);
#else #else
const char* error_str = (const char*)GifErrorString(); const char* error_str = (const char*)GifErrorString();
(void)gif;
#endif #endif
if (error_str == NULL) error_str = "Unknown error"; if (error_str == NULL) error_str = "Unknown error";
fprintf(stderr, "GIFLib Error: %s\n", error_str); fprintf(stderr, "GIFLib Error %d: %s\n", gif_error, error_str);
#else #else
fprintf(stderr, "GIFLib Error: "); (void)gif;
fprintf(stderr, "GIFLib Error %d: ", gif_error);
PrintGifError(); PrintGifError();
fprintf(stderr, "\n"); fprintf(stderr, "\n");
#endif #endif
@@ -184,7 +187,7 @@ static void Help(void) {
int main(int argc, const char *argv[]) { int main(int argc, const char *argv[]) {
int verbose = 0; int verbose = 0;
int gif_error = -1; int gif_error = GIF_ERROR;
WebPMuxError err = WEBP_MUX_OK; WebPMuxError err = WEBP_MUX_OK;
int ok = 0; int ok = 0;
const char *in_file = NULL, *out_file = NULL; const char *in_file = NULL, *out_file = NULL;
@@ -340,11 +343,11 @@ int main(int argc, const char *argv[]) {
goto End; goto End;
} }
if (verbose) { if (verbose) {
fprintf(stderr, "Added frame %dx%d (offset:%d,%d duration:%d) ", printf("Added frame %dx%d (offset:%d,%d duration:%d) ",
view.width, view.height, frame.x_offset, frame.y_offset, view.width, view.height, frame.x_offset, frame.y_offset,
frame.duration); frame.duration);
fprintf(stderr, "dispose:%d transparent index:%d\n", printf("dispose:%d transparent index:%d\n",
frame.dispose_method, transparent_index); frame.dispose_method, transparent_index);
} }
WebPDataClear(&frame.bitstream); WebPDataClear(&frame.bitstream);
break; break;
@@ -389,7 +392,7 @@ int main(int argc, const char *argv[]) {
if (data == NULL) goto End; // Loop count sub-block missing. if (data == NULL) goto End; // Loop count sub-block missing.
if (data[0] != 3 && data[1] != 1) break; // wrong size/marker if (data[0] != 3 && data[1] != 1) break; // wrong size/marker
anim.loop_count = data[2] | (data[3] << 8); anim.loop_count = data[2] | (data[3] << 8);
if (verbose) fprintf(stderr, "Loop count: %d\n", anim.loop_count); if (verbose) printf("Loop count: %d\n", anim.loop_count);
} else if (!memcmp(data + 1, "XMP dataXMP", 11)) { } else if (!memcmp(data + 1, "XMP dataXMP", 11)) {
// Read XMP metadata. // Read XMP metadata.
WebPData xmp; WebPData xmp;
@@ -398,7 +401,7 @@ int main(int argc, const char *argv[]) {
xmp.bytes = (uint8_t*)data; xmp.bytes = (uint8_t*)data;
xmp.size = data[0] + 1; xmp.size = data[0] + 1;
WebPMuxSetChunk(mux, "XMP ", &xmp, 1); WebPMuxSetChunk(mux, "XMP ", &xmp, 1);
if (verbose) fprintf(stderr, "XMP size: %zu\n", xmp.size); if (verbose) printf("XMP size: %zu\n", xmp.size);
} else if (!memcmp(data + 1, "ICCRGBG1012", 11)) { } else if (!memcmp(data + 1, "ICCRGBG1012", 11)) {
// Read ICC profile. // Read ICC profile.
WebPData icc; WebPData icc;
@@ -407,7 +410,7 @@ int main(int argc, const char *argv[]) {
icc.bytes = (uint8_t*)data; icc.bytes = (uint8_t*)data;
icc.size = data[0] + 1; icc.size = data[0] + 1;
WebPMuxSetChunk(mux, "ICCP", &icc, 1); WebPMuxSetChunk(mux, "ICCP", &icc, 1);
if (verbose) fprintf(stderr, "ICC size: %zu\n", icc.size); if (verbose) printf("ICC size: %zu\n", icc.size);
} }
break; break;
} }
@@ -451,12 +454,18 @@ int main(int argc, const char *argv[]) {
fprintf(stderr, "Error writing output file: %s\n", out_file); fprintf(stderr, "Error writing output file: %s\n", out_file);
goto End; goto End;
} }
if (!quiet) fprintf(stderr, "Saved output file: %s\n", out_file); if (!quiet) {
printf("Saved output file: %s\n", out_file);
}
} else {
if (!quiet) {
printf("Nothing written; use -o flag to save the result.\n");
}
} }
// All OK. // All OK.
ok = 1; ok = 1;
gif_error = 0; gif_error = GIF_OK;
End: End:
WebPDataClear(&webp_data); WebPDataClear(&webp_data);
@@ -464,8 +473,8 @@ int main(int argc, const char *argv[]) {
WebPPictureFree(&picture); WebPPictureFree(&picture);
if (out != NULL && out_file != NULL) fclose(out); if (out != NULL && out_file != NULL) fclose(out);
if (gif_error != 0) { if (gif_error != GIF_OK) {
DisplayGifError(gif); DisplayGifError(gif, gif_error);
} }
if (gif != NULL) { if (gif != NULL) {
DGifCloseFile(gif); DGifCloseFile(gif);