Blender Git Loki

GHOST: Enable debug context on offscreen context too

This was a long standing TODO. This was also preventing debug callbacks
form other context than the main window.

Cleanup: GPU: Remove GPU_draw_primitive and default_vao_

These are not used anymore and can be replicated using the GPUBatch API.

Cleanup: Use GPUBatch for icon and area drawing

This is in order to remove GPU_draw_primitive to streamline the drawing
abstraction.

Cleanup: Comment formatting in unit.c

New Boolean: Always expose solver propery, add warning

After discussion with @howardt, it seems the solver property should
always be exposed, even in lite builds. This commit removes the
ifdefs for that property and adds a warning if the "Exact" solver is
used when Blender is compiled without GMP.

These changes apply to the boolean modifier as well.

cleanup: Fix GMP spacing in cmake configurations

It now follows the spacing around it.

Fix unused parameter warning in `snap_increment_apply_ex()`

No functional changes.

Remove patch for HDF5 library

Delete `hdf5.diff`, because it's no longer used. Since Blender 2.90 the
optional support for HDF5 has been dropped, but this file accidentally
wasn't deleted.

See 0102b9d47edf and 0c3843622726.

No functional changes.

Cleanup/Refactor: Split the snap to increments code

Now we have a better distinction of what is snap to grid and what is
snap to increments.

The code also allows the implementation of mixed snap for these modes.

Cleanup: GPU: Remove unused attr_binding and primitive code

GPUImmediate: GL backend isolation

This is part of the Vulkan backend task T68990.

This is mostly a cleanup, however, there is a small change:
We don't use a special Vertex Array binding function for Immediate
anymore and just reuse the one for batches.
This might create a bit more state changes but this could be fixed
easily if it causes perf regression.

# Conflicts:
# source/blender/gpu/intern/gpu_context.cc

GLContext: Fix framebuffer deletion to GPUContext

This has more meaning as it's the base class who holds the pointer.

# Conflicts:
# source/blender/gpu/opengl/gl_context.cc

Cleanup: GPU_immediate: Change assert to BLI_assert

Fix T79973: Re-ordering face maps messes up the names of other face maps

Use a remap function instead a swap.

Reviewed By: campbellbarton

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

Cleanup: reduce variable scope

Cleanup: reduce variable scope and use LISTBASE_FOREACH

Fix T80126: Alembic Import dialogue has overwrite protection UI

Use `FILE_OPENFILE` when importing, rather than `FILE_SAVE`.

Reviewed By: sybren

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

Linux: 2.90 release information in appdata

Cleanup: Resolve unused variable warning in lite build

Clang Tidy: Fix readability-delete-null-pointer warnings