Blender Git Loki

Fix RNA un-escaping multiple slashes and strings ending with a slash

Fix for T78823 resolved the issue reported but didn't
properly support multiple back-slashes.

Fix T78823: Slash in custom property name does not work

This fixes inserting key-frames for any collection names
containing a back-slash too (bones, modifiers, sequence strips etc).

Revert "Fix T78823: Slash in custom property name does not work"

This reverts commit cbae82ba960a0baaae6437b176a310f078ce07d8.

This change introduced the following problems:

- We could no longer reliably duplicate or use an existing
custom property names.
- We could no longer assume a bone or ID name
can be used in a custom-property.
- Importers that support custom properties (such as FBX)
could fail with an exception creating custom properties.

Cleanup: use doxy sections for graph_edit.c

creator: remove reference to SDL_AUDIODRIVER from --help text

This was added when Linux's audio support
often needed to be manually configured.

Further 3rd party libraries have their own environment variables & docs
these need not be part of Blender's help message unless they're likely
to be needed to properly run Blender.

Cleanup: use common 'MOD_WELD_MODE_' prefix

Modifier: Add "Connected" mode to the weld modifier

Implement improvement from T73139 for merging along edges.
It is now called "Connected" mode, while the default is called "All".

With the recent performance improvement, the Connected Mode is in some
cases only double the speed than the usual merge all strategy but in
other cases it may be even faster. The bottleneck is somewhere further
down the line of merging geometry.

The motivation for this patch came from T80897, because the merging in
complex solidify is making it very slow.
Now merging can be removed from solidify without greater consequences,
as this is just a quicker and more advanced algorithm to do the same
thing that solidify currently does slowly.

Reviewed by: mano-wii, campbellbarton

Ref D8966

Fix T83559: File Browser uses wrong operation

When opening a temporary File Browser, we have to make sure the file selection
parameters are refreshed. When opening it in a new Window that would always be
the case, if the File Browser uses a maximized window (as set in the
Preferences), it might reuse space-data from a previous use. So we have to
force the refresh.
Also renamed the relevant function to be more clear about what it's doing.

Mistake in 95b3c4c966f9.

Removing ISC license

Blender actually uses the ICS license, not ISC.

Missing README.blender license files + New BSD cases

I also changed New BSD to BSD 3 Clause.

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

Fix T83114: incorrect normals after joining subdivided meshes

This just makes sure that the mesh coming out of the subdivision
surface node has correct normals. Ideally, we would lazily compute
derived data like normals in more cases, but that can be solved later.
Correctness is more important right now.

In order to solve this better, it would be nice if functions like
`BKE_mesh_ensure_normals` would take a `const Mesh *`.
The mesh could be considered to be logically const, because
normals are derived data that is only cached for the current
state of the mesh.

Fix T83337: boundary smooth input in subdivision surface node is inverted

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

Fix T81950: GPencil - Draw - Stoke Placement Stroke Feature Broken

Buffer strokes weren't being the excluded from depth only draw calls
so were being included in depth tests. They are now excluded by
bypassing the creation of the buffer strokes.

Reviewed By: fclem

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

Cycles: Remove "OptiX support is experimental" notice

OptiX support is not in fact experimental anymore, so it is time for that notice to go.
All Cycles features that are currently supported on the GPU do work now when OptiX is selected.

Cycles: Enable baking panel in OptiX and redirect those requests to CUDA for now

This enables support for baking when OptiX is active, but uses CUDA for that behind the scenes, since
the way baking is currently implemented does not work well with OptiX.

Reviewed By: brecht

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

Cleanup: Use enum for file selection type definitions

Makes it more clear that these belong together and allows using the enum type
rather than just `int`.

Cleanup: Correct an own earlier commit to use an existing utility function

Didn't know this function existed, better to use it then to avoid verbosity.

File Browser: Refactor access to the selection parameters struct

* Avoid direct access to `SpaceFile.params`, use a getter instead. This matters
because once the asset-browser changes are in, there will be an alternative
selection parameter object. The getter can return the correct one.
* Rename the function to ensure the parameters. The old name
`ED_fileselect_get_params()` wasn't a mere getter, it would create the
parameters if necessary. Now we have an actual getter, so better be clear.
* In some instances, I replaced the old "get" function with the new mere
getter. So the ensure logic is called less often. However, in these cases we
should be able to assume the selection parameters were created already as
part of the editor creation routine.

The term "active" in the new function names may seem a bit odd in the current
context, but that is a preparation for the Asset Browser merge as well. Like
said, there will be two file selection parameter objects in the space.

Cleanup: Use guarded allocator for data-block names returned from file reading

Direcly using the C library allocator functions is usually avoided in favor of
our guarded allocator. It's more useful when debugging.

Fix unexpected "/" path after normalizing empty directory path

Caused problems in the Asset Browser branch when passing an empty path. The
function shouldn't blindly add a slash but sanity-check the input parameters
first.