converted more mixed tab/space indentations to tabs. only whitespace changes.
[blender.git] / source / blender / makesrna / intern / rna_nodetree.c
index 9d4575b..b6d0f77 100644 (file)
@@ -1,4 +1,4 @@
-/**
+/*
  * $Id$
  *
  * ***** BEGIN GPL LICENSE BLOCK *****
  * ***** END GPL LICENSE BLOCK *****
  */
 
+/** \file blender/makesrna/intern/rna_nodetree.c
+ *  \ingroup RNA
+ */
+
+
 #include <stdlib.h>
 #include <string.h>
 
 #include "RNA_define.h"
+#include "RNA_enum_types.h"
 
 #include "rna_internal.h"
 
 
 #include "MEM_guardedalloc.h"
 
+
+EnumPropertyItem node_socket_type_items[] = {
+       {SOCK_VALUE,  "VALUE",     0,    "Value",     ""},
+       {SOCK_VECTOR, "VECTOR",    0,    "Vector",    ""},
+       {SOCK_RGBA,   "RGBA",      0,    "RGBA",      ""},
+       {0, NULL, 0, NULL, NULL}};
+
+EnumPropertyItem node_math_items[] = {
+{ 0, "ADD",          0, "Add",          ""},
+{ 1, "SUBTRACT",     0, "Subtract",     ""},
+{ 2, "MULTIPLY",     0, "Multiply",     ""},
+{ 3, "DIVIDE",       0, "Divide",       ""},
+{ 4, "SINE",         0, "Sine",         ""},
+{ 5, "COSINE",       0, "Cosine",       ""},
+{ 6, "TANGENT",      0, "Tangent",      ""},
+{ 7, "ARCSINE",      0, "Arcsine",      ""},
+{ 8, "ARCCOSINE",    0, "Arccosine",    ""},
+{ 9, "ARCTANGENT",   0, "Arctangent",   ""},
+{10, "POWER",        0, "Power",        ""},
+{11, "LOGARITHM",    0, "Logarithm",    ""},
+{12, "MINIMUM",      0, "Minimum",      ""},
+{13, "MAXIMUM",      0, "Maximum",      ""},
+{14, "ROUND",        0, "Round",        ""},
+{15, "LESS_THAN",    0, "Less Than",    ""},
+{16, "GREATER_THAN", 0, "Greater Than", ""},
+{0, NULL, 0, NULL, NULL}};
+
+EnumPropertyItem node_vec_math_items[] = {
+{0, "ADD",           0, "Add",           ""},
+{1, "SUBTRACT",      0, "Subtract",      ""},
+{2, "AVERAGE",       0, "Average",       ""},
+{3, "DOT_PRODUCT",   0, "Dot Product",   ""},
+{4, "CROSS_PRODUCT", 0, "Cross Product", ""},
+{5, "NORMALIZE",     0, "Normalize",     ""},
+{0, NULL, 0, NULL, NULL}};
+
+EnumPropertyItem node_filter_items[] = {
+{0, "SOFTEN",  0, "Soften",  ""},
+{1, "SHARPEN", 0, "Sharpen", ""},
+{2, "LAPLACE", 0, "Laplace", ""},
+{3, "SOBEL",   0, "Sobel",   ""},
+{4, "PREWITT", 0, "Prewitt", ""},
+{5, "KIRSCH",  0, "Kirsch",  ""},
+{6, "SHADOW",  0, "Shadow",  ""},
+{0, NULL, 0, NULL, NULL}};
+
 #ifdef RNA_RUNTIME
 
 #include "BLI_linklist.h"
@@ -125,12 +177,22 @@ static char *rna_NodeSocket_path(PointerRNA *ptr)
        bNode *node;
        int socketindex;
        
-       if (!nodeFindNode(ntree, sock, &node, NULL)) return NULL;
-
+       /* group sockets */
+       socketindex = BLI_findindex(&ntree->inputs, sock);
+       if (socketindex != -1)
+               return BLI_sprintfN("inputs[%d]", socketindex);
+       
+       socketindex = BLI_findindex(&ntree->outputs, sock);
+       if (socketindex != -1)
+               return BLI_sprintfN("outputs[%d]", socketindex);
+       
+       /* node sockets */
+       if (!nodeFindNode(ntree, sock, &node, NULL, NULL)) return NULL;
+       
        socketindex = BLI_findindex(&node->inputs, sock);
        if (socketindex != -1)
                return BLI_sprintfN("nodes[\"%s\"].inputs[%d]", node->name, socketindex);
-
+       
        socketindex = BLI_findindex(&node->outputs, sock);
        if (socketindex != -1)
                return BLI_sprintfN("nodes[\"%s\"].outputs[%d]", node->name, socketindex);
@@ -213,7 +275,7 @@ static void rna_NodeGroup_update(Main *bmain, Scene *scene, PointerRNA *ptr)
        bNodeTree *ntree= (bNodeTree*)ptr->id.data;
        bNode *node= (bNode*)ptr->data;
        
-       nodeVerifyGroup((bNodeTree *)node->id);
+       nodeGroupVerify((bNodeTree *)node->id);
        
        node_update(bmain, scene, ntree, node);
 }
@@ -230,68 +292,30 @@ static void rna_Node_name_set(PointerRNA *ptr, const char *value)
        BLI_strncpy(node->name, value, sizeof(node->name));
        
        nodeUniqueName(ntree, node);
-       node->flag |= NODE_CUSTOM_NAME;
        
        /* fix all the animation data which may link to this */
        BKE_all_animdata_fix_paths_rename("nodes", oldname, node->name);
 }
 
-/* this should be done at display time! if no custom names are set */
-#if 0
-static void rna_Node_update_username(Main *bmain, Scene *scene, PointerRNA *ptr)
+static void rna_NodeSocket_update(Main *bmain, Scene *scene, PointerRNA *ptr)
 {
-       bNode *node= (bNode*)ptr->data;
-       const char *name;
-
+       bNodeTree *ntree= (bNodeTree*)ptr->id.data;
+       bNodeSocket *sock= (bNodeSocket*)ptr->data;
+       bNode *node;
        
-       /*
-       if (!node->username[0]) {
-               if(node->id) {
-                       BLI_strncpy(node->username, node->id->name+2, NODE_MAXSTR);
-               }
-               else {
-               
-                       switch(node->typeinfo->type) {
-                               case SH_NODE_MIX_RGB:
-                               case CMP_NODE_MIX_RGB:
-                               case TEX_NODE_MIX_RGB:
-                                       if(RNA_enum_name(node_blend_type_items, node->custom1, &name))
-                                               BLI_strncpy(node->username, name, NODE_MAXSTR);
-                                       break;
-                               case CMP_NODE_FILTER:
-                                       if(RNA_enum_name(node_filter_items, node->custom1, &name))
-                                               BLI_strncpy(node->username, name, NODE_MAXSTR);
-                                       break;
-                               case CMP_NODE_FLIP:
-                                       if(RNA_enum_name(node_flip_items, node->custom1, &name))
-                                               BLI_strncpy(node->username, name, NODE_MAXSTR);
-                                       break;
-                               case SH_NODE_MATH:
-                               case CMP_NODE_MATH:
-                               case TEX_NODE_MATH:
-                                       if(RNA_enum_name(node_math_items, node->custom1, &name))
-                                               BLI_strncpy(node->username, name, NODE_MAXSTR);
-                                       break;
-                               case SH_NODE_VECT_MATH:
-                                       if(RNA_enum_name(node_vec_math_items, node->custom1, &name))
-                                               BLI_strncpy(node->username, name, NODE_MAXSTR);
-                                       break;
-                       }
-                */
-               }
-       }
-
-       rna_Node_update(bmain, scene, ptr);
+       if (nodeFindNode(ntree, sock, &node, NULL, NULL))
+               node_update(bmain, scene, ntree, node);
 }
-#endif
 
-static void rna_NodeSocket_update(Main *bmain, Scene *scene, PointerRNA *ptr)
+static void rna_NodeGroupSocket_update(Main *bmain, Scene *scene, PointerRNA *ptr)
 {
        bNodeTree *ntree= (bNodeTree*)ptr->id.data;
        bNodeSocket *sock= (bNodeSocket*)ptr->data;
        bNode *node;
        
-       if (nodeFindNode(ntree, sock, &node, NULL))
+       nodeGroupVerify(ntree);
+       
+       if (nodeFindNode(ntree, sock, &node, NULL, NULL))
                node_update(bmain, scene, ntree, node);
 }
 
@@ -438,10 +462,10 @@ static bNode *rna_NodeTree_node_new(bNodeTree *ntree, bContext *C, ReportList *r
        node = nodeAddNodeType(ntree, type, group, NULL);
 
        if (node == NULL) {
-                BKE_reportf(reports, RPT_ERROR, "Unable to create node");
+               BKE_reportf(reports, RPT_ERROR, "Unable to create node");
        }
        else {
-               nodeVerifyGroup(ntree); /* update group node socket links*/
+               nodeGroupVerify(ntree); /* update group node socket links*/
                NodeTagChanged(ntree, node);
                WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
 
@@ -499,7 +523,7 @@ static void rna_NodeTree_node_remove(bNodeTree *ntree, ReportList *reports, bNod
                        id_us_min(node->id);
 
                nodeFreeNode(ntree, node);
-               nodeVerifyGroup(ntree); /* update group node socket links*/
+               nodeGroupVerify(ntree); /* update group node socket links*/
 
                WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
        }
@@ -508,15 +532,14 @@ static void rna_NodeTree_node_remove(bNodeTree *ntree, ReportList *reports, bNod
 static bNodeLink *rna_NodeTree_link_new(bNodeTree *ntree, ReportList *reports, bNodeSocket *in, bNodeSocket *out)
 {
        bNodeLink *ret;
-       bNode *fromnode, *tonode;
+       bNode *fromnode= NULL, *tonode= NULL;
+       int from_in_out, to_in_out;
 
-       if (!nodeFindNode(ntree, in, &fromnode, NULL)) {
-               BKE_reportf(reports, RPT_ERROR, "Unable to locate input socket's node in nodetree");
-               return NULL;
-       }
-
-       if (!nodeFindNode(ntree, out, &tonode, NULL)) {
-               BKE_reportf(reports, RPT_ERROR, "Unable to locate output socket's node in nodetree");
+       nodeFindNode(ntree, in, &fromnode, NULL, &from_in_out);
+       nodeFindNode(ntree, out, &tonode, NULL, &to_in_out);
+       
+       if (&from_in_out == &to_in_out) {
+               BKE_reportf(reports, RPT_ERROR, "Same input/output direction of sockets");
                return NULL;
        }
 
@@ -524,15 +547,16 @@ static bNodeLink *rna_NodeTree_link_new(bNodeTree *ntree, ReportList *reports, b
        nodeRemSocketLinks(ntree, out);
 
        ret= nodeAddLink(ntree, fromnode, in, tonode, out);
+       
+       if(ret) {
+               NodeTagChanged(ntree, tonode);
 
-       NodeTagChanged(ntree, tonode);
-
-       nodeVerifyGroup(ntree); /* update group node socket links*/
-
-       ntreeSolveOrder(ntree);
+               nodeGroupVerify(ntree); /* update group node socket links*/
 
-       WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+               ntreeSolveOrder(ntree);
 
+               WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+       }
        return ret;
 }
 
@@ -544,10 +568,76 @@ static void rna_NodeTree_link_remove(bNodeTree *ntree, ReportList *reports, bNod
        else {
                nodeRemLink(ntree, link);
                ntreeSolveOrder(ntree);
-               nodeVerifyGroup(ntree); /* update group node socket links*/
+               nodeGroupVerify(ntree); /* update group node socket links*/
+
+               WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+       }
+}
+
+static bNodeSocket *rna_NodeTree_input_new(bNodeTree *ntree, ReportList *UNUSED(reports), const char *name, int type)
+{
+       /* XXX should check if tree is a group here! no good way to do this currently. */
+       bNodeSocket *gsock= nodeGroupAddSocket(ntree, name, type, SOCK_IN);
+       
+       nodeGroupVerify(ntree); /* update group node socket links*/
+       WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+       return gsock;
+}
+
+static bNodeSocket *rna_NodeTree_output_new(bNodeTree *ntree, ReportList *UNUSED(reports), const char *name, int type)
+{
+       /* XXX should check if tree is a group here! no good way to do this currently. */
+       bNodeSocket *gsock= nodeGroupAddSocket(ntree, name, type, SOCK_OUT);
+       
+       nodeGroupVerify(ntree); /* update group node socket links*/
+       WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+       return gsock;
+}
+
+static bNodeSocket *rna_NodeTree_input_expose(bNodeTree *ntree, ReportList *reports, bNodeSocket *sock, int add_link)
+{
+       bNode *node;
+       bNodeSocket *gsock;
+       int index, in_out;
+       
+       if (!nodeFindNode(ntree, sock, &node, &index, &in_out))
+               BKE_reportf(reports, RPT_ERROR, "Unable to locate socket in nodetree");
+       else if (in_out!=SOCK_IN)
+               BKE_reportf(reports, RPT_ERROR, "Socket is not an input");
+       else {
+               /* XXX should check if tree is a group here! no good way to do this currently. */
+               gsock = nodeGroupAddSocket(ntree, sock->name, sock->type, SOCK_IN);
+               if (add_link)
+                       nodeAddLink(ntree, NULL, gsock, node, sock);
+               
+               nodeGroupVerify(ntree); /* update group node socket links*/
+               WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+               return gsock;
+       }
+       return NULL;
+}
 
+static bNodeSocket *rna_NodeTree_output_expose(bNodeTree *ntree, ReportList *reports, bNodeSocket *sock, int add_link)
+{
+       bNode *node;
+       bNodeSocket *gsock;
+       int index, in_out;
+       
+       if (!nodeFindNode(ntree, sock, &node, &index, &in_out))
+               BKE_reportf(reports, RPT_ERROR, "Unable to locate socket in nodetree");
+       else if (in_out!=SOCK_OUT)
+               BKE_reportf(reports, RPT_ERROR, "Socket is not an output");
+       else {
+               /* XXX should check if tree is a group here! no good way to do this currently. */
+               gsock = nodeGroupAddSocket(ntree, sock->name, sock->type, SOCK_OUT);
+               if (add_link)
+                       nodeAddLink(ntree, node, sock, NULL, gsock);
+               
+               nodeGroupVerify(ntree); /* update group node socket links*/
                WM_main_add_notifier(NC_NODE|NA_EDITED, ntree);
+               return gsock;
        }
+       return NULL;
 }
 
 #else
@@ -566,72 +656,12 @@ static EnumPropertyItem prop_tri_channel_items[] = {
 { 3, "B", 0, "B", ""},
 {0, NULL, 0, NULL, NULL}};
 
-static EnumPropertyItem node_blend_type_items[] = {
-{ 0, "MIX",          0, "Mix",         ""},
-{ 1, "ADD",          0, "Add",         ""},
-{ 3, "SUBTRACT",     0, "Subtract",    ""},
-{ 2, "MULTIPLY",     0, "Multiply",    ""},
-{ 4, "SCREEN",       0, "Screen",      ""},
-{ 9, "OVERLAY",      0, "Overlay",     ""},
-{ 5, "DIVIDE",       0, "Divide",      ""},
-{ 6, "DIFFERENCE",   0, "Difference",  ""},
-{ 7, "DARKEN",       0, "Darken",      ""},
-{ 8, "LIGHTEN",      0, "Lighten",     ""},
-{10, "DODGE",        0, "Dodge",       ""},
-{11, "BURN",         0, "Burn",        ""},
-{15, "COLOR",        0, "Color",       ""},
-{14, "VALUE",        0, "Value",       ""},
-{13, "SATURATION",   0, "Saturation",  ""},
-{12, "HUE",          0, "Hue",         ""},
-{16, "SOFT_LIGHT",   0, "Soft Light",  ""},
-{17, "LINEAR_LIGHT", 0, "Linear Light",""},
-{0, NULL, 0, NULL, NULL}};
-
 static EnumPropertyItem node_flip_items[] = {
 {0, "X",  0, "Flip X",     ""},
 {1, "Y",  0, "Flip Y",     ""},
 {2, "XY", 0, "Flip X & Y", ""},
 {0, NULL, 0, NULL, NULL}};
 
-static EnumPropertyItem node_math_items[] = {
-{ 0, "ADD",          0, "Add",          ""},
-{ 1, "SUBTRACT",     0, "Subtract",     ""},
-{ 2, "MULTIPLY",     0, "Multiply",     ""},
-{ 3, "DIVIDE",       0, "Divide",       ""},
-{ 4, "SINE",         0, "Sine",         ""},
-{ 5, "COSINE",       0, "Cosine",       ""},
-{ 6, "TANGENT",      0, "Tangent",      ""},
-{ 7, "ARCSINE",      0, "Arcsine",      ""},
-{ 8, "ARCCOSINE",    0, "Arccosine",    ""},
-{ 9, "ARCTANGENT",   0, "Arctangent",   ""},
-{10, "POWER",        0, "Power",        ""},
-{11, "LOGARITHM",    0, "Logarithm",    ""},
-{12, "MINIMUM",      0, "Minimum",      ""},
-{13, "MAXIMUM",      0, "Maximum",      ""},
-{14, "ROUND",        0, "Round",        ""},
-{15, "LESS_THAN",    0, "Less Than",    ""},
-{16, "GREATER_THAN", 0, "Greater Than", ""},
-{0, NULL, 0, NULL, NULL}};
-
-static EnumPropertyItem node_vec_math_items[] = {
-{0, "ADD",           0, "Add",           ""},
-{1, "SUBTRACT",      0, "Subtract",      ""},
-{2, "AVERAGE",       0, "Average",       ""},
-{3, "DOT_PRODUCT",   0, "Dot Product",   ""},
-{4, "CROSS_PRODUCT", 0, "Cross Product", ""},
-{5, "NORMALIZE",     0, "Normalize",     ""},
-{0, NULL, 0, NULL, NULL}};
-
-static EnumPropertyItem node_filter_items[] = {
-{0, "SOFTEN",  0, "Soften",  ""},
-{1, "SHARPEN", 0, "Sharpen", ""},
-{2, "LAPLACE", 0, "Laplace", ""},
-{3, "SOBEL",   0, "Sobel",   ""},
-{4, "PREWITT", 0, "Prewitt", ""},
-{5, "KIRSCH",  0, "Kirsch",  ""},
-{6, "SHADOW",  0, "Shadow",  ""},
-{0, NULL, 0, NULL, NULL}};
-
 static EnumPropertyItem node_ycc_items[] = {
 { 0, "ITUBT601", 0, "ITU 601",  ""},
 { 1, "ITUBT709", 0, "ITU 709",  ""},
@@ -846,7 +876,7 @@ static void def_mix_rgb(StructRNA *srna)
        
        prop = RNA_def_property(srna, "blend_type", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "custom1");
-       RNA_def_property_enum_items(prop, node_blend_type_items);
+       RNA_def_property_enum_items(prop, ramp_blend_items);
        RNA_def_property_ui_text(prop, "Blend Type", "");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
@@ -915,7 +945,7 @@ static void def_sh_mapping(StructRNA *srna)
        RNA_def_property_ui_range(prop, -10.f, 10.f, 0.1f, 2);
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_mapping_update");
        
-       prop= RNA_def_property(srna, "rotation", PROP_FLOAT, PROP_EULER);
+       prop= RNA_def_property(srna, "rotation", PROP_FLOAT, PROP_XYZ); /* Not PROP_EUL, this is already in degrees, not radians */
        RNA_def_property_float_sdna(prop, NULL, "rot");
        RNA_def_property_ui_text(prop, "Rotation", "Rotation offset for the input coordinate");
        RNA_def_property_ui_range(prop, -360.f, 360.f, 1.f, 2);
@@ -1029,6 +1059,12 @@ static void def_cmp_blur(StructRNA *srna)
                {R_FILTER_MITCH,      "MITCH",      0, "Mitch",         ""},
                {0, NULL, 0, NULL, NULL}};
 
+       static EnumPropertyItem aspect_correction_type_items[] = {
+               {CMP_NODE_BLUR_ASPECT_NONE,     "NONE", 0,      "None", ""},
+               {CMP_NODE_BLUR_ASPECT_Y,        "Y",    0,      "Y",    ""},
+               {CMP_NODE_BLUR_ASPECT_X,        "X",    0,      "X",    ""},
+               {0, NULL, 0, NULL, NULL}};
+
        RNA_def_struct_sdna_from(srna, "NodeBlurData", "storage");
        
        prop = RNA_def_property(srna, "size_x", PROP_INT, PROP_NONE);
@@ -1048,21 +1084,27 @@ static void def_cmp_blur(StructRNA *srna)
        RNA_def_property_ui_text(prop, "Relative", "Use relative (percent) values to define blur radius");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
+       prop = RNA_def_property(srna, "aspect_correction", PROP_ENUM, PROP_NONE);
+       RNA_def_property_enum_sdna(prop, NULL, "aspect");
+       RNA_def_property_enum_items(prop, aspect_correction_type_items);
+       RNA_def_property_ui_text(prop, "Aspect Correction", "Type of aspect correction to use");
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
+
        prop = RNA_def_property(srna, "factor", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "fac");
        RNA_def_property_range(prop, 0.0f, 2.0f);
        RNA_def_property_ui_text(prop, "Factor", "");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
-       prop = RNA_def_property(srna, "factor_x", PROP_FLOAT, PROP_NONE);
+       prop = RNA_def_property(srna, "factor_x", PROP_FLOAT, PROP_PERCENTAGE);
        RNA_def_property_float_sdna(prop, NULL, "percentx");
-       RNA_def_property_range(prop, 0.0f, 1.0f);
+       RNA_def_property_range(prop, 0.0f, 100.0f);
        RNA_def_property_ui_text(prop, "Relative Size X", "");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
-       prop = RNA_def_property(srna, "factor_y", PROP_FLOAT, PROP_NONE);
+       prop = RNA_def_property(srna, "factor_y", PROP_FLOAT, PROP_PERCENTAGE);
        RNA_def_property_float_sdna(prop, NULL, "percenty");
-       RNA_def_property_range(prop, 0.0f, 1.0f);
+       RNA_def_property_range(prop, 0.0f, 100.0f);
        RNA_def_property_ui_text(prop, "Relative Size Y", "");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
@@ -1172,7 +1214,7 @@ static void def_cmp_vector_blur(StructRNA *srna)
        
        prop = RNA_def_property(srna, "use_curved", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "curved", 1);
-       RNA_def_property_ui_text(prop, "Curved", "Interpolate between frames in a bezier curve, rather than linearly");
+       RNA_def_property_ui_text(prop, "Curved", "Interpolate between frames in a Bezier curve, rather than linearly");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 }
 
@@ -1505,7 +1547,7 @@ static void def_cmp_color_spill(StructRNA *srna)
 
        prop = RNA_def_property(srna, "use_unspill", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_sdna(prop, NULL, "unspill", 0);
-       RNA_def_property_ui_text(prop, "Unspill", "Compensate all channels (diffenrently) by hand");
+       RNA_def_property_ui_text(prop, "Unspill", "Compensate all channels (differently) by hand");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
        prop = RNA_def_property(srna, "unspill_red", PROP_FLOAT, PROP_NONE);
@@ -1626,8 +1668,8 @@ static void def_cmp_channel_matte(StructRNA *srna)
 
        prop = RNA_def_property(srna, "limit_channel", PROP_ENUM, PROP_NONE);
        RNA_def_property_enum_sdna(prop, NULL, "channel");
-   RNA_def_property_enum_items(prop, prop_tri_channel_items);
-   RNA_def_property_enum_funcs(prop, NULL, NULL, "rna_Node_channel_itemf");
+       RNA_def_property_enum_items(prop, prop_tri_channel_items);
+       RNA_def_property_enum_funcs(prop, NULL, NULL, "rna_Node_channel_itemf");
        RNA_def_property_ui_text(prop, "Limit Channel", "Limit by this channels value");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
@@ -1767,13 +1809,13 @@ static void def_cmp_defocus(StructRNA *srna)
        
        prop = RNA_def_property(srna, "use_zbuffer", PROP_BOOLEAN, PROP_NONE);
        RNA_def_property_boolean_negative_sdna(prop, NULL, "no_zbuf", 1);
-       RNA_def_property_ui_text(prop, "Use Z-Buffer", "Disable when using an image as input instead of actual zbuffer (auto enabled if node not image based, eg. time node)");
+       RNA_def_property_ui_text(prop, "Use Z-Buffer", "Disable when using an image as input instead of actual z-buffer (auto enabled if node not image based, eg. time node)");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
        
        prop = RNA_def_property(srna, "z_scale", PROP_FLOAT, PROP_NONE);
        RNA_def_property_float_sdna(prop, NULL, "scale");
        RNA_def_property_range(prop, 0.0f, 1000.0f);
-       RNA_def_property_ui_text(prop, "Z-Scale", "Scales the Z input when not using a zbuffer, controls maximum blur designated by the color white or input value 1");
+       RNA_def_property_ui_text(prop, "Z-Scale", "Scales the Z input when not using a z-buffer, controls maximum blur designated by the color white or input value 1");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 }
 
@@ -1801,10 +1843,10 @@ static void def_cmp_crop(StructRNA *srna)
        RNA_def_property_ui_text(prop, "Crop Image Size", "Whether to crop the size of the input image");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
-   prop = RNA_def_property(srna, "relative", PROP_BOOLEAN, PROP_NONE);
-   RNA_def_property_boolean_sdna(prop, NULL, "custom2", 1);
-   RNA_def_property_ui_text(prop, "Relative", "Use relative values to crop image");
-   RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
+       prop = RNA_def_property(srna, "relative", PROP_BOOLEAN, PROP_NONE);
+       RNA_def_property_boolean_sdna(prop, NULL, "custom2", 1);
+       RNA_def_property_ui_text(prop, "Relative", "Use relative values to crop image");
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
        RNA_def_struct_sdna_from(srna, "NodeTwoXYs", "storage");
 
@@ -1832,29 +1874,29 @@ static void def_cmp_crop(StructRNA *srna)
        RNA_def_property_ui_text(prop, "Y2", "");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
-   prop = RNA_def_property(srna, "rel_min_x", PROP_FLOAT, PROP_NONE);
-   RNA_def_property_float_sdna(prop, NULL, "fac_x1");
-   RNA_def_property_range(prop, 0.0, 1.0);
-   RNA_def_property_ui_text(prop, "X1", "");
-   RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
+       prop = RNA_def_property(srna, "rel_min_x", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "fac_x1");
+       RNA_def_property_range(prop, 0.0, 1.0);
+       RNA_def_property_ui_text(prop, "X1", "");
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
-   prop = RNA_def_property(srna, "rel_max_x", PROP_FLOAT, PROP_NONE);
-   RNA_def_property_float_sdna(prop, NULL, "fac_x2");
-   RNA_def_property_range(prop, 0.0, 1.0);
-   RNA_def_property_ui_text(prop, "X2", "");
-   RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
+       prop = RNA_def_property(srna, "rel_max_x", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "fac_x2");
+       RNA_def_property_range(prop, 0.0, 1.0);
+       RNA_def_property_ui_text(prop, "X2", "");
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
-   prop = RNA_def_property(srna, "rel_min_y", PROP_FLOAT, PROP_NONE);
-   RNA_def_property_float_sdna(prop, NULL, "fac_y1");
-   RNA_def_property_range(prop, 0.0, 1.0);
-   RNA_def_property_ui_text(prop, "Y1", "");
-   RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
+       prop = RNA_def_property(srna, "rel_min_y", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "fac_y1");
+       RNA_def_property_range(prop, 0.0, 1.0);
+       RNA_def_property_ui_text(prop, "Y1", "");
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 
-   prop = RNA_def_property(srna, "rel_max_y", PROP_FLOAT, PROP_NONE);
-   RNA_def_property_float_sdna(prop, NULL, "fac_y2");
-   RNA_def_property_range(prop, 0.0, 1.0);
-   RNA_def_property_ui_text(prop, "Y2", "");
-   RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
+       prop = RNA_def_property(srna, "rel_max_y", PROP_FLOAT, PROP_NONE);
+       RNA_def_property_float_sdna(prop, NULL, "fac_y2");
+       RNA_def_property_range(prop, 0.0, 1.0);
+       RNA_def_property_ui_text(prop, "Y2", "");
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
 }
 
 static void def_cmp_dblur(StructRNA *srna)
@@ -2461,12 +2503,6 @@ static void rna_def_node_socket(BlenderRNA *brna)
        StructRNA *srna;
        PropertyRNA *prop;
 
-       static EnumPropertyItem node_socket_type_items[] = {
-               {SOCK_VALUE,  "VALUE",     0,    "Value",     ""},
-               {SOCK_VECTOR, "VECTOR",    0,    "Vector",    ""},
-               {SOCK_RGBA,   "RGBA",      0,    "RGBA",      ""},
-               {0, NULL, 0, NULL, NULL}};
-
        srna = RNA_def_struct(brna, "NodeSocket", NULL);
        RNA_def_struct_ui_text(srna, "Node Socket", "Input or output socket of a node");
        RNA_def_struct_refine_func(srna, "rna_NodeSocketType_refine");
@@ -2475,9 +2511,11 @@ static void rna_def_node_socket(BlenderRNA *brna)
        RNA_def_struct_path_func(srna, "rna_NodeSocket_path");
 
        prop = RNA_def_property(srna, "name", PROP_STRING, PROP_NONE);
-       RNA_def_property_clear_flag(prop, PROP_EDITABLE);
+       /* XXX must be editable for group sockets. if necessary use a special rna definition for these */
+//     RNA_def_property_clear_flag(prop, PROP_EDITABLE);
        RNA_def_property_ui_text(prop, "Name", "Socket name");
        RNA_def_struct_name_property(srna, prop);
+       RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_NodeGroupSocket_update");
 
        /* can add back if there is any use in reading them */
 #if 0
@@ -2577,7 +2615,7 @@ static void rna_def_node(BlenderRNA *brna)
        RNA_def_property_update(prop, NC_NODE, "rna_Node_update");
        
        prop = RNA_def_property(srna, "name", PROP_STRING, PROP_NONE);
-       RNA_def_property_ui_text(prop, "Name", "Node name");
+       RNA_def_property_ui_text(prop, "Name", "Unique node identifier");
        RNA_def_struct_name_property(srna, prop);
        RNA_def_property_string_funcs(prop, NULL, NULL, "rna_Node_name_set");
        RNA_def_property_update(prop, NC_NODE|NA_EDITED, "rna_Node_update");
@@ -2591,6 +2629,11 @@ static void rna_def_node(BlenderRNA *brna)
        RNA_def_property_collection_sdna(prop, NULL, "outputs", NULL);
        RNA_def_property_struct_type(prop, "NodeSocket");
        RNA_def_property_ui_text(prop, "Outputs", "");
+       
+       prop = RNA_def_property(srna, "label", PROP_STRING, PROP_NONE);
+       RNA_def_property_string_sdna(prop, NULL, "label");
+       RNA_def_property_ui_text(prop, "Label", "Optional custom node label");
+       RNA_def_property_update(prop, NC_NODE, "rna_Node_update");
 }
 
 static void rna_def_node_link(BlenderRNA *brna)
@@ -2624,12 +2667,41 @@ static void rna_def_node_link(BlenderRNA *brna)
        RNA_def_property_ui_text(prop, "To socket", "");
 }
 
+static void rna_def_group_sockets_api(BlenderRNA *brna, PropertyRNA *cprop, int in_out)
+{
+       StructRNA *srna;
+       PropertyRNA *parm;
+       FunctionRNA *func;
+
+       RNA_def_property_srna(cprop, (in_out==SOCK_IN ? "GroupInputs" : "GroupOutputs"));
+       srna= RNA_def_struct(brna, (in_out==SOCK_IN ? "GroupInputs" : "GroupOutputs"), NULL);
+       RNA_def_struct_sdna(srna, "bNodeTree");
+       RNA_def_struct_ui_text(srna, "Group Sockets", "Collection of group sockets");
+
+       func= RNA_def_function(srna, "new", (in_out==SOCK_IN ? "rna_NodeTree_input_new" : "rna_NodeTree_output_new"));
+       RNA_def_function_ui_description(func, "Add a socket to the group tree.");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
+       RNA_def_string(func, "name", "Socket", 32, "Name", "Name of the socket");
+       RNA_def_enum(func, "type", node_socket_type_items, SOCK_VALUE, "Type", "Type of socket");
+       /* return value */
+       parm= RNA_def_pointer(func, "socket", "NodeSocket", "", "New socket.");
+       RNA_def_function_return(func, parm);
+
+       func= RNA_def_function(srna, "expose", (in_out==SOCK_IN ? "rna_NodeTree_input_expose" : "rna_NodeTree_output_expose"));
+       RNA_def_function_ui_description(func, "Expose an internal socket in the group tree.");
+       RNA_def_function_flag(func, FUNC_USE_REPORTS);
+       RNA_def_pointer(func, "sock", "NodeSocket", "Socket", "Internal node socket to expose");
+       RNA_def_property_flag(parm, PROP_REQUIRED);
+       RNA_def_boolean(func, "add_link", TRUE, "Add Link", "If TRUE, adds a link to the internal socket");
+       /* return value */
+       parm= RNA_def_pointer(func, "socket", "NodeSocket", "", "New socket.");
+       RNA_def_function_return(func, parm);
+}
+
 static void rna_def_nodetree(BlenderRNA *brna)
 {
        StructRNA *srna;
        PropertyRNA *prop;
-       /* FunctionRNA *func; */
-       /* PropertyRNA *parm; */
 
        static EnumPropertyItem nodetree_type_items[] = {
                {NTREE_SHADER,      "SHADER",       0,    "Shader",       ""},
@@ -2659,29 +2731,24 @@ static void rna_def_nodetree(BlenderRNA *brna)
        RNA_def_property_flag(prop, PROP_EDITABLE);
        RNA_def_property_struct_type(prop, "GreasePencil");
        RNA_def_property_ui_text(prop, "Grease Pencil Data", "Grease Pencil datablock");
-
-       /* these are too much like operators, better to have data level access 
-        * ngroup = bpy.data.node_groups.new() 
-        * ngroup.nodes.new(....) etc. */
-#if 0
-       func= RNA_def_function(srna, "group_add", "nodeMakeGroupFromSelected");
-       RNA_def_function_ui_description(func, "Make a group from the active nodes.");
-       /* return */
-       parm= RNA_def_pointer(func, "group", "Node", "", "New group.");
-       RNA_def_function_return(func, parm);
-
-       func= RNA_def_function(srna, "ungroup", "nodeGroupUnGroup");
-       RNA_def_function_ui_description(func, "Ungroup node group");
-       parm= RNA_def_pointer(func, "group", "Node", "", "The group to ungroup.");
-       RNA_def_property_flag(parm, PROP_REQUIRED);
-       parm= RNA_def_int(func, "bool", 0, 0, 1, "Bool", "", 0, 1);
-       RNA_def_function_return(func, parm);
-#endif
        
        prop = RNA_def_property(srna, "type", PROP_ENUM, PROP_NONE);
        RNA_def_property_clear_flag(prop, PROP_EDITABLE);
        RNA_def_property_enum_items(prop, nodetree_type_items);
        RNA_def_property_ui_text(prop, "Type", "Node Tree type");
+
+       /* group sockets */
+       prop = RNA_def_property(srna, "inputs", PROP_COLLECTION, PROP_NONE);
+       RNA_def_property_collection_sdna(prop, NULL, "inputs", NULL);
+       RNA_def_property_struct_type(prop, "NodeSocket");
+       RNA_def_property_ui_text(prop, "Inputs", "");
+       rna_def_group_sockets_api(brna, prop, SOCK_IN);
+       
+       prop = RNA_def_property(srna, "outputs", PROP_COLLECTION, PROP_NONE);
+       RNA_def_property_collection_sdna(prop, NULL, "outputs", NULL);
+       RNA_def_property_struct_type(prop, "NodeSocket");
+       RNA_def_property_ui_text(prop, "Outputs", "");
+       rna_def_group_sockets_api(brna, prop, SOCK_OUT);
 }
 
 static void rna_def_composite_nodetree(BlenderRNA *brna)