From 2c3ef36a0b7fe301e32137a17635ceefaa1de791 Mon Sep 17 00:00:00 2001 From: Antonio Vazquez Date: Sun, 7 Jun 2020 13:06:03 +0200 Subject: Fix T77520: GPencil viewlayer filter produce crash with masking layers If a layer is used for masking, it cannot be filtered by viewlayer because the masked layer needs to have the mask layers in the draw pipeline. This check is only done in final render. --- source/blender/blenkernel/intern/gpencil.c | 25 ++++++++++++++++++++++++- 1 file changed, 24 insertions(+), 1 deletion(-) (limited to 'source/blender/blenkernel') diff --git a/source/blender/blenkernel/intern/gpencil.c b/source/blender/blenkernel/intern/gpencil.c index 99ee2a0fc9e..1f6c0e96519 100644 --- a/source/blender/blenkernel/intern/gpencil.c +++ b/source/blender/blenkernel/intern/gpencil.c @@ -1832,6 +1832,25 @@ bool BKE_gpencil_from_image(SpaceImage *sima, bGPDframe *gpf, const float size, return done; } +/** + * Helper to check if a layers is used as mask + * \param gpd Grease pencil datablock + * \param gpl_mask Actual Layer + * \return True if the layer is a mask + */ +static bool gpencil_is_layer_mask(bGPdata *gpd, bGPDlayer *gpl_mask) +{ + LISTBASE_FOREACH (bGPDlayer *, gpl, &gpd->layers) { + LISTBASE_FOREACH (bGPDlayer_Mask *, mask, &gpl->mask_layers) { + if (STREQ(gpl_mask->info, mask->name)) { + return true; + } + } + } + + return false; +} + /* -------------------------------------------------------------------- */ /** \name Iterators * @@ -1873,7 +1892,11 @@ void BKE_gpencil_visible_stroke_iter(ViewLayer *view_layer, * This is used only in final render and never in Viewport. */ if ((view_layer != NULL) && (gpl->viewlayername[0] != '\0') && (!STREQ(view_layer->name, gpl->viewlayername))) { - continue; + /* If the layer is used as mask, cannot be filtered or the masking system + * will crash because needs the mask layer in the draw pipeline. */ + if (!gpencil_is_layer_mask(gpd, gpl)) { + continue; + } } if (is_multiedit) { -- cgit v1.2.3