[Bf-blender-cvs] [29bf8d456e3] greasepencil-object: Cleanup: More style changes
Antonio Vazquez
noreply at git.blender.org
Wed Jun 13 16:18:41 CEST 2018
Commit: 29bf8d456e35e65b27f757e54e5fd0a998daffbd
Author: Antonio Vazquez
Date: Wed Jun 13 16:18:12 2018 +0200
Branches: greasepencil-object
https://developer.blender.org/rB29bf8d456e35e65b27f757e54e5fd0a998daffbd
Cleanup: More style changes
===================================================================
M source/blender/blenkernel/BKE_object.h
M source/blender/blenkernel/intern/object.c
M source/blender/depsgraph/intern/builder/deg_builder_relations.cc
===================================================================
diff --git a/source/blender/blenkernel/BKE_object.h b/source/blender/blenkernel/BKE_object.h
index 720c8cda818..20b9cfd2c07 100644
--- a/source/blender/blenkernel/BKE_object.h
+++ b/source/blender/blenkernel/BKE_object.h
@@ -114,7 +114,7 @@ struct Object *BKE_object_add_from(
ATTR_NONNULL(1, 2, 3, 6) ATTR_RETURNS_NONNULL;
struct Object *BKE_object_add_for_data(
struct Main *bmain, struct ViewLayer *view_layer,
- int type, const char *name, struct ID *data, bool add_user) ATTR_RETURNS_NONNULL;
+ int type, const char *name, struct ID *data, bool do_id_user) ATTR_RETURNS_NONNULL;
void *BKE_object_obdata_add_from_type(
struct Main *bmain,
int type, const char *name)
diff --git a/source/blender/blenkernel/intern/object.c b/source/blender/blenkernel/intern/object.c
index a4d38d74d31..4a0e3eff0c0 100644
--- a/source/blender/blenkernel/intern/object.c
+++ b/source/blender/blenkernel/intern/object.c
@@ -228,23 +228,24 @@ void BKE_object_modifier_hook_reset(Object *ob, HookModifierData *hmd)
void BKE_object_modifier_gpencil_hook_reset(Object *ob, HookGpencilModifierData *hmd)
{
+ if (hmd->object == NULL) {
+ return;
+ }
/* reset functionality */
- if (hmd->object) {
- bPoseChannel *pchan = BKE_pose_channel_find_name(hmd->object->pose, hmd->subtarget);
+ bPoseChannel *pchan = BKE_pose_channel_find_name(hmd->object->pose, hmd->subtarget);
- if (hmd->subtarget[0] && pchan) {
- float imat[4][4], mat[4][4];
+ if (hmd->subtarget[0] && pchan) {
+ float imat[4][4], mat[4][4];
- /* calculate the world-space matrix for the pose-channel target first, then carry on as usual */
- mul_m4_m4m4(mat, hmd->object->obmat, pchan->pose_mat);
+ /* calculate the world-space matrix for the pose-channel target first, then carry on as usual */
+ mul_m4_m4m4(mat, hmd->object->obmat, pchan->pose_mat);
- invert_m4_m4(imat, mat);
- mul_m4_m4m4(hmd->parentinv, imat, ob->obmat);
- }
- else {
- invert_m4_m4(hmd->object->imat, hmd->object->obmat);
- mul_m4_m4m4(hmd->parentinv, hmd->object->imat, ob->obmat);
- }
+ invert_m4_m4(imat, mat);
+ mul_m4_m4m4(hmd->parentinv, imat, ob->obmat);
+ }
+ else {
+ invert_m4_m4(hmd->object->imat, hmd->object->obmat);
+ mul_m4_m4m4(hmd->parentinv, hmd->object->imat, ob->obmat);
}
}
@@ -867,12 +868,12 @@ Object *BKE_object_add_from(
*
* \param data The datablock to assign as ob->data for the new object.
* This is assumed to be of the correct type.
- * \param add_user If true, id_us_plus() will be called on data when
+ * \param do_id_user If true, id_us_plus() will be called on data when
* assigning it to the object.
*/
Object *BKE_object_add_for_data(
Main *bmain, ViewLayer *view_layer,
- int type, const char *name, ID *data, bool add_user)
+ int type, const char *name, ID *data, bool do_id_user)
{
Object *ob;
Base *base;
@@ -881,7 +882,7 @@ Object *BKE_object_add_for_data(
/* same as object_add_common, except we don't create new ob->data */
ob = BKE_object_add_only_object(bmain, type, name);
ob->data = data;
- if (add_user) id_us_plus(data);
+ if (do_id_user) id_us_plus(data);
BKE_view_layer_base_deselect_all(view_layer);
DEG_id_tag_update_ex(bmain, &ob->id, OB_RECALC_OB | OB_RECALC_DATA | OB_RECALC_TIME);
diff --git a/source/blender/depsgraph/intern/builder/deg_builder_relations.cc b/source/blender/depsgraph/intern/builder/deg_builder_relations.cc
index bcdf560e60f..9e3412830c4 100644
--- a/source/blender/depsgraph/intern/builder/deg_builder_relations.cc
+++ b/source/blender/depsgraph/intern/builder/deg_builder_relations.cc
@@ -1909,7 +1909,9 @@ void DepsgraphRelationBuilder::build_object_data_geometry_datablock(ID *obdata)
*/
TimeSourceKey time_key;
ComponentKey geometry_key(obdata, DEG_NODE_TYPE_GEOMETRY);
- add_relation(time_key, geometry_key, "GP Frame Change");
+ add_relation(time_key,
+ geometry_key,
+ "GP Frame Change");
/* Geometry cache also needs to be recalculated when Material
* settings change (e.g. when fill.opacity changes on/off,
@@ -1917,11 +1919,13 @@ void DepsgraphRelationBuilder::build_object_data_geometry_datablock(ID *obdata)
*/
for (int i = 0; i < gpd->totcol; i++) {
Material *ma = gpd->mat[i];
- if ((ma) && (ma->gp_style)) {
+ if ((ma != NULL) && (ma->gp_style != NULL)) {
OperationKey material_key(&ma->id,
- DEG_NODE_TYPE_SHADING,
- DEG_OPCODE_MATERIAL_UPDATE);
- add_relation(material_key, geometry_key, "Material -> GP Data");
+ DEG_NODE_TYPE_SHADING,
+ DEG_OPCODE_MATERIAL_UPDATE);
+ add_relation(material_key,
+ geometry_key,
+ "Material -> GP Data");
}
}
break;
More information about the Bf-blender-cvs
mailing list