Age | Commit message (Collapse) | Author |
|
# Conflicts:
# source/blender/python/intern/CMakeLists.txt
# source/blender/python/intern/bpy_interface.c
|
|
|
|
|
|
It was impossible for drivers to use shape key properties, modifiers
generate a new mesh. After mesh evaluation the shape keys are no longer
necessary, and because of this the `key` pointer was not copied. As
drivers work on evaluated data, however, they do need this `key`
pointer.
This commit makes the `key` pointer available in evaluated meshes, but
this is somewhat dangerous. There was an explicit reason why the key on
result was kept at null pointer: to have the evaluated mesh in a
consistent state. Assigning this pointer makes it potentially
inconsistent, as the evaluated mesh and the original shape key may have
different topologies.
Reviewed By: sergey
Differential Revision: https://developer.blender.org/D7785
|
|
|
|
# Conflicts:
# source/blender/blenkernel/intern/ipo.c
# source/blender/blenkernel/intern/mask_rasterize.c
# source/blender/blenkernel/intern/nla.c
# source/blender/blenkernel/intern/particle.c
# source/blender/blenkernel/intern/softbody.c
# source/blender/blenloader/intern/readfile.c
# source/blender/bmesh/tools/bmesh_bevel.c
# source/blender/editors/interface/interface.c
# source/blender/editors/interface/interface_anim.c
# source/blender/modifiers/intern/MOD_datatransfer.c
# source/blender/python/intern/bpy_rna.c
# source/blender/python/intern/bpy_rna_anim.c
|
|
Check the simpler case first and return early.
|
|
|
|
|
|
Exit early if the bevel width is zero instead of putting
the main function in an if statement.
|
|
This commit generally moves variable declarations to the smallest scope
the variables are used in. This makes the code more readable by
making it clearer when variables are used and by removing the block
of variable declarations at the top of each function.
|
|
The preview of points was only done when the edge is wire.
Now the preview of points is done when the edge is not connected to any
quad.
Also to avoid edge slide in this case, all new vertices created in this
specific case are not selected.
Differential Revision: https://developer.blender.org/D7457
|
|
|
|
Move uv_poly_center to BM_face_uv_calc_center_median as
it was only defined in uvedit_intern.h
|
|
|
|
And make them part of the blender_test runner. The one exception is blenlib
performance tests, which we don't want to run by default. They remain in their
own executable.
Differential Revision: https://developer.blender.org/D8498
|
|
Also order sizeof(..) first to promote other values to size_t.
|
|
|
|
|
|
This addresses warnings from Clang-Tidy's `readability-else-after-return`
rule. This should be the final commit of the series of commits that
addresses this particular rule.
No functional changes.
|
|
# Conflicts:
# source/blender/blenkernel/BKE_DerivedMesh.h
# source/blender/blenkernel/intern/DerivedMesh.c
# source/blender/blenkernel/intern/anim_data.c
# source/blender/blenlib/PIL_time_utildefines.h
# source/blender/editors/space_info/info_intern.h
# source/blender/windowmanager/intern/wm_event_system.c
# source/blender/windowmanager/intern/wm_files.c
|
|
|
|
This replaces header include guards with `#pragma once`.
A couple of include guards are not removed yet (e.g. `__RNA_TYPES_H__`),
because they are used in other places.
This patch has been generated by P1561 followed by `make format`.
Differential Revision: https://developer.blender.org/D8466
|
|
# Conflicts:
# source/blender/io/alembic/exporter/abc_export_capi.cc
|
|
Previously a for loop with two iterations was used to calculate the
3D segment locations for the input number and the higher power of 2.
Splitting off the inside of the for loop to a separate function makes
the code more readable.
This commit also includes a simple timer for bevel, enabled with a
define. Interestingly, the cleanup in this commit happended to give
a 3% speedup on a Ryzen 3700x for a bevel calculation with 64
segments.
|
|
|
|
- separate logs into 2 macros for easy of turning on/off
|
|
- now BM_CHECK_ELEMENT uses clog
|
|
|
|
|
|
|
|
The bevel code initialized the CurveProfile with the "higher power of 2"
segments after the normal number of segments, leaving the widget in an
incorrect state after the calculation. A simple fix is to re-order the
initializations, doing the input number second.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
# Conflicts:
# source/blender/blenkernel/BKE_global.h
|
|
The abbreviation 'init' is brief, unambiguous and already used
in thousands of places, also initialize is often accidentally
written with British spelling.
|
|
|
|
|
|
|
|
|
|
New code to calculate UV islands made an incorrect assumption about
maximum stack size.
|
|
New code to calculate UV islands made an incorrect assumption about
the maximum size of a stack.
|
|
This matches the change that was done to the bevel modifier so that the
interface for the modifier, the active tool, and the operator are consistent.
This commit extends the refactor to the bmesh implementation too, so
that the parameters in the implementation don't stray too far from what
is exposed.
Tests are adjusted and still pass.
|
|
|
|
Internally UV selection considered close UV's to be connected.
While this could be convenient in some cases,
it complicates logic for more advanced selection operations that
need to check when UV's should be considered part of the same vertex
since simple threshold checks would give different results depending
on the order of UV's tested.
Users must now run "Merge by Distance" instead of relying
on this selection threshold.
|
|
A bug introduced when I made the choice of interpolation face cleverer.
|