never take a reference over a descriptor

This commit is contained in:
Lingjie Zhu 2018-07-14 11:06:29 +08:00
parent cfd47020bd
commit eabcfd2d08
4 changed files with 7 additions and 7 deletions

View File

@ -26,7 +26,7 @@ public:
/// @{
/// Computes and returns fitting error from face f to proxy.
FT compute_error(const TriangleMesh &tm, const face_descriptor &f, const Proxy &proxy) const;
FT compute_error(const TriangleMesh &tm, const face_descriptor f, const Proxy &proxy) const;
/// Computes and returns fitted proxy from a range of faces.
/// @tparam FaceRange a range of

View File

@ -85,7 +85,7 @@ public:
* @param px proxy
* @return computed error
*/
FT compute_error(const TriangleMesh &tm, const face_descriptor &f, const Proxy &px) const {
FT compute_error(const TriangleMesh &tm, const face_descriptor f, const Proxy &px) const {
(void)(tm);
Vector_3 v = m_sum_functor(get(m_fnmap, f), m_scale_functor(px, FT(-1.0)));
return get(m_famap, f) * m_scalar_product_functor(v, v);
@ -105,7 +105,7 @@ public:
// fitting normal
Vector_3 norm = CGAL::NULL_VECTOR;
BOOST_FOREACH(const face_descriptor &f, faces) {
BOOST_FOREACH(const face_descriptor f, faces) {
norm = m_sum_functor(norm,
m_scale_functor(get(m_fnmap, f), get(m_famap, f)));
}

View File

@ -77,7 +77,7 @@ public:
* @param px proxy
* @return computed error
*/
FT compute_error(const TriangleMesh &tm, const face_descriptor &f, const Proxy &px) const {
FT compute_error(const TriangleMesh &tm, const face_descriptor f, const Proxy &px) const {
(void)(tm);
halfedge_descriptor he = halfedge(f, *m_tm);
const Point_3 &p0 = m_vpmap[source(he, *m_tm)];
@ -110,7 +110,7 @@ public:
CGAL_assertion(!faces.empty());
std::list<Triangle_3> tris;
BOOST_FOREACH(const face_descriptor &f, faces) {
BOOST_FOREACH(const face_descriptor f, faces) {
const halfedge_descriptor he = halfedge(f, *m_tm);
const Point_3 &p0 = m_vpmap[source(he, *m_tm)];
const Point_3 &p1 = m_vpmap[target(he, *m_tm)];

View File

@ -134,7 +134,7 @@ public:
#endif
// The proxy wrapper for approximation.
struct Proxy_wrapper {
Proxy_wrapper(const Proxy &p, const std::size_t &i, const face_descriptor &s, const FT &e)
Proxy_wrapper(const Proxy &p, const std::size_t &i, const face_descriptor s, const FT &e)
: px(p), idx(i), seed(s), err(e) {}
Proxy px; // parameterized proxy
@ -158,7 +158,7 @@ private:
// The facet candidate to be queued.
struct Facet_to_integrate {
Facet_to_integrate(const face_descriptor &f_, const std::size_t &px_, const FT &err_)
Facet_to_integrate(const face_descriptor f_, const std::size_t &px_, const FT &err_)
: f(f_), px(px_), err(err_) {}
bool operator<(const Facet_to_integrate &rhs) const {