[Bf-blender-cvs] [ebbeddd2f46] blender2.8: Subsurf: Use edge sharpness directly from converter
Sergey Sharybin
noreply at git.blender.org
Wed Aug 1 18:43:29 CEST 2018
Commit: ebbeddd2f4617505bb2e13b6d773a3da6bd327c4
Author: Sergey Sharybin
Date: Wed Aug 1 15:13:24 2018 +0200
Branches: blender2.8
https://developer.blender.org/rBebbeddd2f4617505bb2e13b6d773a3da6bd327c4
Subsurf: Use edge sharpness directly from converter
Seems it's behaving correct now, surely more tests needed, but
this is required for now to move forward.
===================================================================
M intern/opensubdiv/internal/opensubdiv_converter_factory.cc
M intern/opensubdiv/internal/opensubdiv_converter_internal.cc
M intern/opensubdiv/internal/opensubdiv_converter_internal.h
M intern/opensubdiv/internal/opensubdiv_topology_refiner.cc
===================================================================
diff --git a/intern/opensubdiv/internal/opensubdiv_converter_factory.cc b/intern/opensubdiv/internal/opensubdiv_converter_factory.cc
index 06707cbf418..02b6e93a1ae 100644
--- a/intern/opensubdiv/internal/opensubdiv_converter_factory.cc
+++ b/intern/opensubdiv/internal/opensubdiv_converter_factory.cc
@@ -336,7 +336,7 @@ inline bool TopologyRefinerFactory<TopologyRefinerData>::assignComponentTags(
const int num_edges = converter->getNumEdges(converter);
for (int edge_index = 0; edge_index < num_edges; ++edge_index) {
const float sharpness =
- opensubdiv_capi::getCompatibleEdgeSharpness(converter, edge_index);
+ converter->getEdgeSharpness(converter, edge_index);
setBaseEdgeSharpness(refiner, edge_index, sharpness);
}
// OpenSubdiv expects non-manifold vertices to be sharp but at the time it
diff --git a/intern/opensubdiv/internal/opensubdiv_converter_internal.cc b/intern/opensubdiv/internal/opensubdiv_converter_internal.cc
index 32815dc34dc..2f7a7109ea1 100644
--- a/intern/opensubdiv/internal/opensubdiv_converter_internal.cc
+++ b/intern/opensubdiv/internal/opensubdiv_converter_internal.cc
@@ -84,13 +84,4 @@ getCAPIFVarLinearInterpolationFromOSD(
return OSD_FVAR_LINEAR_INTERPOLATION_NONE;
}
-float getCompatibleEdgeSharpness(const OpenSubdiv_Converter* converter,
- int edge_index) {
- if (converter->getNumEdgeFaces(converter, edge_index) == 2) {
- return converter->getEdgeSharpness(converter, edge_index);
- } else {
- return OpenSubdiv::Sdc::Crease::SHARPNESS_INFINITE;
- }
-}
-
} // namespace opensubdiv_capi
diff --git a/intern/opensubdiv/internal/opensubdiv_converter_internal.h b/intern/opensubdiv/internal/opensubdiv_converter_internal.h
index c47cdd1004d..411514c45dc 100644
--- a/intern/opensubdiv/internal/opensubdiv_converter_internal.h
+++ b/intern/opensubdiv/internal/opensubdiv_converter_internal.h
@@ -46,10 +46,6 @@ OpenSubdiv_FVarLinearInterpolation
getCAPIFVarLinearInterpolationFromOSD(
OpenSubdiv::Sdc::Options::FVarLinearInterpolation linear_interpolation);
-// Get edge sharpness in a way which makes OpenSubdiv happy.
-float getCompatibleEdgeSharpness(const OpenSubdiv_Converter* converter,
- int edge_index);
-
} // namespace opensubdiv_capi
#endif // OPENSUBDIV_CONVERTER_INTERNAL_H_
diff --git a/intern/opensubdiv/internal/opensubdiv_topology_refiner.cc b/intern/opensubdiv/internal/opensubdiv_topology_refiner.cc
index 93f6fdc14a1..5e1e0bd86b3 100644
--- a/intern/opensubdiv/internal/opensubdiv_topology_refiner.cc
+++ b/intern/opensubdiv/internal/opensubdiv_topology_refiner.cc
@@ -340,7 +340,7 @@ bool checkEdgeSharpnessMatch(
for (int edge_index = 0; edge_index < num_edges; ++edge_index) {
const float sharpness = base_level.GetEdgeSharpness(edge_index);
const float conv_sharpness =
- opensubdiv_capi::getCompatibleEdgeSharpness(converter, edge_index);
+ converter->getEdgeSharpness(converter, edge_index);
if (sharpness != conv_sharpness) {
return false;
}
More information about the Bf-blender-cvs
mailing list