diff options
author | Hans Goudey <h.goudey@me.com> | 2022-11-11 18:07:54 +0300 |
---|---|---|
committer | Hans Goudey <h.goudey@me.com> | 2022-11-11 18:07:54 +0300 |
commit | ff7645c5edd211c5317b5fd9e6d0320c20667a21 (patch) | |
tree | 4bd40797a8e09a09035381ffde43055f6ff75453 /source/blender/editors/interface/interface_region_color_picker.cc | |
parent | e508de041712cc31588cbc8d63f81970eedf3be0 (diff) |
Fix T102404: Behavior change in CustomData API
Previously the `CustomData_add_layer` function always returned
the existing layer data when used for types that can only have one
layer. This made it work like an "ensure layer exists" function for those
types. That was used in various places to make code more concise.
0a7308a0f149 changed that to always "recreate" the layer even
when it existed. Maybe this is more logical for an "add layer" function,
but that's not clear, and it breaks a bunch of existing code that relied
on the current behavior. Rather than spending a bunch of time going
through uses of the CustomData API, this patch resets the behavior
to what it was before, but adds an assert and a comment to help
avoid memory leaks and other issues. We should focus on moving
to the attribute API instead.
Differential Revision: https://developer.blender.org/D16458
Diffstat (limited to 'source/blender/editors/interface/interface_region_color_picker.cc')
0 files changed, 0 insertions, 0 deletions