Blender Git Loki

Fix/workaround 'convert object' messing up linked data.

'Convert To...' Object operation has very weird effect of actually
working at obdata level, not object level, which means *all* objects
(even unselected/hidden/in other scenes/...) using same obdata will be
converted to new selected type.

IMHO this is very bad behavior, but... not a bug really, so do not
change this for now.

But at least, do not do that when working on some linked data, else it
leaves Blend file in invalid (incoherent) state until next reload.

So workaround for now is to enforce the 'Keep Original' option when some
linked object/obdata is affected by the operation.

Also fixed somewhat broken usercount handling in Curve->Mesh part.

Eevee: Use PROP_NONE for probe grid resolution

It makes no sense to use PROP_PIXEL as unit here.

automated some UI selections

now smoke domains set volume cache automatically, liquid domains set surface cache automatically

Layers: Scene copy should copy selection as well.

We need this for Depsgraph, otherwise CoW scene will have different selection properties.

New icon for Outliner and changes in Stroke Paint icon

The outliner icon for grease pencil must be gray to keep UI consistency.

Changed the stroke paint icon to make more visible.

Icon design credits go to Matias Mendiola

Cleanup: make Group Unlink outliner action use `BKE_libblock_delete() too.

Same as in rB2bd51474a44...

Merge branch '28' into custom-manipulators

Cleanup: remove useless call to `BKE_screen_free()` in screen deletion.

`BKE_libblock_free()` will call it itself.

Fix dangerous code when deleting Scene.

That one was probably not an actual issue, except maybe in some corner
cases (like deleting a linked scene also used by some other linked scene).

Again, better not try to do smart & complex freeing logic outside of
BKE_library area, let's keep spaghetti nitghmare in a single place!

Updates to manipulator API

While this is work-in-progress from custom-manipulators branch
its stable so adding into 2.8 so we don't get too much out of sync.

- ManipulatorGroupType's are moved out of the manipulator-map and are now
global (like operators, panels etc) and added into spaces as needed.
Without this all operators that might ever use a manipulator in the 3D
view would be polling the viewport.
- Add optional get/set callbacks for non-RNA properties
Needed so re-usable manipulators can control values that
don't correspond to a single properly or need conversion.
- Fix divide by zero bug in arrow manipulator (when moving zero pixels).

Cleanup: make Group Unlink operator use BKE_libblock_delete(), since that's what it is doing.

Previous code (same as what `BKE_libblock_free_us()` is doing when
usercount reach 0) was probably OK in that specific case, but still not
good idea, and potentially risky.

Fix collada importer doing own handling of usercount/freeing.

Better use generic `BKE_libblock_free_us()`.

Fix potentially dnagerous code in doversionning of brush.

Even though in that specific it was probably safe-ish, there is no
guarantee at this point Brush we want to remove are not used somewhere,
better take the slightly slower, much safer `BKE_libblock_delete()` path here.

Fix serious bug in 'curve-to-mesh' conversion code.

Eeeeeek!^2 Calling unconditionnaly ID freeing `BKE_libblock_free()` on a
datablock (ob->data, i.e. Curve) that may be used elsewhere...
Veryveryvery bad!

Fix potential memory leak in Sequencer sound strip creation code.

Merge branch '28' into custom-manipulators

Dial manipulator didn't have a modal callback

Until 3d-viewport manipulators change functionality we need to it
working how it already is.

So add modal operator which is called from draw when there are no
properties applied to the manipulator.

This will be needed to have a general rotation manipulator.

Fix T51463: Eevee motion blur not working (with AMD)

GLSL needs FragColor to be initialized. The default vec4 value seems to be
implementation dependent. Or it's a bug on Mesa/AMD.

Fix error with object scale and stroke thickness

With new calculation of previous commit for stroke thickness the scale of the object was not used and need to be used.

Now the scale is included and an average value is used because the axis change is not directly related to the stroke thickness.

Fix strict cflags compilation after recent const changes