VFlip() and FFlip are const members but there is no need of returning a pointer to a Const element

This commit is contained in:
Paolo Cignoni 2012-11-09 11:11:50 +00:00
parent 6b48acfa9f
commit 1a8c1dafe2
1 changed files with 4 additions and 26 deletions

View File

@ -211,24 +211,16 @@ public:
assert(f->V(f->Prev(z))!=v && (f->V(f->Next(z))==v || f->V(z)==v));
}
// return the vertex that it should have if we make FlipV;
VertexType *VFlip()
{
assert(f->V(f->Prev(z))!=v && (f->V(f->Next(z))==v || f->V(z)==v));
if(f->V(f->Next(z))==v) return f->V(z);
else return f->V(f->Next(z));
}
// return the vertex that it should have if we make FlipV;
const VertexType *VFlip() const
/// return the vertex that it should have if we make FlipV;
VertexType *VFlip() const
{
assert(f->cV(f->Prev(z))!=v && (f->cV(f->Next(z))==v || f->cV(z)==v));
if(f->cV(f->Next(z))==v) return f->cV(z);
else return f->cV(f->Next(z));
}
// return the face that it should have if we make FlipF;
const FaceType *FFlip() const
/// return the face that it should have if we make FlipF;
FaceType *FFlip() const
{
assert( f->FFp(z)->FFp(f->FFi(z))==f );
assert(f->V(f->Prev(z))!=v && (f->V(f->Next(z))==v || f->V((z+0)%f->VN())==v));
@ -373,20 +365,6 @@ public:
}
};
template <class FaceType>
/** Class PosN.
This structure is equivalent to a Pos, but it contains a normal.
@param FaceType (Template-Parameter) Specifies the type of the faces
*/
class PosN : public Pos<FaceType>
{
public:
typedef typename FaceType::CoordType CoordType;
//normale per visualizzazione creaseangle
CoordType normal;
};
/** Class VFIterator.
This class is used as an iterator over the VF adjacency.
It allow to easily traverse all the faces around a given vertex v;