Blender Git Loki

Fix (unreported) critical mistake in Collection liblink code in readfile.c.

Prelimenary step to fix T77460.

Not sure how or when that thing was done, but since that call walks
around collections relationships, it's an utterly critical violation of
liblinking principles (code here should never, ever 'get outside' of its
own ID scope).

This was wroking so far only because code called through this function
(`BKE_collection_parent_relations_rebuild`) was only following parents
pointers (in `BKE_collection_find_cycle()`), which would be either valid
or non-existent.

But next commit is going to change that to also check collection's
objects instancing of other collections.

Fix T77853: Error reloading linked library.

Issue related to how ID refcounting was changed when loading blendfile
data...

Fix T77774: New undo code broken by 'make local' behavior.

This is actually a nice issue due to too much optimization...
* Making an ID local just reuse the linked one whenever possible, instead of
actually making a copy of it.
* Therefore, the collection containing that ID is seen as unchanged, since
the pointer itself remained the same.
* But on undo step, there is no way to reuse that local object, which then
gets deleted, and linked one gets re-created - at a different address.
* Collection, however, since unchanged, is not updated at all and thus keeps
reference to the to-be-deleted local object, instead of the linked one.
* Issue gets even worse with viewlayers, this leads to the crash.

To address this, this patch adds a 'virtual' update flags that does nothing
in update case, but will ensure that the affected IDs using the one made local
are properly detected as changed across the relevant undo step.

Note that the recalc flags were chosen mostly for a logical reason, and also
because they are already properly dealt with and cleared by undo code,
so this looks like the optimal solution.

Reviewed By: brecht

Maniphest Tasks: T77774

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

Fix missing hinting information in default font

Updates the font file to bring back hinting data, which got lost somehow
in dc3f073d1c52.

Second and (hopefully) final fix for T75898.

Fix missing hinting information in monospace font

Updates the monospace font file to bring back hinting data, which got
lost somehow in dc3f073d1c52.

Partial fix for T75898.

Fix for T77095: work around render artifacts with AMD Radeon RX 4xx and 5xx

Bumped blender version to 2.83.1

Bumped file subversion to 19

Fix T77657: NVIDIA Quadro FX4800 crash on startup

Limit support for `GLEW_ARB_base_instance` to OpenGL 4.0 and higher. NVIDIA Quadro FX 4800
(TeraScale) report that they support GLEW_ARB_base_instance, but the driver does not support
`GLEW_ARB_draw_indirect` as it has an OpenGL3 context what also matches the minimum needed
requirements.

We use `GLEW_ARB_draw_indirect` as a target for `glMapBuffer(Range)` what is part of the
OpenGL 4 API. So better disable it when we don't have an OpenGL4 context.

Note: fix should be ported to Blender 2.83 LTS

Reviewed By: Cl�ment Foucault

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

GPencil: Improve viewlayer masking check

Check if the mask layer is used in a layer that is going to be used in the current viewlayer.

Related to T77667

Fix T77148: Crash changing multiple values for sequencer strips

This was caused by assuming all strips were the same type.

Fix T77047: Dyntopo Sample detail size on hidden mesh causes crash

The `Toolbar` and `Sidebar` hide the corresponding panel
`VIEW3D_PT_sculpt_dyntopo` by polling for context.sculpt_object and
context.tool_settings.sculpt. In the Active Tool in the Properties
Editor this poll does not return False though, thus the
sample_detail_size is possible from there.

Second security check (the operator poll `SCULPT_mode_poll`) checks the
active object -- that is still valid even if hidden, so we are allowed
to execute the operator. However the active object becomes NULL once the
area is switched in `sample_detail()` -- see `CTX_wm_area_set`), leading
to the crash.

Dont think there is a quick and easy way to do this in the poll from the
Properties Editor, so just check for a valid active abject in the
operator and return OPERATOR_CANCELLED if we dont have it.

Maniphest Tasks: T77047

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

Fix T74101: File Browser in macOS fullscreen crashes or makes windows unusable

When closing the File Browser window after making it fullscreen, Blender would
either crash or all windows would disappear, with no obvious way to bring them
back.

The "fix" is to not allow fullscreen for File Browsers (or any future "dialog"
windows), but only maximizing. From what I can tell that's how secondary
windows are supposed to work on macOS. What we previously did seemed like
something macOS doesn't handle cleanly, and I didn't find a simple way to do so
on our side.

Fix T77504: Operator search gives wrong results

Regression in e8ab0137f8766

Fix T76894: Disable clipping region selection in material/rendered mode

In this case the draw engine isn't responsible for the selection, but
the editor is.

Reviewed By: Cl�ment Foucault

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

Fix T72936: Incorrect gizmo orientation with inherit rotation disabled

Fix menu operator/search clipping the last character

Noted in T77504

Fix T77603: OSL parser fails when script ends with comment without newline

Reviewers: brecht

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

Fix crash running "Edit Voxel Size" operator outside of a main 3D View region

Accessed `RegionView3D` data from context, which of course would only be set if
actually executed from a main 3D View region.

Buildbot: Cleanup, remove unused script and change naming

Follow upstream convention.