The paint cursor now uses a single call to
SCULPT_cursor_geometry_info_update and the Sculpt API to update both the
radius and the SculptSession data, so random access needs to be
initialized for dyntopo before using API functions.
Details
Details
Diff Detail
Diff Detail
- Repository
- rB Blender
- Branch
- T79597 (branched from master)
- Build Status
Buildable 9392 Build 9392: arc lint + arc unit
Event Timeline
Comment Actions
The code seems fine, but the description is confusing.
How does the single call nature affects on the need to initialize random access? It's more like "ensure lazily initializable data is initialized before access". If it's a single call, two calls, or whatever is orthogonal.
P.S. On a separate note the SCULPT_vertex_random_access_init should be called SCULPT_vertex_random_access_ensure
Comment Actions
Dynamic topology being completely broken is a high priority bug to fix, can we please fix this more quickly?
Comment Actions
The rename did happen in master already, so update the patch to the latest master and it will be fine to commit.