From bc3e38ca3a4228b7191e9fec39288c9f37ec53b1 Mon Sep 17 00:00:00 2001 From: Bastien Montagne Date: Wed, 6 Jan 2021 12:25:35 +0100 Subject: Cleanup/refactor: UndoType: Clarify `use_context` variable. Rename it to mark it is only for `encode` callbacks, fix `encode` callback of text undo to early fail in case it gets a NULL context, add an assert to `BKE_undosys_step_push_with_type` that context is not NULL when undotype requires a valid one. Note that in practice this should not change anything, currently it seems that we always get a valid context in `BKE_undosys_step_push_with_type`? --- source/blender/editors/sculpt_paint/sculpt_undo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'source/blender/editors/sculpt_paint/sculpt_undo.c') diff --git a/source/blender/editors/sculpt_paint/sculpt_undo.c b/source/blender/editors/sculpt_paint/sculpt_undo.c index 5bbe18935cc..11a279b5a5a 100644 --- a/source/blender/editors/sculpt_paint/sculpt_undo.c +++ b/source/blender/editors/sculpt_paint/sculpt_undo.c @@ -1576,7 +1576,7 @@ void ED_sculpt_undosys_type(UndoType *ut) ut->step_decode = sculpt_undosys_step_decode; ut->step_free = sculpt_undosys_step_free; - ut->use_context = true; + ut->use_context_for_encode = true; ut->step_size = sizeof(SculptUndoStep); } -- cgit v1.2.3