CustomData: deprecate CD_ID_MCOL
authorCampbell Barton <ideasman42@gmail.com>
Fri, 10 Jul 2015 06:47:39 +0000 (16:47 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Fri, 10 Jul 2015 06:47:39 +0000 (16:47 +1000)
source/blender/blenkernel/intern/cdderivedmesh.c
source/blender/blenkernel/intern/customdata.c
source/blender/makesdna/DNA_customdata_types.h
source/blender/makesdna/DNA_object_types.h
source/gameengine/Converter/BL_BlenderDataConversion.cpp

index 2250db1327ee867515667bd78e4f6260f4e90af7..d60c5a52dbadc5572fe4914d9f7c24100ab7e4bd 100644 (file)
@@ -623,8 +623,7 @@ static void cdDM_drawMappedFaces(DerivedMesh *dm,
                index_mp_to_orig = NULL;
        }
 
-
-       colType = CD_ID_MCOL;
+       colType = CD_TEXTURE_MCOL;
        mcol = DM_get_tessface_data_layer(dm, colType);
        if (!mcol) {
                colType = CD_PREVIEW_MCOL;
index eced263d493f2d0ef20674110ae12d5b03c3514a..bdc128915a5193464cfadc3f73dd132cae64754f 100644 (file)
@@ -1199,8 +1199,7 @@ static const LayerTypeInfo LAYERTYPEINFO[CD_NUMTYPES] = {
        /* 0: CD_MVERT */
        {sizeof(MVert), "MVert", 1, NULL, NULL, NULL, NULL, NULL, NULL},
        /* 1: CD_MSTICKY */  /* DEPRECATED */
-       {sizeof(float) * 2, "", 1, NULL, NULL, NULL, NULL, NULL,
-        NULL},
+       {sizeof(float) * 2, "", 1, NULL, NULL, NULL, NULL, NULL, NULL},
        /* 2: CD_MDEFORMVERT */
        {sizeof(MDeformVert), "MDeformVert", 1, NULL, layerCopy_mdeformvert,
         layerFree_mdeformvert, layerInterp_mdeformvert, NULL, NULL},
@@ -1221,7 +1220,7 @@ static const LayerTypeInfo LAYERTYPEINFO[CD_NUMTYPES] = {
        /* 3 floats per normal vector */
        {sizeof(float) * 3, "vec3f", 1, NULL, NULL, NULL, layerInterp_normal, NULL, NULL,
         NULL, NULL, NULL, NULL, NULL, layerCopyValue_normal},
-       /* 9: CD_POLYINDEX (deprecated) */
+       /* 9: CD_POLYINDEX */  /* DEPRECATED */
        {sizeof(int), "", 0, NULL, NULL, NULL, NULL, NULL, NULL},
        /* 10: CD_PROP_FLT */
        {sizeof(MFloatProperty), "MFloatProperty", 1, N_("Float"), layerCopy_propFloat, NULL, NULL, NULL},
@@ -1256,9 +1255,8 @@ static const LayerTypeInfo LAYERTYPEINFO[CD_NUMTYPES] = {
        /* 20: CD_PREVIEW_MCOL */
        {sizeof(MCol) * 4, "MCol", 4, N_("PreviewCol"), NULL, NULL, layerInterp_mcol,
         layerSwap_mcol, layerDefault_mcol},
-       /* 21: CD_ID_MCOL */
-       {sizeof(MCol) * 4, "MCol", 4, N_("IDCol"), NULL, NULL, layerInterp_mcol,
-        layerSwap_mcol, layerDefault_mcol},
+       /* 21: CD_ID_MCOL */  /* DEPRECATED */
+       {sizeof(MCol) * 4, "", 0, NULL, NULL, NULL, NULL, NULL, NULL},
        /* 22: CD_TEXTURE_MCOL */
        {sizeof(MCol) * 4, "MCol", 4, N_("TexturedCol"), NULL, NULL, layerInterp_mcol,
         layerSwap_mcol, layerDefault_mcol},
@@ -1336,20 +1334,20 @@ const CustomDataMask CD_MASK_BAREMESH =
     CD_MASK_MVERT | CD_MASK_MEDGE | CD_MASK_MFACE | CD_MASK_MLOOP | CD_MASK_MPOLY | CD_MASK_BWEIGHT;
 const CustomDataMask CD_MASK_MESH =
     CD_MASK_MVERT | CD_MASK_MEDGE | CD_MASK_MFACE |
-    CD_MASK_MSTICKY | CD_MASK_MDEFORMVERT | CD_MASK_MTFACE | CD_MASK_MCOL |
+    CD_MASK_MDEFORMVERT | CD_MASK_MTFACE | CD_MASK_MCOL |
     CD_MASK_PROP_FLT | CD_MASK_PROP_INT | CD_MASK_PROP_STR | CD_MASK_MDISPS |
     CD_MASK_MLOOPUV | CD_MASK_MLOOPCOL | CD_MASK_MPOLY | CD_MASK_MLOOP |
     CD_MASK_MTEXPOLY | CD_MASK_RECAST | CD_MASK_PAINT_MASK |
     CD_MASK_GRID_PAINT_MASK | CD_MASK_MVERT_SKIN | CD_MASK_FREESTYLE_EDGE | CD_MASK_FREESTYLE_FACE |
     CD_MASK_CUSTOMLOOPNORMAL;
 const CustomDataMask CD_MASK_EDITMESH =
-    CD_MASK_MSTICKY | CD_MASK_MDEFORMVERT | CD_MASK_MTFACE | CD_MASK_MLOOPUV |
+    CD_MASK_MDEFORMVERT | CD_MASK_MTFACE | CD_MASK_MLOOPUV |
     CD_MASK_MLOOPCOL | CD_MASK_MTEXPOLY | CD_MASK_SHAPE_KEYINDEX |
     CD_MASK_MCOL | CD_MASK_PROP_FLT | CD_MASK_PROP_INT | CD_MASK_PROP_STR |
     CD_MASK_MDISPS | CD_MASK_SHAPEKEY | CD_MASK_RECAST | CD_MASK_PAINT_MASK |
     CD_MASK_GRID_PAINT_MASK | CD_MASK_MVERT_SKIN | CD_MASK_CUSTOMLOOPNORMAL;
 const CustomDataMask CD_MASK_DERIVEDMESH =
-    CD_MASK_MSTICKY | CD_MASK_MDEFORMVERT | CD_MASK_MTFACE |
+    CD_MASK_MDEFORMVERT | CD_MASK_MTFACE |
     CD_MASK_MCOL | CD_MASK_PROP_FLT | CD_MASK_PROP_INT | CD_MASK_CLOTH_ORCO |
     CD_MASK_MLOOPUV | CD_MASK_MLOOPCOL | CD_MASK_MTEXPOLY | CD_MASK_PREVIEW_MLOOPCOL |
     CD_MASK_PROP_STR | CD_MASK_ORIGSPACE | CD_MASK_ORIGSPACE_MLOOP | CD_MASK_ORCO | CD_MASK_TANGENT |
@@ -1358,7 +1356,7 @@ const CustomDataMask CD_MASK_DERIVEDMESH =
     CD_MASK_CUSTOMLOOPNORMAL;
 const CustomDataMask CD_MASK_BMESH =
     CD_MASK_MLOOPUV | CD_MASK_MLOOPCOL | CD_MASK_MTEXPOLY |
-    CD_MASK_MSTICKY | CD_MASK_MDEFORMVERT | CD_MASK_PROP_FLT | CD_MASK_PROP_INT |
+    CD_MASK_MDEFORMVERT | CD_MASK_PROP_FLT | CD_MASK_PROP_INT |
     CD_MASK_PROP_STR | CD_MASK_SHAPEKEY | CD_MASK_SHAPE_KEYINDEX | CD_MASK_MDISPS |
     CD_MASK_CREASE | CD_MASK_BWEIGHT | CD_MASK_RECAST | CD_MASK_PAINT_MASK |
     CD_MASK_GRID_PAINT_MASK | CD_MASK_MVERT_SKIN | CD_MASK_FREESTYLE_EDGE | CD_MASK_FREESTYLE_FACE |
@@ -1367,7 +1365,7 @@ const CustomDataMask CD_MASK_FACECORNERS =  /* XXX Not used anywhere! */
     CD_MASK_MTFACE | CD_MASK_MCOL | CD_MASK_MTEXPOLY | CD_MASK_MLOOPUV |
     CD_MASK_MLOOPCOL | CD_MASK_NORMAL | CD_MASK_MLOOPTANGENT;
 const CustomDataMask CD_MASK_EVERYTHING =
-    CD_MASK_MVERT | CD_MASK_MSTICKY /* DEPRECATED */ | CD_MASK_MDEFORMVERT | CD_MASK_MEDGE | CD_MASK_MFACE |
+    CD_MASK_MVERT | CD_MASK_MDEFORMVERT | CD_MASK_MEDGE | CD_MASK_MFACE |
     CD_MASK_MTFACE | CD_MASK_MCOL | CD_MASK_ORIGINDEX | CD_MASK_NORMAL /* | CD_MASK_POLYINDEX */ | CD_MASK_PROP_FLT |
     CD_MASK_PROP_INT | CD_MASK_PROP_STR | CD_MASK_ORIGSPACE | CD_MASK_ORCO | CD_MASK_MTEXPOLY | CD_MASK_MLOOPUV |
     CD_MASK_MLOOPCOL | CD_MASK_TANGENT | CD_MASK_MDISPS | CD_MASK_PREVIEW_MCOL | CD_MASK_CLOTH_ORCO | CD_MASK_RECAST |
index 74f5967db13ed8649f0bbf6a4d3e172982dab552..a92c8b3307c2ba779e84970e85aa96c7d54f44d1 100644 (file)
@@ -76,7 +76,9 @@ typedef struct CustomData {
 /* CustomData.type */
 typedef enum CustomDataType {
        CD_MVERT            = 0,
+#ifdef DNA_DEPRECATED
        CD_MSTICKY          = 1,  /* DEPRECATED */
+#endif
        CD_MDEFORMVERT      = 2,
        CD_MEDGE            = 3,
        CD_MFACE            = 4,
@@ -96,7 +98,7 @@ typedef enum CustomDataType {
        CD_TANGENT          = 18,
        CD_MDISPS           = 19,
        CD_PREVIEW_MCOL     = 20,  /* for displaying weightpaint colors */
-       CD_ID_MCOL          = 21,
+/*     CD_ID_MCOL          = 21, */
        CD_TEXTURE_MCOL     = 22,
        CD_CLOTH_ORCO       = 23,
        CD_RECAST           = 24,
@@ -127,7 +129,7 @@ typedef enum CustomDataType {
 
 /* Bits for CustomDataMask */
 #define CD_MASK_MVERT          (1 << CD_MVERT)
-#define CD_MASK_MSTICKY                (1 << CD_MSTICKY)  /* DEPRECATED */
+// #define CD_MASK_MSTICKY             (1 << CD_MSTICKY)  /* DEPRECATED */
 #define CD_MASK_MDEFORMVERT    (1 << CD_MDEFORMVERT)
 #define CD_MASK_MEDGE          (1 << CD_MEDGE)
 #define CD_MASK_MFACE          (1 << CD_MFACE)
index 60547ab522a0c6a6d953cb6e316f7a2875339614..0a454d4f28144167c302ece886ebe083fbb6b144 100644 (file)
@@ -382,9 +382,9 @@ enum {
        PARTYPE       = (1 << 4) - 1,
        PAROBJECT     = 0,
 #ifdef DNA_DEPRECATED
-       PARCURVE      = 1,
+       PARCURVE      = 1,  /* Deprecated. */
 #endif
-       // PARKEY        = 2,  /* UNUSED */
+       PARKEY        = 2,  /* XXX Unused, deprecated? */
 
        PARSKEL       = 4,
        PARVERT1      = 5,
index 8ccfeadea7bae9183b2a7648d0d416c3aed91597..04fa29c3b1a348e54b69dcc9ef30e72337be10e3 100644 (file)
@@ -2174,8 +2174,6 @@ void BL_ConvertBlenderObjects(struct Main* maggie,
                        case PARSKEL: // skinned - ignore
                                break;
                        case PAROBJECT:
-                       case PARCURVE:
-                       case PARKEY:
                        case PARVERT3:
                        default:
                                // unhandled