Fix T68943: GPencil Time modifier gets strange value in offset parameter
This is due a limitation in the RNA property when the range is too extreme. As we don't need that, the value was set to SHRT_MAX frames as maximum offset. Also fixed the same problem in other modules of Grease Pencil.
This commit is contained in:
parent
6e8a76c96a
commit
8f50cdd7d5
Notes:
blender-bot
2023-02-14 01:11:05 +01:00
Referenced by issue #68943, GP time offset modifier, frame offset value changes to -2147483647 when using the inc/dec icons
|
@ -1649,7 +1649,7 @@ static void rna_def_gpencil_grid(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "lines", PROP_INT, PROP_NONE);
|
||||
RNA_def_property_int_sdna(prop, NULL, "lines");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_int_default(prop, GP_DEFAULT_GRID_LINES);
|
||||
RNA_def_property_ui_text(
|
||||
prop, "Grid Subdivisions", "Number of subdivisions in each side of symmetry line");
|
||||
|
|
|
@ -1023,7 +1023,7 @@ static void rna_def_modifier_gpenciltime(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "offset", PROP_INT, PROP_NONE);
|
||||
RNA_def_property_int_sdna(prop, NULL, "offset");
|
||||
RNA_def_property_range(prop, -INT_MAX, INT_MAX);
|
||||
RNA_def_property_range(prop, SHRT_MIN, SHRT_MAX);
|
||||
RNA_def_property_ui_text(
|
||||
prop, "Frame Offset", "Number of frames to offset original keyframe number or frame to fix");
|
||||
RNA_def_property_update(prop, 0, "rna_GpencilModifier_update");
|
||||
|
@ -1256,7 +1256,7 @@ static void rna_def_modifier_gpencilinstance(BlenderRNA *brna)
|
|||
RNA_def_property_update(prop, 0, "rna_GpencilModifier_update");
|
||||
|
||||
prop = RNA_def_property(srna, "count", PROP_INT, PROP_NONE);
|
||||
RNA_def_property_range(prop, 1, INT_MAX);
|
||||
RNA_def_property_range(prop, 1, SHRT_MAX);
|
||||
RNA_def_property_ui_range(prop, 1, 50, 1, -1);
|
||||
RNA_def_property_ui_text(prop, "Count", "Number of items");
|
||||
RNA_def_property_update(prop, 0, "rna_GpencilModifier_update");
|
||||
|
@ -1321,7 +1321,7 @@ static void rna_def_modifier_gpencilinstance(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "replace_material", PROP_INT, PROP_NONE);
|
||||
RNA_def_property_int_sdna(prop, NULL, "mat_rpl");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_ui_text(
|
||||
prop,
|
||||
"Material",
|
||||
|
|
|
@ -212,7 +212,7 @@ static void rna_def_shader_fx_blur(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "factor", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "radius");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_ui_text(prop, "Factor", "Factor of Blur");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
||||
|
@ -329,7 +329,7 @@ static void rna_def_shader_fx_pixel(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "size", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "size");
|
||||
RNA_def_property_range(prop, 1, INT_MAX);
|
||||
RNA_def_property_range(prop, 1, SHRT_MAX);
|
||||
RNA_def_property_array(prop, 2);
|
||||
RNA_def_property_ui_text(prop, "Size", "Pixel size");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
@ -359,7 +359,7 @@ static void rna_def_shader_fx_rim(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "offset", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "offset");
|
||||
RNA_def_property_range(prop, -INT_MAX, INT_MAX);
|
||||
RNA_def_property_range(prop, SHRT_MIN, SHRT_MAX);
|
||||
RNA_def_property_ui_text(prop, "Offset", "Offset of the rim");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
||||
|
@ -385,7 +385,7 @@ static void rna_def_shader_fx_rim(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "blur", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "blur");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_ui_text(
|
||||
prop, "Blur", "Number of pixels for blurring rim (set to 0 to disable)");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
@ -423,7 +423,7 @@ static void rna_def_shader_fx_shadow(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "offset", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "offset");
|
||||
RNA_def_property_range(prop, -INT_MAX, INT_MAX);
|
||||
RNA_def_property_range(prop, SHRT_MIN, SHRT_MAX);
|
||||
RNA_def_property_ui_text(prop, "Offset", "Offset of the shadow");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
||||
|
@ -473,7 +473,7 @@ static void rna_def_shader_fx_shadow(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "blur", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "blur");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_ui_text(
|
||||
prop, "Blur", "Number of pixels for blurring shadow (set to 0 to disable)");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
@ -538,7 +538,7 @@ static void rna_def_shader_fx_glow(BlenderRNA *brna)
|
|||
* but only makes public first array element. */
|
||||
prop = RNA_def_property(srna, "radius", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "blur[0]");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_ui_text(
|
||||
prop, "Radius", "Number of pixels for blurring glow (set to 0 to disable)");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
@ -569,7 +569,7 @@ static void rna_def_shader_fx_swirl(BlenderRNA *brna)
|
|||
|
||||
prop = RNA_def_property(srna, "radius", PROP_INT, PROP_PIXEL);
|
||||
RNA_def_property_int_sdna(prop, NULL, "radius");
|
||||
RNA_def_property_range(prop, 0, INT_MAX);
|
||||
RNA_def_property_range(prop, 0, SHRT_MAX);
|
||||
RNA_def_property_ui_text(prop, "Radius", "Radius to apply");
|
||||
RNA_def_property_update(prop, NC_OBJECT | ND_MODIFIER, "rna_ShaderFx_update");
|
||||
|
||||
|
|
Loading…
Reference in New Issue