Blender Git Loki

Fix (unreported) bad copying of Ocean modifier.

Was needlessly complicated code, forgot to copy a value (foam_fade), and
was utterly leaking memory!

PyAPI: Fix warning about indent

Add Mesh and Key support to new copying code.

Make object copying fully compatible/aware of new copy flag system.

(continuation of previous WIP commit, sorry about that one :/ ).

This commits changes quite a few things, distributing new copying flags
into sub-data copying code (mostly concerns ID refcounting or not).

It also removes ID refcounting handling from Modifiers' copy callback
(this was ugly from the start, proved to be problematic in current
master, and generally bad practice). This is now done by calling code.

Also, it brings back ID refcounting handling to main BKE_library's copy
code, which means in generic ID copying lower-level IDType-specific copy
code should not use it at all. Support at lower-level remains needed
though, unfortunately, to cope with partial copying tools etc.

WIP

Cleanup: doxygen comments

Also remove duplicate & mismatching comments from grease-pencil header.
Keep comments close to implementation to avoid getting out of sync.

Alembic tests: make failures a bit easier to diagnose.

Fix T51840: UI redraw in node editor header missing on pointcache bake

Missing a notifier handler in the node editor

Guarded allocator: Fix type in macro definition

The crash did not happen yet because we always had proper vmemh defined in
the parent scope.

Patch by Ivan Ivanov (aka obiwanus), thanks!

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

Fix compiler warnings from own recent rB0d5c7e5e36b9.

Fix (unreported) Dynamic Paint modifier not increasing ID usercount in copy function.

*Sigh* One more example of why we should keep ID management handling in
as few places as possible! It's impossible to keep more than a few
places in sync regarding which ID pointer is refcounted etc.

Fix unreported: Copy-pasting nodes crashes when they have an undefined type

Use for/else instead of setting 'ok' variable.

This is more efficient, and this use case is exactly what the else clause
is for.

Fix (unreported) bad copying code of Surface Deform modifier.

Fix T51762: Unit test script_alembic_import is failing.

Implemented workaround for use with the legacy depsgraph.

Docs: Fix file:line links in generated API docs

Fix T51774: Children particles hair interpolation not correct with textures or dp.

Children where always getting at least one segment of fixed length...

Now fully hidden ones (zero length) get no segment at all.

Note that even very short ones keep getting one 'unit' length segment - would
rather avoid changing that at this point, given how complex children
particles 'length' can get with all kind of modifiers... Think we can
live with that for now anyway.

Add Cone: tip soft-min should be zero

Default value should be included in range.

Add same new ID freeing API as copying one.

Idea is the same, looks like it will be a tad simpler than with copy
though, since we should not need to change each ID type freeing func, as
ID usercount handling is done in main BKE_library code (would like to do
that for copy as well, but it's not that simple).

Fleshing a bit new copy logic (using Object datablock as Guinea pig).