X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;f=src%2Fxo-file.c;h=082d399f36c3d4d26ae9bc7970d9a685ae091cb6;hb=9f09269d8918dfa930543f4a15de4e7276719e5e;hp=7d2786036f38feb0e4a1231e176ba97d7f2dcca4;hpb=e648cece76b671b5b0e165acc6d14178e8de1000;p=xournal.git diff --git a/src/xo-file.c b/src/xo-file.c index 7d27860..082d399 100644 --- a/src/xo-file.c +++ b/src/xo-file.c @@ -1,3 +1,18 @@ +/* + * This program is free software; you can redistribute it and/or + * modify it under the terms of the GNU General Public + * License as published by the Free Software Foundation; either + * version 2 of the License, or (at your option) any later version. + * + * This software is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * General Public License for more details. + * + * You should have received a copy of the GNU General Public License + * along with this program. If not, see . + */ + #ifdef HAVE_CONFIG_H # include #endif @@ -11,13 +26,16 @@ #include #include #include -#include -#include #include #include #include #include +#ifndef WIN32 + #include + #include +#endif + #include "xournal.h" #include "xo-interface.h" #include "xo-support.h" @@ -25,8 +43,9 @@ #include "xo-misc.h" #include "xo-file.h" #include "xo-paint.h" +#include "xo-image.h" -const char *tool_names[NUM_TOOLS] = {"pen", "eraser", "highlighter", "text", "", "selectrect", "vertspace", "hand"}; +const char *tool_names[NUM_TOOLS] = {"pen", "eraser", "highlighter", "text", "selectregion", "selectrect", "vertspace", "hand", "image"}; const char *color_names[COLOR_MAX] = {"black", "blue", "red", "green", "gray", "lightblue", "lightgreen", "magenta", "orange", "yellow", "white"}; const char *bgtype_names[3] = {"solid", "pixmap", "pdf"}; @@ -68,6 +87,47 @@ void chk_attach_names(void) } } +/* Write image to file: returns true on success, false on error. + The image is written as a base64 encoded PNG. */ + +gboolean write_image(gzFile f, Item *item) +{ + gchar *base64_str; + + if (item->image_png == NULL) { + if (!gdk_pixbuf_save_to_buffer(item->image, &item->image_png, &item->image_png_len, "png", NULL, NULL)) { + item->image_png_len = 0; // failed for some reason, so forget it + return FALSE; + } + } + + base64_str = g_base64_encode(item->image_png, item->image_png_len); + gzputs(f, base64_str); + g_free(base64_str); + return TRUE; +} + +// create pixbuf from base64 encoded PNG, or return NULL on failure + +GdkPixbuf *read_pixbuf(const gchar *base64_str, gsize base64_strlen) +{ + gchar *base64_str2; + gchar *png_buf; + gsize png_buflen; + GdkPixbuf *pixbuf; + + // We have to copy the string in order to null terminate it, sigh. + base64_str2 = g_memdup(base64_str, base64_strlen+1); + base64_str2[base64_strlen] = 0; + png_buf = g_base64_decode(base64_str2, &png_buflen); + + pixbuf = pixbuf_from_buffer(png_buf, png_buflen); + + g_free(png_buf); + g_free(base64_str2); + return pixbuf; +} + // saves the journal to a file: returns true on success, false on error gboolean save_journal(const char *filename) @@ -83,7 +143,7 @@ gboolean save_journal(const char *filename) GList *pagelist, *layerlist, *itemlist, *list; GtkWidget *dialog; - f = gzopen(filename, "w"); + f = gzopen(filename, "wb"); if (f==NULL) return FALSE; chk_attach_names(); @@ -143,7 +203,7 @@ gboolean save_journal(const char *filename) success = FALSE; if (bgpdf.status != STATUS_NOT_INIT && bgpdf.file_contents != NULL) { - tmpf = fopen(tmpfn, "w"); + tmpf = fopen(tmpfn, "wb"); if (tmpf != NULL && fwrite(bgpdf.file_contents, 1, bgpdf.file_length, tmpf) == bgpdf.file_length) success = TRUE; fclose(tmpf); @@ -196,9 +256,17 @@ gboolean save_journal(const char *filename) else gzprintf(f, "#%08x", item->brush.color_rgba); tmpstr = g_markup_escape_text(item->text, -1); - gzprintf(f, "\">%s\n", tmpstr); + gzputs(f, "\">"); + gzputs(f, tmpstr); // gzprintf() can't handle > 4095 bytes + gzputs(f, "\n"); g_free(tmpstr); } + if (item->type == ITEM_IMAGE) { + gzprintf(f, "", + item->bbox.left, item->bbox.top, item->bbox.right, item->bbox.bottom); + if (!write_image(f, item)) success = FALSE; + gzprintf(f, "\n"); + } } gzprintf(f, "\n"); } @@ -219,6 +287,7 @@ gboolean close_journal(void) // free everything... reset_selection(); + reset_recognizer(); clear_redo_stack(); clear_undo_stack(); @@ -231,10 +300,16 @@ gboolean close_journal(void) } // sanitize a string containing floats, in case it may have , instead of . +// also replace Windows-produced 1.#J by inf void cleanup_numeric(char *s) { - while (*s!=0) { if (*s==',') *s='.'; s++; } + while (*s!=0) { + if (*s==',') *s='.'; + if (*s=='1' && s[1]=='.' && s[2]=='#' && s[3]=='J') + { *s='i'; s[1]='n'; s[2]='f'; s[3]=' '; } + s++; + } } // the XML parser functions for open_journal() @@ -342,7 +417,7 @@ void xoj_parser_start_element(GMarkupParseContext *context, } // there's also the case of hex (#rrggbbaa) colors if (tmpPage->bg->color_no == COLOR_OTHER && **attribute_values == '#') { - tmpPage->bg->color_rgba = strtol(*attribute_values + 1, &ptr, 16); + tmpPage->bg->color_rgba = strtoul(*attribute_values + 1, &ptr, 16); if (*ptr!=0) *error = xoj_invalid(); } has_attr |= 2; @@ -484,7 +559,7 @@ void xoj_parser_start_element(GMarkupParseContext *context, } // there's also the case of hex (#rrggbbaa) colors if (tmpItem->brush.color_no == COLOR_OTHER && **attribute_values == '#') { - tmpItem->brush.color_rgba = strtol(*attribute_values + 1, &ptr, 16); + tmpItem->brush.color_rgba = strtoul(*attribute_values + 1, &ptr, 16); if (*ptr!=0) *error = xoj_invalid(); } has_attr |= 2; @@ -563,7 +638,7 @@ void xoj_parser_start_element(GMarkupParseContext *context, } // there's also the case of hex (#rrggbbaa) colors if (tmpItem->brush.color_no == COLOR_OTHER && **attribute_values == '#') { - tmpItem->brush.color_rgba = strtol(*attribute_values + 1, &ptr, 16); + tmpItem->brush.color_rgba = strtoul(*attribute_values + 1, &ptr, 16); if (*ptr!=0) *error = xoj_invalid(); } has_attr |= 16; @@ -574,6 +649,56 @@ void xoj_parser_start_element(GMarkupParseContext *context, } if (has_attr!=31) *error = xoj_invalid(); } + else if (!strcmp(element_name, "image")) { // start of a image item + if (tmpLayer == NULL || tmpItem != NULL) { + *error = xoj_invalid(); + return; + } + tmpItem = (struct Item *)g_malloc0(sizeof(struct Item)); + tmpItem->type = ITEM_IMAGE; + tmpItem->canvas_item = NULL; + tmpItem->image=NULL; + tmpItem->image_png = NULL; + tmpItem->image_png_len = 0; + tmpLayer->items = g_list_append(tmpLayer->items, tmpItem); + tmpLayer->nitems++; + // scan for x, y + has_attr = 0; + while (*attribute_names!=NULL) { + if (!strcmp(*attribute_names, "left")) { + if (has_attr & 1) *error = xoj_invalid(); + cleanup_numeric((gchar *)*attribute_values); + tmpItem->bbox.left = g_ascii_strtod(*attribute_values, &ptr); + if (ptr == *attribute_values) *error = xoj_invalid(); + has_attr |= 1; + } + else if (!strcmp(*attribute_names, "top")) { + if (has_attr & 2) *error = xoj_invalid(); + cleanup_numeric((gchar *)*attribute_values); + tmpItem->bbox.top = g_ascii_strtod(*attribute_values, &ptr); + if (ptr == *attribute_values) *error = xoj_invalid(); + has_attr |= 2; + } + else if (!strcmp(*attribute_names, "right")) { + if (has_attr & 4) *error = xoj_invalid(); + cleanup_numeric((gchar *)*attribute_values); + tmpItem->bbox.right = g_ascii_strtod(*attribute_values, &ptr); + if (ptr == *attribute_values) *error = xoj_invalid(); + has_attr |= 4; + } + else if (!strcmp(*attribute_names, "bottom")) { + if (has_attr & 8) *error = xoj_invalid(); + cleanup_numeric((gchar *)*attribute_values); + tmpItem->bbox.bottom = g_ascii_strtod(*attribute_values, &ptr); + if (ptr == *attribute_values) *error = xoj_invalid(); + has_attr |= 8; + } + else *error = xoj_invalid(); + attribute_names++; + attribute_values++; + } + if (has_attr!=15) *error = xoj_invalid(); + } } void xoj_parser_end_element(GMarkupParseContext *context, @@ -609,6 +734,13 @@ void xoj_parser_end_element(GMarkupParseContext *context, } tmpItem = NULL; } + if (!strcmp(element_name, "image")) { + if (tmpItem == NULL) { + *error = xoj_invalid(); + return; + } + tmpItem = NULL; + } } void xoj_parser_text(GMarkupParseContext *context, @@ -629,7 +761,7 @@ void xoj_parser_text(GMarkupParseContext *context, if (ptr == text) break; text_len -= (ptr - text); text = ptr; - if (!finite(ui.cur_path.coords[n])) { + if (!finite_sized(ui.cur_path.coords[n])) { if (n>=2) ui.cur_path.coords[n] = ui.cur_path.coords[n-2]; else ui.cur_path.coords[n] = 0; } @@ -646,6 +778,9 @@ void xoj_parser_text(GMarkupParseContext *context, g_memmove(tmpItem->text, text, text_len); tmpItem->text[text_len]=0; } + if (!strcmp(element_name, "image")) { + tmpItem->image = read_pixbuf(text, text_len); + } } gboolean user_wants_second_chance(char **filename) @@ -664,7 +799,10 @@ gboolean user_wants_second_chance(char **filename) dialog = gtk_file_chooser_dialog_new(_("Open PDF"), GTK_WINDOW (winMain), GTK_FILE_CHOOSER_ACTION_OPEN, GTK_STOCK_CANCEL, GTK_RESPONSE_CANCEL, GTK_STOCK_OPEN, GTK_RESPONSE_OK, NULL); - +#ifdef FILE_DIALOG_SIZE_BUGFIX + gtk_window_set_default_size(GTK_WINDOW(dialog), 500, 400); +#endif + filt_all = gtk_file_filter_new(); gtk_file_filter_set_name(filt_all, _("All files")); gtk_file_filter_add_pattern(filt_all, "*"); @@ -674,6 +812,8 @@ gboolean user_wants_second_chance(char **filename) gtk_file_chooser_add_filter(GTK_FILE_CHOOSER (dialog), filt_pdf); gtk_file_chooser_add_filter(GTK_FILE_CHOOSER (dialog), filt_all); + if (ui.default_path!=NULL) gtk_file_chooser_set_current_folder(GTK_FILE_CHOOSER (dialog), ui.default_path); + if (gtk_dialog_run(GTK_DIALOG(dialog)) != GTK_RESPONSE_OK) { gtk_widget_destroy(dialog); return FALSE; @@ -696,11 +836,25 @@ gboolean open_journal(char *filename) gzFile f; char buffer[1000]; int len; - gchar *tmpfn; + gchar *tmpfn, *tmpfn2, *p, *q; gboolean maybe_pdf; - f = gzopen(filename, "r"); + tmpfn = g_strdup_printf("%s.xoj", filename); + if (ui.autoload_pdf_xoj && g_file_test(tmpfn, G_FILE_TEST_EXISTS) && + (g_str_has_suffix(filename, ".pdf") || g_str_has_suffix(filename, ".PDF"))) + { + valid = open_journal(tmpfn); + g_free(tmpfn); + return valid; + } + g_free(tmpfn); + + f = gzopen(filename, "rb"); if (f==NULL) return FALSE; + if (filename[0]=='/') { + if (ui.default_path != NULL) g_free(ui.default_path); + ui.default_path = g_path_get_dirname(filename); + } context = g_markup_parse_context_new(&parser, 0, NULL, NULL); valid = TRUE; @@ -755,7 +909,21 @@ gboolean open_journal(char *filename) else tmpfn = g_strdup(tmpBg_pdf->filename->s); valid = init_bgpdf(tmpfn, FALSE, tmpBg_pdf->file_domain); - // in case the file name became invalid + // if file name is invalid: first try in xoj file's directory + if (!valid && tmpBg_pdf->file_domain != DOMAIN_ATTACH) { + p = g_path_get_dirname(filename); + q = g_path_get_basename(tmpfn); + tmpfn2 = g_strdup_printf("%s/%s", p, q); + g_free(p); g_free(q); + valid = init_bgpdf(tmpfn2, FALSE, tmpBg_pdf->file_domain); + if (valid) { // change the file name... + printf("substituting %s -> %s\n", tmpfn, tmpfn2); + g_free(tmpBg_pdf->filename->s); + tmpBg_pdf->filename->s = tmpfn2; + } + else g_free(tmpfn2); + } + // if file name is invalid: next prompt user if (!valid && tmpBg_pdf->file_domain != DOMAIN_ATTACH) if (user_wants_second_chance(&tmpfn)) { valid = init_bgpdf(tmpfn, FALSE, tmpBg_pdf->file_domain); @@ -830,7 +998,7 @@ GList *attempt_load_gv_bg(char *filename) char *pipename; int buflen, remnlen, file_pageno; - f = fopen(filename, "r"); + f = fopen(filename, "rb"); if (f == NULL) return NULL; buf = g_malloc(BUFSIZE); // a reasonable buffer size if (fread(buf, 1, 4, f) !=4 || @@ -842,7 +1010,7 @@ GList *attempt_load_gv_bg(char *filename) fclose(f); pipename = g_strdup_printf(GS_CMDLINE, (double)GS_BITMAP_DPI, filename); - gs_pipe = popen(pipename, "r"); + gs_pipe = popen(pipename, "rb"); g_free(pipename); bg_list = NULL; @@ -888,6 +1056,7 @@ GList *attempt_load_gv_bg(char *filename) struct Background *attempt_screenshot_bg(void) { +#ifndef WIN32 struct Background *bg; GdkPixbuf *pix; XEvent x_event; @@ -924,6 +1093,10 @@ struct Background *attempt_screenshot_bg(void) bg->filename = new_refstring(NULL); bg->file_domain = DOMAIN_ATTACH; return bg; +#else + // not implemented under WIN32 + return FALSE; +#endif } /************** pdf annotation ***************/ @@ -952,6 +1125,8 @@ gboolean bgpdf_scheduler_callback(gpointer data) PopplerPage *pdfpage; gdouble height, width; int scaled_height, scaled_width; + GdkPixmap *pixmap; + cairo_t *cr; // if all requests have been cancelled, remove ourselves from main loop if (bgpdf.requests == NULL) { bgpdf.pid = 0; return FALSE; } @@ -969,11 +1144,26 @@ gboolean bgpdf_scheduler_callback(gpointer data) poppler_page_get_size(pdfpage, &width, &height); scaled_width = (int) (req->dpi * width/72); scaled_height = (int) (req->dpi * height/72); - pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, - FALSE, 8, scaled_width, scaled_height); - poppler_page_render_to_pixbuf( + + if (ui.poppler_force_cairo) { // poppler -> cairo -> pixmap -> pixbuf + pixmap = gdk_pixmap_new(GTK_WIDGET(canvas)->window, scaled_width, scaled_height, -1); + cr = gdk_cairo_create(pixmap); + cairo_set_source_rgb(cr, 1., 1., 1.); + cairo_paint(cr); + cairo_scale(cr, scaled_width/width, scaled_height/height); + poppler_page_render(pdfpage, cr); + cairo_destroy(cr); + pixbuf = gdk_pixbuf_get_from_drawable(NULL, GDK_DRAWABLE(pixmap), + NULL, 0, 0, 0, 0, scaled_width, scaled_height); + g_object_unref(pixmap); + } + else { // directly poppler -> pixbuf: faster, but bitmap font bug + pixbuf = gdk_pixbuf_new(GDK_COLORSPACE_RGB, + FALSE, 8, scaled_width, scaled_height); + wrapper_poppler_page_render_to_pixbuf( pdfpage, 0, 0, scaled_width, scaled_height, req->dpi/72, 0, pixbuf); + } g_object_unref(pdfpage); set_cursor_busy(FALSE); } @@ -1011,13 +1201,13 @@ gboolean bgpdf_scheduler_callback(gpointer data) /* make a request */ -void add_bgpdf_request(int pageno, double zoom) +gboolean add_bgpdf_request(int pageno, double zoom) { struct BgPdfRequest *req, *cmp_req; GList *list; - + if (bgpdf.status == STATUS_NOT_INIT) - return; // don't accept requests + return FALSE; // don't accept requests req = g_new(struct BgPdfRequest, 1); req->pageno = pageno; req->dpi = 72*zoom; @@ -1033,6 +1223,7 @@ void add_bgpdf_request(int pageno, double zoom) // make the request bgpdf.requests = g_list_append(bgpdf.requests, req); if (!bgpdf.pid) bgpdf.pid = g_idle_add(bgpdf_scheduler_callback, NULL); + return TRUE; } /* shutdown the PDF reader */ @@ -1081,6 +1272,7 @@ gboolean init_bgpdf(char *pdfname, gboolean create_pages, int file_domain) struct Page *pg; PopplerPage *pdfpage; gdouble width, height; + gchar *uri; if (bgpdf.status != STATUS_NOT_INIT) return FALSE; @@ -1100,8 +1292,22 @@ gboolean init_bgpdf(char *pdfname, gboolean create_pages, int file_domain) bgpdf.pid = 0; bgpdf.has_failed = FALSE; - bgpdf.document = poppler_document_new_from_data(bgpdf.file_contents, bgpdf.file_length, NULL, NULL); - if (bgpdf.document == NULL) shutdown_bgpdf(); +/* poppler_document_new_from_data() starts at 0.6.1, but we want to + be compatible with poppler 0.5.4 = latest in CentOS as of sept 2009 */ + uri = g_filename_to_uri(pdfname, NULL, NULL); + if (!uri) uri = g_strdup_printf("file://%s", pdfname); + bgpdf.document = poppler_document_new_from_file(uri, NULL, NULL); + g_free(uri); +/* with poppler 0.6.1 or later, can replace the above 4 lines by: + bgpdf.document = poppler_document_new_from_data(bgpdf.file_contents, + bgpdf.file_length, NULL, NULL); +*/ + if (bgpdf.document == NULL) { shutdown_bgpdf(); return FALSE; } + + if (pdfname[0]=='/' && ui.filename == NULL) { + if (ui.default_path!=NULL) g_free(ui.default_path); + ui.default_path = g_path_get_dirname(pdfname); + } if (!create_pages) return TRUE; // we're done @@ -1204,7 +1410,8 @@ void update_mru_menu(void) for (i=0; icolor_no])); + (ui.default_page.bg->color_no>=0)? + g_strdup(bgcolor_names[ui.default_page.bg->color_no]): + g_strdup_printf("#%08x", ui.default_page.bg->color_rgba)); update_keyval("paper", "style", _(" the default paper style (plain, lined, ruled, or graph)"), g_strdup(bgstyle_names[ui.default_page.bg->ruling])); @@ -1472,9 +1698,6 @@ void save_config_to_file(void) update_keyval("paper", "print_ruling", _(" include paper ruling when printing or exporting to PDF (true/false)"), g_strdup(ui.print_ruling?"true":"false")); - update_keyval("paper", "antialias_bg", - _(" antialiased bitmap backgrounds (true/false)"), - g_strdup(ui.antialias_bg?"true":"false")); update_keyval("paper", "progressive_bg", _(" just-in-time update of page backgrounds (true/false)"), g_strdup(ui.progressive_bg?"true":"false")); @@ -1486,11 +1709,13 @@ void save_config_to_file(void) g_strdup_printf("%d", PDFTOPPM_PRINTING_DPI)); update_keyval("tools", "startup_tool", - _(" selected tool at startup (pen, eraser, highlighter, selectrect, vertspace, hand)"), + _(" selected tool at startup (pen, eraser, highlighter, selectregion, selectrect, vertspace, hand, image)"), g_strdup(tool_names[ui.startuptool])); update_keyval("tools", "pen_color", _(" default pen color"), - g_strdup(color_names[ui.default_brushes[TOOL_PEN].color_no])); + (ui.default_brushes[TOOL_PEN].color_no>=0)? + g_strdup(color_names[ui.default_brushes[TOOL_PEN].color_no]): + g_strdup_printf("#%08x", ui.default_brushes[TOOL_PEN].color_rgba)); update_keyval("tools", "pen_thickness", _(" default pen thickness (fine = 1, medium = 2, thick = 3)"), g_strdup_printf("%d", ui.default_brushes[TOOL_PEN].thickness_no)); @@ -1508,7 +1733,9 @@ void save_config_to_file(void) g_strdup_printf("%d", ui.default_brushes[TOOL_ERASER].tool_options)); update_keyval("tools", "highlighter_color", _(" default highlighter color"), - g_strdup(color_names[ui.default_brushes[TOOL_HIGHLIGHTER].color_no])); + (ui.default_brushes[TOOL_HIGHLIGHTER].color_no>=0)? + g_strdup(color_names[ui.default_brushes[TOOL_HIGHLIGHTER].color_no]): + g_strdup_printf("#%08x", ui.default_brushes[TOOL_HIGHLIGHTER].color_rgba)); update_keyval("tools", "highlighter_thickness", _(" default highlighter thickness (fine = 1, medium = 2, thick = 3)"), g_strdup_printf("%d", ui.default_brushes[TOOL_HIGHLIGHTER].thickness_no)); @@ -1519,15 +1746,18 @@ void save_config_to_file(void) _(" default highlighter is in shape recognizer mode (true/false)"), g_strdup(ui.default_brushes[TOOL_HIGHLIGHTER].recognizer?"true":"false")); update_keyval("tools", "btn2_tool", - _(" button 2 tool (pen, eraser, highlighter, text, selectrect, vertspace, hand)"), + _(" button 2 tool (pen, eraser, highlighter, text, selectregion, selectrect, vertspace, hand, image)"), g_strdup(tool_names[ui.toolno[1]])); update_keyval("tools", "btn2_linked", _(" button 2 brush linked to primary brush (true/false) (overrides all other settings)"), g_strdup((ui.linked_brush[1]==BRUSH_LINKED)?"true":"false")); update_keyval("tools", "btn2_color", _(" button 2 brush color (for pen or highlighter only)"), - g_strdup((ui.toolno[1]=0)? + g_strdup(color_names[ui.brushes[1][ui.toolno[1]].color_no]): + g_strdup_printf("#%08x", ui.brushes[1][ui.toolno[1]].color_rgba)): + g_strdup("white")); update_keyval("tools", "btn2_thickness", _(" button 2 brush thickness (pen, eraser, or highlighter only)"), g_strdup_printf("%d", (ui.toolno[1]=0)? + g_strdup(color_names[ui.brushes[2][ui.toolno[2]].color_no]): + g_strdup_printf("#%08x", ui.brushes[2][ui.toolno[2]].color_rgba)): + g_strdup("white")); update_keyval("tools", "btn3_thickness", _(" button 3 brush thickness (pen, eraser, or highlighter only)"), g_strdup_printf("%d", (ui.toolno[2]color_no), bgcolor_names, COLOR_MAX); - ui.default_page.bg->color_rgba = predef_bgcolors_rgba[ui.default_page.bg->color_no]; + parse_keyval_enum_color("paper", "color", + &(ui.default_page.bg->color_no), &(ui.default_page.bg->color_rgba), + bgcolor_names, predef_bgcolors_rgba, COLOR_MAX); parse_keyval_enum("paper", "style", &(ui.default_page.bg->ruling), bgstyle_names, 4); parse_keyval_boolean("paper", "apply_all", &ui.bg_apply_all_pages); parse_keyval_enum("paper", "default_unit", &ui.default_unit, unit_names, 4); - parse_keyval_boolean("paper", "antialias_bg", &ui.antialias_bg); parse_keyval_boolean("paper", "progressive_bg", &ui.progressive_bg); parse_keyval_boolean("paper", "print_ruling", &ui.print_ruling); parse_keyval_int("paper", "gs_bitmap_dpi", &GS_BITMAP_DPI, 1, 1200); @@ -1797,13 +2055,17 @@ void load_config_from_file(void) parse_keyval_enum("tools", "startup_tool", &ui.startuptool, tool_names, NUM_TOOLS); ui.toolno[0] = ui.startuptool; - parse_keyval_enum("tools", "pen_color", &(ui.brushes[0][TOOL_PEN].color_no), color_names, COLOR_MAX); + parse_keyval_enum_color("tools", "pen_color", + &(ui.brushes[0][TOOL_PEN].color_no), &(ui.brushes[0][TOOL_PEN].color_rgba), + color_names, predef_colors_rgba, COLOR_MAX); parse_keyval_int("tools", "pen_thickness", &(ui.brushes[0][TOOL_PEN].thickness_no), 0, 4); parse_keyval_boolean("tools", "pen_ruler", &(ui.brushes[0][TOOL_PEN].ruler)); parse_keyval_boolean("tools", "pen_recognizer", &(ui.brushes[0][TOOL_PEN].recognizer)); parse_keyval_int("tools", "eraser_thickness", &(ui.brushes[0][TOOL_ERASER].thickness_no), 1, 3); parse_keyval_int("tools", "eraser_mode", &(ui.brushes[0][TOOL_ERASER].tool_options), 0, 2); - parse_keyval_enum("tools", "highlighter_color", &(ui.brushes[0][TOOL_HIGHLIGHTER].color_no), color_names, COLOR_MAX); + parse_keyval_enum_color("tools", "highlighter_color", + &(ui.brushes[0][TOOL_HIGHLIGHTER].color_no), &(ui.brushes[0][TOOL_HIGHLIGHTER].color_rgba), + color_names, predef_colors_rgba, COLOR_MAX); parse_keyval_int("tools", "highlighter_thickness", &(ui.brushes[0][TOOL_HIGHLIGHTER].thickness_no), 0, 4); parse_keyval_boolean("tools", "highlighter_ruler", &(ui.brushes[0][TOOL_HIGHLIGHTER].ruler)); parse_keyval_boolean("tools", "highlighter_recognizer", &(ui.brushes[0][TOOL_HIGHLIGHTER].recognizer)); @@ -1822,7 +2084,9 @@ void load_config_from_file(void) if (ui.toolno[1]==TOOL_PEN || ui.toolno[1]==TOOL_HIGHLIGHTER) { parse_keyval_boolean("tools", "btn2_ruler", &(ui.brushes[1][ui.toolno[1]].ruler)); parse_keyval_boolean("tools", "btn2_recognizer", &(ui.brushes[1][ui.toolno[1]].recognizer)); - parse_keyval_enum("tools", "btn2_color", &(ui.brushes[1][ui.toolno[1]].color_no), color_names, COLOR_MAX); + parse_keyval_enum_color("tools", "btn2_color", + &(ui.brushes[1][ui.toolno[1]].color_no), &(ui.brushes[1][ui.toolno[1]].color_rgba), + color_names, predef_colors_rgba, COLOR_MAX); } if (ui.toolno[1]