From 0970a8618d5b1cbe855531b881f36c2e6e25cc90 Mon Sep 17 00:00:00 2001
From: ip_gpu <ip_gpu@mail.ru>
Date: Mon, 30 Oct 2017 16:53:14 +0500
Subject: [PATCH] fixed from PVS-Studio:

V533 It is likely that a wrong variable is being incremented inside the 'for' operator. Consider reviewing 'i'. sparsematrix.inl 212
V535 The variable 'i' is being used for this loop and for the outer loop. Check lines: 255, 256. geometry.h 256
V607 Ownerless expression 'j < remains'. allocator.h 120
---
 Src/Allocator.h      | 2 +-
 Src/Geometry.h       | 1 -
 Src/SparseMatrix.inl | 2 +-
 3 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/Src/Allocator.h b/Src/Allocator.h
index 3ba23d9d..58024901 100644
--- a/Src/Allocator.h
+++ b/Src/Allocator.h
@@ -117,7 +117,7 @@ class Allocator
 				remains=state.remains;
 			}
 			else{
-				for(int j=0;j<state.remains;j<remains){
+				for(int j=0;j<state.remains;j++){
 					memory[index][j].~T();
 					new(&memory[index][j]) T();
 				}
diff --git a/Src/Geometry.h b/Src/Geometry.h
index ea8c951c..f8188854 100644
--- a/Src/Geometry.h
+++ b/Src/Geometry.h
@@ -253,7 +253,6 @@ class Triangle{
 		double d=0;
 		int i,j;
 		for(i=0;i<3;i++){
-	  for(i=0;i<3;i++)
 			for(j=0;j<3;j++){d+=(p[(i+1)%3][j]-p[i][j])*(p[(i+1)%3][j]-p[i][j]);}
 		}
 		return Area()/d;
diff --git a/Src/SparseMatrix.inl b/Src/SparseMatrix.inl
index 763590ae..902f8fe7 100644
--- a/Src/SparseMatrix.inl
+++ b/Src/SparseMatrix.inl
@@ -209,7 +209,7 @@ SparseMatrix<T> SparseMatrix<T>::operator * (const T& V) const
 template<class T>
 SparseMatrix<T>& SparseMatrix<T>::operator *= (const T& V)
 {
-	for( int i=0 ; i<rows ; i++ ) for( int ii=0 ; ii<rowSizes[i] ; i++ ) m_ppElements[i][ii].Value *= V;
+	for( int i=0 ; i<rows ; i++ ) for( int ii=0 ; ii<rowSizes[i] ; ii++ ) m_ppElements[i][ii].Value *= V;
 	return *this;
 }