Code cleanup: use bool for static methods
[blender.git] / source / blender / editors / space_node / node_buttons.c
index 31e3cebeb3fcda4bd9070716e94b79a46acaef47..035287c0411cf17333ef363aee68b5185d385cd4 100644 (file)
@@ -62,74 +62,7 @@ static int active_nodetree_poll(const bContext *C, PanelType *UNUSED(pt))
 {
        SpaceNode *snode = CTX_wm_space_node(C);
        
-       return (snode && ntreeIsValid(snode->nodetree));
-}
-
-/* poll callback for active node */
-static int active_node_poll(const bContext *C, PanelType *UNUSED(pt))
-{
-       SpaceNode *snode = CTX_wm_space_node(C);
-       
-       return (snode && ntreeIsValid(snode->edittree) && nodeGetActive(snode->edittree));
-}
-
-/* active node */
-static void active_node_panel(const bContext *C, Panel *pa)
-{
-       SpaceNode *snode = CTX_wm_space_node(C);
-       bNodeTree *ntree = (snode) ? snode->edittree : NULL;
-       bNode *node = (ntree) ? nodeGetActive(ntree) : NULL; // xxx... for editing group nodes
-       uiLayout *layout, *row, *col, *sub;
-       PointerRNA ptr, opptr;
-       
-       /* verify pointers, and create RNA pointer for the node */
-       if (ELEM(NULL, ntree, node))
-               return;
-       //if (node->id) /* for group nodes */
-       //      RNA_pointer_create(node->id, &RNA_Node, node, &ptr);
-       //else
-       RNA_pointer_create(&ntree->id, &RNA_Node, node, &ptr);
-       
-       layout = uiLayoutColumn(pa->layout, FALSE);
-       uiLayoutSetContextPointer(layout, "node", &ptr);
-       
-       /* draw this node's name, etc. */
-       uiItemR(layout, &ptr, "label", 0, NULL, ICON_NODE);
-       uiItemS(layout);
-       uiItemR(layout, &ptr, "name", 0, NULL, ICON_NODE);
-       uiItemS(layout);
-       
-       uiItemO(layout, NULL, 0, "NODE_OT_hide_socket_toggle");
-       uiItemS(layout);
-       uiItemS(layout);
-
-       row = uiLayoutRow(layout, FALSE);
-       
-       col = uiLayoutColumn(row, TRUE);
-       uiItemM(col, (bContext *)C, "NODE_MT_node_color_presets", NULL, 0);
-       uiItemR(col, &ptr, "use_custom_color", UI_ITEM_R_ICON_ONLY, NULL, ICON_NONE);
-       sub = uiLayoutRow(col, FALSE);
-       if (!(node->flag & NODE_CUSTOM_COLOR))
-               uiLayoutSetEnabled(sub, FALSE);
-       uiItemR(sub, &ptr, "color", 0, "", ICON_NONE);
-       
-       col = uiLayoutColumn(row, TRUE);
-       uiItemO(col, "", ICON_ZOOMIN, "node.node_color_preset_add");
-       opptr = uiItemFullO(col, "node.node_color_preset_add", "", ICON_ZOOMOUT, NULL, WM_OP_INVOKE_DEFAULT, UI_ITEM_O_RETURN_PROPS);
-       RNA_boolean_set(&opptr, "remove_active", 1);
-       uiItemM(col, (bContext *)C, "NODE_MT_node_color_specials", "", ICON_DOWNARROW_HLT);
-       
-       /* draw this node's settings */
-       if (node->typeinfo && node->typeinfo->uifuncbut) {
-               uiItemS(layout);
-               uiItemS(layout);
-               node->typeinfo->uifuncbut(layout, (bContext *)C, &ptr);
-       }
-       else if (node->typeinfo && node->typeinfo->uifunc) {
-               uiItemS(layout);
-               uiItemS(layout);
-               node->typeinfo->uifunc(layout, (bContext *)C, &ptr);
-       }
+       return (snode && snode->nodetree);
 }
 
 static int node_sockets_poll(const bContext *C, PanelType *UNUSED(pt))
@@ -167,7 +100,7 @@ static int node_tree_interface_poll(const bContext *C, PanelType *UNUSED(pt))
        return (snode && snode->edittree && (snode->edittree->inputs.first || snode->edittree->outputs.first));
 }
 
-static int node_tree_find_active_socket(bNodeTree *ntree, bNodeSocket **r_sock, int *r_in_out)
+static bool node_tree_find_active_socket(bNodeTree *ntree, bNodeSocket **r_sock, int *r_in_out)
 {
        bNodeSocket *sock;
        for (sock = ntree->inputs.first; sock; sock = sock->next) {
@@ -211,14 +144,16 @@ static void node_tree_interface_panel(const bContext *C, Panel *pa)
        
        split = uiLayoutRow(row, TRUE);
        col = uiLayoutColumn(split, TRUE);
-       uiItemL(col, "Inputs:", ICON_NONE);
-       uiTemplateList(col, (bContext *)C, "NODE_UL_interface_sockets", "", &ptr, "inputs", &ptr, "active_input", 0, 0, 0);
+       uiItemL(col, IFACE_("Inputs:"), ICON_NONE);
+       uiTemplateList(col, (bContext *)C, "NODE_UL_interface_sockets", "inputs", &ptr, "inputs", &ptr, "active_input",
+                      0, 0, 0, 0);
        opptr = uiItemFullO(col, "NODE_OT_tree_socket_add", "", ICON_PLUS, NULL, WM_OP_EXEC_DEFAULT, UI_ITEM_O_RETURN_PROPS);
        RNA_enum_set(&opptr, "in_out", SOCK_IN);
        
        col = uiLayoutColumn(split, TRUE);
-       uiItemL(col, "Outputs:", ICON_NONE);
-       uiTemplateList(col, (bContext *)C, "NODE_UL_interface_sockets", "", &ptr, "outputs", &ptr, "active_output", 0, 0, 0);
+       uiItemL(col, IFACE_("Outputs:"), ICON_NONE);
+       uiTemplateList(col, (bContext *)C, "NODE_UL_interface_sockets", "outputs", &ptr, "outputs", &ptr, "active_output",
+                      0, 0, 0, 0);
        opptr = uiItemFullO(col, "NODE_OT_tree_socket_add", "", ICON_PLUS, NULL, WM_OP_EXEC_DEFAULT, UI_ITEM_O_RETURN_PROPS);
        RNA_enum_set(&opptr, "in_out", SOCK_OUT);
        
@@ -233,10 +168,10 @@ static void node_tree_interface_panel(const bContext *C, Panel *pa)
                uiItemR(row, &sockptr, "name", 0, NULL, ICON_NONE);
                uiItemO(row, "", ICON_X, "NODE_OT_tree_socket_remove");
                
-               uiItemS(layout);
-               
-               if (sock->typeinfo->interface_draw)
+               if (sock->typeinfo->interface_draw) {
+                       uiItemS(layout);
                        sock->typeinfo->interface_draw((bContext *)C, layout, &sockptr);
+               }
        }
 }
 
@@ -246,16 +181,10 @@ void node_buttons_register(ARegionType *art)
 {
        PanelType *pt;
        
-       pt = MEM_callocN(sizeof(PanelType), "spacetype node panel active node");
-       strcpy(pt->idname, "NODE_PT_item");
-       strcpy(pt->label, IFACE_("Active Node"));
-       pt->draw = active_node_panel;
-       pt->poll = active_node_poll;
-       BLI_addtail(&art->paneltypes, pt);
-
        pt = MEM_callocN(sizeof(PanelType), "spacetype node panel node sockets");
        strcpy(pt->idname, "NODE_PT_sockets");
-       strcpy(pt->label, "Sockets");
+       strcpy(pt->label, N_("Sockets"));
+       strcpy(pt->translation_context, BLF_I18NCONTEXT_DEFAULT_BPYRNA);
        pt->draw = node_sockets_panel;
        pt->poll = node_sockets_poll;
        pt->flag |= PNL_DEFAULT_CLOSED;
@@ -263,21 +192,23 @@ void node_buttons_register(ARegionType *art)
 
        pt = MEM_callocN(sizeof(PanelType), "spacetype node panel tree interface");
        strcpy(pt->idname, "NODE_PT_node_tree_interface");
-       strcpy(pt->label, "Interface");
+       strcpy(pt->label, N_("Interface"));
+       strcpy(pt->translation_context, BLF_I18NCONTEXT_DEFAULT_BPYRNA);
        pt->draw = node_tree_interface_panel;
        pt->poll = node_tree_interface_poll;
        BLI_addtail(&art->paneltypes, pt);
 
        pt = MEM_callocN(sizeof(PanelType), "spacetype node panel gpencil");
        strcpy(pt->idname, "NODE_PT_gpencil");
-       strcpy(pt->label, "Grease Pencil");
+       strcpy(pt->label, N_("Grease Pencil"));
+       strcpy(pt->translation_context, BLF_I18NCONTEXT_DEFAULT_BPYRNA);
        pt->draw_header = gpencil_panel_standard_header;
        pt->draw = gpencil_panel_standard;
        pt->poll = active_nodetree_poll;
        BLI_addtail(&art->paneltypes, pt);
 }
 
-static int node_properties(bContext *C, wmOperator *UNUSED(op))
+static int node_properties_toggle_exec(bContext *C, wmOperator *UNUSED(op))
 {
        ScrArea *sa = CTX_wm_area(C);
        ARegion *ar = node_has_buttons_region(sa);
@@ -301,7 +232,7 @@ void NODE_OT_properties(wmOperatorType *ot)
        ot->description = "Toggles the properties panel display";
        ot->idname = "NODE_OT_properties";
        
-       ot->exec = node_properties;
+       ot->exec = node_properties_toggle_exec;
        ot->poll = node_properties_poll;
        
        /* flags */