[Bf-blender-cvs] [7029810b04c] master: GPencil: Add new Distance property to Simplify modifier

Antonio Vazquez noreply at git.blender.org
Sat Aug 10 17:16:21 CEST 2019


Commit: 7029810b04c37e8dd0a7256047038fc6f8192309
Author: Antonio Vazquez
Date:   Sat Aug 10 17:16:12 2019 +0200
Branches: master
https://developer.blender.org/rB7029810b04c37e8dd0a7256047038fc6f8192309

GPencil: Add new Distance property to Simplify modifier

Before, the length variable was reused for Sample and Merge Simplify. Now, the Merge has its own distance property and variable in the modifier struct.

===================================================================

M	release/scripts/startup/bl_ui/properties_data_modifier.py
M	source/blender/gpencil_modifiers/intern/MOD_gpencilsimplify.c
M	source/blender/makesdna/DNA_gpencil_modifier_types.h
M	source/blender/makesrna/intern/rna_gpencil_modifier.c

===================================================================

diff --git a/release/scripts/startup/bl_ui/properties_data_modifier.py b/release/scripts/startup/bl_ui/properties_data_modifier.py
index 316ce818530..1d1ee2e52be 100644
--- a/release/scripts/startup/bl_ui/properties_data_modifier.py
+++ b/release/scripts/startup/bl_ui/properties_data_modifier.py
@@ -1793,7 +1793,7 @@ class DATA_PT_gpencil_modifiers(ModifierButtonsPanel, Panel):
         elif md.mode == 'SAMPLE':
             col.prop(md, "length")
         elif md.mode == 'MERGE':
-            col.prop(md, "length", text="Threshold")
+            col.prop(md, "distance")
 
         col = layout.column()
         col.separator()
diff --git a/source/blender/gpencil_modifiers/intern/MOD_gpencilsimplify.c b/source/blender/gpencil_modifiers/intern/MOD_gpencilsimplify.c
index a27fb27d518..2168e7f07ec 100644
--- a/source/blender/gpencil_modifiers/intern/MOD_gpencilsimplify.c
+++ b/source/blender/gpencil_modifiers/intern/MOD_gpencilsimplify.c
@@ -46,6 +46,7 @@ static void initData(GpencilModifierData *md)
   gpmd->step = 1;
   gpmd->factor = 0.0f;
   gpmd->length = 0.1f;
+  gpmd->distance = 0.1f;
   gpmd->layername[0] = '\0';
 }
 
@@ -94,7 +95,7 @@ static void deformStroke(GpencilModifierData *md,
       break;
     }
     case GP_SIMPLIFY_MERGE: {
-      BKE_gpencil_merge_distance_stroke(gpf, gps, mmd->length, true);
+      BKE_gpencil_merge_distance_stroke(gpf, gps, mmd->distance, true);
       break;
     }
     default:
diff --git a/source/blender/makesdna/DNA_gpencil_modifier_types.h b/source/blender/makesdna/DNA_gpencil_modifier_types.h
index 82628065014..83ee8be7f68 100644
--- a/source/blender/makesdna/DNA_gpencil_modifier_types.h
+++ b/source/blender/makesdna/DNA_gpencil_modifier_types.h
@@ -502,8 +502,11 @@ typedef struct SimplifyGpencilModifierData {
   short step;
   /** Custom index for passes. */
   int layer_pass;
-  /* Sample length */
+  /** Sample length */
   float length;
+  /** Merge distance */
+  float distance;
+  char _pad[4];
 } SimplifyGpencilModifierData;
 
 typedef enum eSimplifyGpencil_Flag {
diff --git a/source/blender/makesrna/intern/rna_gpencil_modifier.c b/source/blender/makesrna/intern/rna_gpencil_modifier.c
index 48ef4b6c29e..de24b5d3f4b 100644
--- a/source/blender/makesrna/intern/rna_gpencil_modifier.c
+++ b/source/blender/makesrna/intern/rna_gpencil_modifier.c
@@ -689,9 +689,16 @@ static void rna_def_modifier_gpencilsimplify(BlenderRNA *brna)
   /* Sample */
   prop = RNA_def_property(srna, "length", PROP_FLOAT, PROP_NONE);
   RNA_def_property_float_sdna(prop, NULL, "length");
-  RNA_def_property_range(prop, 0, 10);
+  RNA_def_property_range(prop, 0, 10.0f);
   RNA_def_property_ui_text(prop, "Length", "Length of each segment");
   RNA_def_property_update(prop, 0, "rna_GpencilModifier_update");
+
+  /* Distance */
+  prop = RNA_def_property(srna, "distance", PROP_FLOAT, PROP_NONE);
+  RNA_def_property_float_sdna(prop, NULL, "distance");
+  RNA_def_property_range(prop, 0, 100.0f);
+  RNA_def_property_ui_text(prop, "Distance", "Distance between vertex");
+  RNA_def_property_update(prop, 0, "rna_GpencilModifier_update");
 }
 
 static void rna_def_modifier_gpencilthick(BlenderRNA *brna)



More information about the Bf-blender-cvs mailing list