Merge lp:~inkscape.dev/inkscape/npdfix into lp:~inkscape.dev/inkscape/trunk

[Kris]

Found this part with possible null pointer dereference and made a proposal to fix it.

I would appreciate it:
if anyone could review it and
if you could change the code directly in case there are drawbacks to the suggested fix

[Mc (mc...)]

Functionally equivalent code.

review: Approve

=== modified file 'src/ui/interface.cpp'
--- src/ui/interface.cpp	2015-09-16 18:22:07 +0000
+++ src/ui/interface.cpp	2015-11-16 22:21:40 +0000
@@ -846,13 +846,11 @@
 #endif
                     }
                 } else if (menu_pntr->attribute("check") != NULL) {
-                    SPAction *action = NULL;
                     if (verb->get_code() != SP_VERB_NONE) {
-                        action = verb->get_action(Inkscape::ActionContext(view));
-                    }
-                    sp_ui_menu_append_check_item_from_verb(GTK_MENU(menu), view, action->name, action->tip, NULL,
+                        SPAction *action = verb->get_action(Inkscape::ActionContext(view));
+                        sp_ui_menu_append_check_item_from_verb(GTK_MENU(menu), view, action->name, action->tip, NULL,
                             checkitem_toggled, checkitem_update, verb);
-
+                    }
                 } else {
                     sp_ui_menu_append_item_from_verb(GTK_MENU(menu), verb, view);
                     group = NULL;