Cppcheck warnings.

--HG--
branch : develop
This commit is contained in:
dismine 2014-12-21 13:35:12 +02:00
parent bd6cf95340
commit d8dfa651ff
4 changed files with 11 additions and 15 deletions

View file

@ -66,7 +66,7 @@ QVector<QPointF> VAbstractCurve::GetSegmentPoints(const QPointF &begin, const QP
//--------------------------------------------------------------------------------------------------------------------- //---------------------------------------------------------------------------------------------------------------------
QVector<QPointF> VAbstractCurve::FromBegin(const QVector<QPointF> &points, const QPointF &begin) const QVector<QPointF> VAbstractCurve::FromBegin(const QVector<QPointF> &points, const QPointF &begin)
{ {
if (points.count() >= 2) if (points.count() >= 2)
{ {

View file

@ -53,7 +53,7 @@ public:
protected: protected:
QPainterPath ShowDirection(const QVector<QPointF> &points) const; QPainterPath ShowDirection(const QVector<QPointF> &points) const;
private: private:
QVector<QPointF> FromBegin(const QVector<QPointF> &points, const QPointF &begin) const; static QVector<QPointF> FromBegin(const QVector<QPointF> &points, const QPointF &begin);
QVector<QPointF> ToEnd(const QVector<QPointF> &points, const QPointF &end) const; QVector<QPointF> ToEnd(const QVector<QPointF> &points, const QPointF &end) const;
}; };

View file

@ -94,7 +94,6 @@ QString VPatternConverter::XSDSchema(int ver) const
{ {
const QString errorMsg(tr("Unexpected version \"%1\".").arg(ver, 0, 16)); const QString errorMsg(tr("Unexpected version \"%1\".").arg(ver, 0, 16));
throw VException(errorMsg); throw VException(errorMsg);
break;
} }
} }
} }
@ -134,7 +133,7 @@ void VPatternConverter::ApplyPatches()
QFile file(backupFileName); QFile file(backupFileName);
file.remove(); file.remove();
throw e; throw;
} }
} }

View file

@ -763,7 +763,6 @@ static halfedge_t* del_valid_link( halfedge_t *b )
{ {
point2d_t *g, *g_p, *d, *d_p; point2d_t *g, *g_p, *d, *d_p;
halfedge_t *gd, *dd, *new_gd, *new_dd; halfedge_t *gd, *dd, *new_gd, *new_dd;
int a;
g = b->vertex; g = b->vertex;
gd = del_valid_left(b); gd = del_valid_left(b);
@ -777,7 +776,7 @@ static halfedge_t* del_valid_link( halfedge_t *b )
if( g != g_p && d != d_p ) if( g != g_p && d != d_p )
{ {
a = in_circle(g, d, g_p, d_p); int a = in_circle(g, d, g_p, d_p);
if( a != ON_CIRCLE ) if( a != ON_CIRCLE )
{ {
@ -822,7 +821,6 @@ static halfedge_t* del_valid_link( halfedge_t *b )
*/ */
static halfedge_t* del_get_lower_tangent( delaunay_t *left, delaunay_t *right ) static halfedge_t* del_get_lower_tangent( delaunay_t *left, delaunay_t *right )
{ {
point2d_t *pl, *pr;
halfedge_t *right_d, *left_d, *new_ld, *new_rd; halfedge_t *right_d, *left_d, *new_ld, *new_rd;
int sl, sr; int sl, sr;
@ -830,8 +828,8 @@ static halfedge_t* del_get_lower_tangent( delaunay_t *left, delaunay_t *right )
right_d = right->leftmost_he; right_d = right->leftmost_he;
do { do {
pl = left_d->prev->pair->vertex; point2d_t *pl = left_d->prev->pair->vertex;
pr = right_d->pair->vertex; point2d_t *pr = right_d->pair->vertex;
if( (sl = classify_point_seg(left_d->vertex, right_d->vertex, pl)) == ON_RIGHT ) { if( (sl = classify_point_seg(left_d->vertex, right_d->vertex, pl)) == ON_RIGHT ) {
left_d = left_d->prev->pair; left_d = left_d->prev->pair;
@ -915,13 +913,12 @@ static void del_link( delaunay_t *result, delaunay_t *left, delaunay_t *right )
void del_divide_and_conquer( delaunay_t *del, int start, int end ) void del_divide_and_conquer( delaunay_t *del, int start, int end )
{ {
delaunay_t left, right; delaunay_t left, right;
int i, n;
n = (end - start + 1); int n = (end - start + 1);
if( n > 3 ) if( n > 3 )
{ {
i = (n / 2) + (n & 1); int i = (n / 2) + (n & 1);
left.points = del->points; left.points = del->points;
right.points = del->points; right.points = del->points;
del_divide_and_conquer( &left, start, start + i - 1 ); del_divide_and_conquer( &left, start, start + i - 1 );
@ -1001,7 +998,7 @@ void del_build_faces( delaunay_t *del )
delaunay2d_t* delaunay2d_from(del_point2d_t *points, unsigned int num_points) { delaunay2d_t* delaunay2d_from(del_point2d_t *points, unsigned int num_points) {
delaunay2d_t* res = NULL; delaunay2d_t* res = NULL;
delaunay_t del; delaunay_t del;
unsigned int i, j, fbuff_size = 0; unsigned int i;
unsigned int* faces = NULL; unsigned int* faces = NULL;
#if PREDICATE == EXACT_PREDICATE #if PREDICATE == EXACT_PREDICATE
@ -1029,13 +1026,13 @@ delaunay2d_t* delaunay2d_from(del_point2d_t *points, unsigned int num_points) {
del_build_faces( &del ); del_build_faces( &del );
fbuff_size = 0; unsigned int fbuff_size = 0;
for( i = 0; i < del.num_faces; i++ ) for( i = 0; i < del.num_faces; i++ )
fbuff_size += del.faces[i].num_verts + 1; fbuff_size += del.faces[i].num_verts + 1;
faces = (unsigned int*)malloc(sizeof(unsigned int) * fbuff_size); faces = (unsigned int*)malloc(sizeof(unsigned int) * fbuff_size);
j = 0; unsigned int j = 0;
for( i = 0; i < del.num_faces; i++ ) for( i = 0; i < del.num_faces; i++ )
{ {
halfedge_t *curr; halfedge_t *curr;