Commit 4dd321bb authored by Jan Möbius's avatar Jan Möbius

Merge branch 'warn2' into 'master'

More warning fixes

See merge request !100
parents 00090fc2 08160c15
Pipeline #3413 passed with stage
in 41 minutes and 37 seconds
......@@ -82,7 +82,7 @@ distPointTriangleSquared( const Point& _p,
const Point v0v1 = _v1 - _v0;
const Point v0v2 = _v2 - _v0;
const Point n = v0v1 % v0v2; // not normalized !
const double d = n.sqrnorm();
const Scalar d = n.sqrnorm();
// Check if the triangle is degenerated
......@@ -94,16 +94,16 @@ distPointTriangleSquared( const Point& _p,
// these are not needed for every point, should still perform
// better with many points against one triangle
const Point v1v2 = _v2 - _v1;
const double inv_v0v2_2 = 1.0 / v0v2.sqrnorm();
const double inv_v0v1_2 = 1.0 / v0v1.sqrnorm();
const double inv_v1v2_2 = 1.0 / v1v2.sqrnorm();
const Scalar inv_v0v2_2 = static_cast<Scalar>(1.0) / v0v2.sqrnorm();
const Scalar inv_v0v1_2 = static_cast<Scalar>(1.0) / v0v1.sqrnorm();
const Scalar inv_v1v2_2 = static_cast<Scalar>(1.0) / v1v2.sqrnorm();
Point v0p = _p - _v0;
Point t = v0p % n;
typename Point::value_type s01, s02, s12;
const double a = (t | v0v2) * -invD;
const double b = (t | v0v1) * invD;
const Scalar a = (t | v0v2) * -invD;
const Scalar b = (t | v0v1) * invD;
if (a < 0)
{
......
......@@ -992,7 +992,7 @@ void FFc<M>::raise(typename M::FaceHandle& _fh, state_t _target_state)
// choose coefficient c
typename M::Scalar c = Base::coeff();
position *= (1.0 - c);
position *= (static_cast<typename M::Scalar>(1.0) - c);
position += MOBJ(_fh).position(_target_state - 1) * c;
MOBJ(_fh).set_position(_target_state, position);
......
......@@ -882,7 +882,8 @@ void CompositeT<MeshType,RealType>::EVc(Coeff& _coeff)
++valence;
}
c = _coeff(valence);
// Coefficients always work on double so we cast them to the correct scalar here
c = static_cast<scalar_t>(_coeff(valence));
for (voh_it = mesh_.voh_iter(*v_it); voh_it.is_valid(); ++voh_it) {
cog += mesh_.data(mesh_.edge_handle(*voh_it)).position() * c;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment