Blender Git Loki
Git Commits -> Revision a07089d
Revision a07089d by Hans Goudey (master) November 24, 2021, 15:39 (GMT) |
Fix T92120 (partially): No bone custom shape with curve object meshes This part of the drawing code assumes that the bone custom object has only one evaluated geometry component, and it also uses the object type to check which data to draw, with the functions like `DRW_cache_object_surface_get` that just take an object input. Those functions usually work on evaluated objects, which use the instancing system to access a temporary object with `object.data` replaced for data types that don't match the original object. That assumption used to work, but now curve, point cloud, or volume objects can have an evaluated mesh which is not accessed with the same object for render engine drawing. The "correct" solution for the way this code is structured would be to loop through all of the geometry components and try to get GPU batches from every one of them. However, that significantly increases complexity in an area that should probably be refactored anyway. This patch treats the mesh as a special case, and only draws the evaluated mesh. The **best** solution in my opinion might be refactoring this area to use the instancing system with some sort of viewport-only flag so the custom shape instances aren't added in the render. The solution is "partial" because the "Wireframe" option only works for meshes from mesh objects, even after this fix, and because other data besides meshes is not displayed at all. Differential Revision: https://developer.blender.org/D13038 |
Commit Details:
Full Hash: a07089dcb10d8f0265220bf5abe07dca03097fe1
Parent Commit: 56b068a
Lines Changed: +17, -4
1 Modified Path:
/source/blender/draw/engines/overlay/overlay_armature.c (+17, -4) (Diff)