Blender Git Loki

GPencil: Fix unreported crash when style is NULL

Cleanup: correct argument order to callback

Currently the callback isn't used,
found when testing a new enum callback.

Merge branch 'blender-v2.91-release' into master

Fix T82553: Outliner F2 renaming issue when item is out of view

- scrolling would be restricted (usually, if the object to be renamed is
in view, this prevents scrolling away without finishing the rename
operation)
- renaming by typing and confirming with Enter was not possible (you
would have to escape, scroll to the object and use F2 again)
- other shortcuts like A and H are still active instead of being handled
as text input

Avoid all these issue by forcing the item into view using
outliner_show_active / outliner_scroll_view.

Maniphest Tasks: T82553

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

Cleanup: Use `NDEBUG` define, `DEBUG` one is not reliable.

Fix 'outliner_scroll_view()' not reaching wanted element

Scrolling to an item after opening relevant parents can go wrong if said
parent e.g. the last in the list [as in: then the Outliner does not
scroll down all the way]
It stems from the fact that 'region->v2d.tot.ymin' is not up-to-date in
outliner_scroll_view after outliner_show_active opens up parents, 'tot'
will only update on a redraw.

Now calculate the trees height on the fly using
'outliner_tree_dimensions()'.

ref D9521
ref T82553

Maniphest Tasks: T82553

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

Exporter: Cleanup: minor rename

fix regression regarding missing BVH update

Merge branch 'master' into cycles_procedural_api

Blender 2.83.9 release

Fix T82602: checking image header reads past buffer bounds

Use the size argument to ensure checking the header doesn't read
past the buffer bounds when reading corrupt/truncated headers
from image files.

Cleanup: pass header size to 'is_a' callbacks

No functional changes, prepare for fixing out-of-bounds access
when reading headers.

Cleanup: use 'filepath' instead of 'name' for ImBuf utilities

Cleanup: use bool for imbuf save callbacks

Cleanup: avoid boolean literals for functions that return int

Merge remote-tracking branch 'origin/master' into lanpr-under-gp

Cleanup: use define for targa header size

Cleanup: remove redundant NULL checks in ImFileType.is_a callback

Most of these callbacks don't do a NULL check,
so there is no need to do this for bmp/png.

Also correct radiance_hdr comments.

Merge branch 'blender-v2.91-release'

ImBuf: replace incorrect strstr use with memcmp

Besides being incorrect as only the first two bytes should be tested,
searching binary data using `strstr` can easily read past buffer bounds.