Welcome to mirror list, hosted at ThFree Co, Russian Federation.

git.blender.org/blender.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHans Goudey <h.goudey@me.com>2022-03-30 03:56:38 +0300
committerHans Goudey <h.goudey@me.com>2022-03-30 03:56:38 +0300
commit87e9451d660e8288d70aa781530b69f4005bf0ea (patch)
treeeb0ad86ddc01db5b5d5a184788a30b89de2ead54
parent72d25fa41d8c5753e4cdc1293d407e16c1431119 (diff)
Fix: Remove special case from curve segment size function
The idea that curves with two points cannot be cyclic came from some existing code, but there's not fundamental reason for it, so remove the check in this function. The case can be handled elsewhere if necessary.
-rw-r--r--source/blender/blenkernel/BKE_curves.hh7
1 files changed, 3 insertions, 4 deletions
diff --git a/source/blender/blenkernel/BKE_curves.hh b/source/blender/blenkernel/BKE_curves.hh
index 05a20917552..13b3d280bc7 100644
--- a/source/blender/blenkernel/BKE_curves.hh
+++ b/source/blender/blenkernel/BKE_curves.hh
@@ -351,13 +351,12 @@ class CurvesGeometry : public ::CurvesGeometry {
namespace curves {
/**
- * The number of segments between control points, accounting for the last segment of cyclic curves,
- * and the fact that curves with two points cannot be cyclic. The logic is simple, but this
- * function should be used to make intentions clearer.
+ * The number of segments between control points, accounting for the last segment of cyclic
+ * curves. The logic is simple, but this function should be used to make intentions clearer.
*/
inline int curve_segment_size(const int points_num, const bool cyclic)
{
- return (cyclic && points_num > 2) ? points_num : points_num - 1;
+ return cyclic ? points_num : points_num - 1;
}
namespace bezier {