diff options
| author | Brecht Van Lommel <brecht@blender.org> | 2020-10-01 18:39:24 +0300 |
|---|---|---|
| committer | Brecht Van Lommel <brecht@blender.org> | 2020-10-01 18:43:41 +0300 |
| commit | 5b8503425a491f94f526d1f65cefea1599cafd92 (patch) | |
| tree | a429b4fa652175e727e7cccd2eee1e164ceabf88 /source/blender/blenkernel/intern | |
| parent | 958fc6944f75f51b9fe5b47f29ff82156b3c6286 (diff) | |
Fix subdivision surface wrong for non-limit surface and subdivision level 1
Remove the workaround for T59395 that increases the minimal refinement to 2
to work around an OpenSubdiv bug. This bug appears to be fixed in the latest
OpenSubdiv version we are using.
Problem found by Piotr Ostrowski.
Ref D9076
Diffstat (limited to 'source/blender/blenkernel/intern')
| -rw-r--r-- | source/blender/blenkernel/intern/subdiv.c | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/source/blender/blenkernel/intern/subdiv.c b/source/blender/blenkernel/intern/subdiv.c index f1cde39f657..e6b51c586c3 100644 --- a/source/blender/blenkernel/intern/subdiv.c +++ b/source/blender/blenkernel/intern/subdiv.c @@ -87,27 +87,6 @@ eSubdivVtxBoundaryInterpolation BKE_subdiv_vtx_boundary_interpolation_from_subsu /* ================================ SETTINGS ================================ */ -static bool check_mesh_has_non_quad(const Mesh *mesh) -{ - for (int poly_index = 0; poly_index < mesh->totpoly; poly_index++) { - const MPoly *poly = &mesh->mpoly[poly_index]; - if (poly->totloop != 4) { - return true; - } - } - return false; -} - -void BKE_subdiv_settings_validate_for_mesh(SubdivSettings *settings, const Mesh *mesh) -{ - if (settings->level != 1) { - return; - } - if (check_mesh_has_non_quad(mesh)) { - settings->level = 2; - } -} - bool BKE_subdiv_settings_equal(const SubdivSettings *settings_a, const SubdivSettings *settings_b) { return (settings_a->is_simple == settings_b->is_simple && |