Blender Git Commits

GPencil: Display dot icon for active material in Assign menu

Now, a dot icon is displayed in front of the current object active material.

Cleanup: comments for is_path_builtin

Also reduce scope of import.

Fix T59065: Blender default keyset is deletable by user!

Previously, you could delete presets that were part of the blender
default install. Now we check if the preset file resides in the bundled
file paths. If so, prevent deletion of the preset.

Reviewed By: Campbell

Differential Revision: https://developer.blender.org/D4522

Fix T71212: Drivers using scene.frame_current don't refresh on playback

Need to make such drivers explicitly dependent on time.

Cleanup: import functions from bpy.props

Some instances used bpy.props when the convention is to import them.

Fix tool leader-key in image space

- This wasn't working for tool-systems that use the space data.
- Don't activate when the keymap is empty.

OpenSubdiv: Use more generic access to patch coordinates

Saves another unnecessary temporary data copy during evaluation.

Depsgraph: Cleanup, use anonymous nsamespace

Jeeps overall line lengths with function definition shorter.

OpenSubdiv: Initial implementation of batched evaluation

The idea is to give multiple coordinates to evaluator and evaluate them
all at once, avoiding any possible overhead.

OpenSubdiv: Make internal evaluator aware of batched evaluation

Allows to pass multiple patch coordinates for evaluation.

OpenSubdiv: Cleanup, remove old compatibility code

Consider that all builders have face-varying evaluation now.

OpenSubdiv: Cleanup, remove unused evaluator member

BLI_task: Add new generic `BLI_task_parallel_iterator()`.

This new function is part of the 'parallel for loops' functions. It
takes an iterator callback to generate items to be processed, in
addition to the usual 'process' func callback.

This allows to use common code from BLI_task for a wide range of custom
iteratiors, whithout having to re-invent the wheel of the whole tasks &
data chuncks handling.

This supports all settings features from `BLI_task_parallel_range()`,
including dynamic and static (if total number of items is knwon)
scheduling, TLS data and its finalize callback, etc.

One question here is whether we should provide usercode with a spinlock
by default, or enforce it to always handle its own sync mechanism.
I kept it, since imho it will be needed very often, and generating one
is pretty cheap even if unused...

----------

Additionaly, this commit converts (currently unused)
`BLI_task_parallel_listbase()` to use that generic code. This was done
mostly as proof of concept, but performance-wise it shows some
interesting data, roughly:
- Very light processing (that should not be threaded anyway) is several
times slower, which is expected due to more overhead in loop management
code.
- Heavier processing can be up to 10% quicker (probably thanks to the
switch from dynamic to static scheduling, which reduces a lot locking
to fill-in the per-tasks chunks of data). Similar speed-up in
non-threaded case comes as a surprise though, not sure what can
explain that.

While this conversion is not really needed, imho we should keep it
(instead of existing code for that function), it's easier to have
complex handling logic in as few places as possible, for maintaining and
for improving it.

Note: That work was initially done to allow for D5372 to be possible... Unfortunately that one proved to be not better than orig code on performances point of view.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D5371

OpenSubdiv: Use smaller buffer sizes and strides

We currently don't support interleaved varying data, so can make
some buffer smaller.

OpenSubdiv: Cleanup, remove unused static methods

OpenSubdiv: Cleanup, remove unused PatchBuffer class

OpenSubdiv: Cleanup, remove unused class

OpenSubdiv: Avoid unnecessary memory copies

Make evaluator itself to write directly to the output buffer.

Brings unmeasurable speedup and makes it possible to simplify
some buffer class logic.

UI: Correct prvicons

Correct source, output, and formatting of prvicons - larger icons used for file browser.

Not Reviewed

UI: Add new toolbar icons for polyline & eyedropper

These are meant for use in Grease Pencil Draw mode.