SVN maintenance.
authorGuillermo S. Romero <gsr.b3d@infernal-iceberg.com>
Fri, 9 Sep 2011 00:46:00 +0000 (00:46 +0000)
committerGuillermo S. Romero <gsr.b3d@infernal-iceberg.com>
Fri, 9 Sep 2011 00:46:00 +0000 (00:46 +0000)
source/blender/makesdna/DNA_modifier_types.h
source/blender/modifiers/intern/MOD_weightvg_util.c
source/blender/modifiers/intern/MOD_weightvg_util.h
source/blender/modifiers/intern/MOD_weightvgedit.c
source/blender/modifiers/intern/MOD_weightvgmix.c
source/blender/modifiers/intern/MOD_weightvgproximity.c

index 32a78cb823a275d05a7214cee6f508c9bb58760d..d09e56a5708cc1d1941b1d5d4b46d6dd4260c92d 100644 (file)
@@ -818,7 +818,7 @@ typedef struct WeightVGEditModifierData {
        int             mask_tex_mapping;
        char    mask_tex_uvlayer_name[32]; /* Name of the UV layer. */
 
-       /* Padding */
+       /* Padding... */
        int pad_i1;
 } WeightVGEditModifierData;
 
@@ -863,7 +863,7 @@ typedef struct WeightVGMixModifierData {
        int             mask_tex_mapping;          /* How to map the texture! */
        char    mask_tex_uvlayer_name[32]; /* Name of the UV layer. */
 
-       /* Padding */
+       /* Padding... */
        int pad_i1;
 } WeightVGMixModifierData;
 
index 51465cc80b09e90335ac3f713fec1322162a3c47..98615c705531b5f5c1873c9be6424e2202a2e556 100644 (file)
@@ -135,7 +135,7 @@ void weightvg_do_mask(int num, const int *indices, float *org_w, const float *ne
                MappingInfoModifierData t_map;
                float (*v_co)[3];
 
-               /* Use new generic get_texture_coords, but do not modify our DNA struct for it
+               /* Use new generic get_texture_coords, but do not modify our DNA struct for it...
                 * XXX Why use a ModifierData stuff here ? Why not a simple, generic struct for parameters ?
                 *     What e.g. if a modifier wants to use several textures ?
                 *     Why use only v_co, and not MVert (or both) ?
index 00c63a3d5d5c83eb3ef6cd03987e447abdf38bce..ce3520f19000ad2162bd73ac4513f14b5ec98bfc 100644 (file)
@@ -44,7 +44,7 @@ struct Tex;
 
 /*
  * XXX I'd like to make modified weights visible in WeightPaint mode,
- *     but couldn't figure a way to do this
+ *     but couldn't figure a way to do this...
  *     Maybe this will need changes in mesh_calc_modifiers (DerivedMesh.c)?
  *     Or the WeightPaint mode code itself?
  */
@@ -53,7 +53,7 @@ struct Tex;
  * Util functions.                    *
  **************************************/
 
-/* We cannot divide by zero (what a surprise).
+/* We cannot divide by zero (what a surprise...).
  * So if -MOD_WEIGHTVGROUP_DIVMODE_ZEROFLOOR < weightf < MOD_WEIGHTVGROUP_DIVMODE_ZEROFLOOR,
  * we clamp weightf to this value (or its negative version).
  * Also used to avoid null power factor.
index 0b6b69b2b3c62a8434e4e0d567f42e56cd838eef..fb6d4dc10e6c18be92e67998b1e38874af21c2be 100644 (file)
@@ -28,7 +28,7 @@
 
 /*
  * XXX I'd like to make modified weights visible in WeightPaint mode,
- *     but couldn't figure a way to do this
+ *     but couldn't figure a way to do this...
  *     Maybe this will need changes in mesh_calc_modifiers (DerivedMesh.c)?
  *     Or the WeightPaint mode code itself?
  */
@@ -211,13 +211,13 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob, DerivedMesh *der
        if (defgrp_idx < 0)
                return dm;
 
-       /* XXX All this to avoid copying dm when not needed However, it nearly doubles compute
-        *     time! See scene 5 of the WeighVG test file
+       /* XXX All this to avoid copying dm when not needed... However, it nearly doubles compute
+        *     time! See scene 5 of the WeighVG test file...
         */
 #if 0
        /* Get actual dverts (ie vertex group data). */
        dvert = dm->getVertDataArray(dm, CD_MDEFORMVERT);
-       /* If no dverts, return unmodified data */
+       /* If no dverts, return unmodified data... */
        if (dvert == NULL)
                return dm;
 
@@ -231,7 +231,7 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob, DerivedMesh *der
        /* Create a copy of our dmesh, only if our affected cdata layer is the same as org mesh. */
        if (dvert == CustomData_get_layer(&ob_m->vdata, CD_MDEFORMVERT)) {
                /* XXX Seems to create problems with weightpaint mode???
-                *     I'm missing something here, I guess
+                *     I'm missing something here, I guess...
                 */
 //             DM_set_only_copy(dm, CD_MASK_MDEFORMVERT); /* Only copy defgroup layer. */
                ret = CDDM_copy(dm);
index 8bfc56fc18fbce0fcf0f84e253800af5982aba3e..f1422a342ebf6a4c7746575a2fcc63ab06d5374a 100644 (file)
@@ -28,7 +28,7 @@
 
 /*
  * XXX I'd like to make modified weights visible in WeightPaint mode,
- *     but couldn't figure a way to do this
+ *     but couldn't figure a way to do this...
  *     Maybe this will need changes in mesh_calc_modifiers (DerivedMesh.c)?
  *     Or the WeightPaint mode code itself?
  */
@@ -62,7 +62,7 @@ static float mix_weight(float weight, float weight2, char mix_mode)
 #if 0
        /*
         * XXX Don't know why, but the switch version takes many CPU time,
-        *     and produces lag in realtime playback
+        *     and produces lag in realtime playback...
         */
        switch (mix_mode)
        {
@@ -258,13 +258,13 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob, DerivedMesh *der
                        return dm;
        }
 
-       /* XXX All this to avoid copying dm when not needed However, it nearly doubles compute
-        *     time! See scene 5 of the WeighVG test file
+       /* XXX All this to avoid copying dm when not needed... However, it nearly doubles compute
+        *     time! See scene 5 of the WeighVG test file...
         */
 #if 0
        /* Get actual dverts (ie vertex group data). */
        dvert = dm->getVertDataArray(dm, CD_MDEFORMVERT);
-       /* If no dverts, return unmodified data */
+       /* If no dverts, return unmodified data... */
        if (dvert == NULL)
                return dm;
 
@@ -278,7 +278,7 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob, DerivedMesh *der
        /* Create a copy of our dmesh, only if our affected cdata layer is the same as org mesh. */
        if (dvert == CustomData_get_layer(&ob_m->vdata, CD_MDEFORMVERT)) {
                /* XXX Seems to create problems with weightpaint mode???
-                *     I'm missing something here, I guess
+                *     I'm missing something here, I guess...
                 */
 //             DM_set_only_copy(dm, CD_MASK_MDEFORMVERT); /* Only copy defgroup layer. */
                ret = CDDM_copy(dm);
index 0dbf8a91d214ec307d47f360cd3beb81805236b1..af3f71d609978aea3ccdf709667d78a1cbb23a6a 100644 (file)
@@ -28,7 +28,7 @@
 
 /*
  * XXX I'd like to make modified weights visible in WeightPaint mode,
- *     but couldn't figure a way to do this
+ *     but couldn't figure a way to do this...
  *     Maybe this will need changes in mesh_calc_modifiers (DerivedMesh.c)?
  *     Or the WeightPaint mode code itself?
  */
@@ -385,13 +385,13 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob, DerivedMesh *der
        if (defgrp_idx < 0)
                return dm;
 
-       /* XXX All this to avoid copying dm when not needed However, it nearly doubles compute
-        *     time! See scene 5 of the WeighVG test file
+       /* XXX All this to avoid copying dm when not needed... However, it nearly doubles compute
+        *     time! See scene 5 of the WeighVG test file...
         */
 #if 0
        /* Get actual dverts (ie vertex group data). */
        dvert = dm->getVertDataArray(dm, CD_MDEFORMVERT);
-       /* If no dverts, return unmodified data */
+       /* If no dverts, return unmodified data... */
        if (dvert == NULL)
                return dm;
 
@@ -405,7 +405,7 @@ static DerivedMesh *applyModifier(ModifierData *md, Object *ob, DerivedMesh *der
        /* Create a copy of our dmesh, only if our affected cdata layer is the same as org mesh. */
        if (dvert == CustomData_get_layer(&ob_m->vdata, CD_MDEFORMVERT)) {
                /* XXX Seems to create problems with weightpaint mode???
-                *     I'm missing something here, I guess
+                *     I'm missing something here, I guess...
                 */
 //             DM_set_only_copy(dm, CD_MASK_MDEFORMVERT); /* Only copy defgroup layer. */
                ret = CDDM_copy(dm);