diff options
author | Sebastian Rasmussen <sebras@gmail.com> | 2017-04-15 00:03:17 +0800 |
---|---|---|
committer | Sebastian Rasmussen <sebras@gmail.com> | 2017-04-20 19:47:03 +0800 |
commit | 96b76d2b9748551df1ef50db0688b39d4670e23a (patch) | |
tree | 494f22c85889f4527506698b38dacaf66e1ed33c /source/pdf | |
parent | 23eddfa1032bd0111b03a8295544c4009be5ead3 (diff) | |
download | mupdf-96b76d2b9748551df1ef50db0688b39d4670e23a.tar.xz |
js: Push annotation error handling down to library.
This makes is possible for JNI code to depend on the
library for error handling.
Diffstat (limited to 'source/pdf')
-rw-r--r-- | source/pdf/pdf-annot-edit.c | 24 |
1 files changed, 17 insertions, 7 deletions
diff --git a/source/pdf/pdf-annot-edit.c b/source/pdf/pdf-annot-edit.c index 04ba658e..a3a0137d 100644 --- a/source/pdf/pdf-annot-edit.c +++ b/source/pdf/pdf-annot-edit.c @@ -2,7 +2,7 @@ #define TEXT_ANNOT_SIZE (25.0) -const char *pdf_string_from_annot_type(fz_annot_type type) +const char *pdf_string_from_annot_type(fz_context *ctx, fz_annot_type type) { switch (type) { @@ -31,11 +31,11 @@ const char *pdf_string_from_annot_type(fz_annot_type type) case PDF_ANNOT_TRAP_NET: return "TrapNet"; case PDF_ANNOT_WATERMARK: return "Watermark"; case PDF_ANNOT_3D: return "3D"; - default: return "Unknown"; + default: return "UNKNOWN"; } } -int pdf_annot_type_from_string(const char *subtype) +int pdf_annot_type_from_string(fz_context *ctx, const char *subtype) { if (!strcmp("Text", subtype)) return PDF_ANNOT_TEXT; if (!strcmp("Link", subtype)) return PDF_ANNOT_LINK; @@ -79,8 +79,14 @@ pdf_create_annot(fz_context *ctx, pdf_page *page, fz_annot_type type) { int ind_obj_num; fz_rect rect = {0.0, 0.0, 0.0, 0.0}; - const char *type_str = pdf_string_from_annot_type(type); - pdf_obj *annot_arr = pdf_dict_get(ctx, page->obj, PDF_NAME_Annots); + const char *type_str; + pdf_obj *annot_arr; + + type_str = pdf_string_from_annot_type(ctx, type); + if (type == PDF_ANNOT_UNKNOWN) + fz_throw(ctx, FZ_ERROR_GENERIC, "cannot create unknown annotation"); + + annot_arr = pdf_dict_get(ctx, page->obj, PDF_NAME_Annots); if (annot_arr == NULL) { annot_arr = pdf_new_array(ctx, doc, 0); @@ -188,7 +194,7 @@ pdf_annot_type(fz_context *ctx, pdf_annot *annot) { pdf_obj *obj = annot->obj; pdf_obj *subtype = pdf_dict_get(ctx, obj, PDF_NAME_Subtype); - return pdf_annot_type_from_string(pdf_to_name(ctx, subtype)); + return pdf_annot_type_from_string(ctx, pdf_to_name(ctx, subtype)); } int @@ -307,7 +313,11 @@ static void pdf_set_annot_color_imp(fz_context *ctx, pdf_annot *annot, pdf_obj *key, int n, const float color[4]) { pdf_document *doc = annot->page->doc; - pdf_obj *obj = pdf_new_array(ctx, doc, 4); + pdf_obj *obj; + if (n != 0 && n != 1 && n != 3 && n != 4) + fz_throw(ctx, FZ_ERROR_GENERIC, "color must be 0, 1, 3 or 4 components"); + + obj = pdf_new_array(ctx, doc, n); switch (n) { default: |