From 39a41bb055a7c207d0732140c1b0b2d618cd3a2f Mon Sep 17 00:00:00 2001
From: Brendan Klare <brendan.klare@noblis.org>
Date: Sat, 26 Jul 2014 21:23:10 -0400
Subject: [PATCH] Update parameters to const reference in eigen utils

---
 openbr/core/eigenutils.cpp | 10 +++++-----
 openbr/core/eigenutils.h   | 10 +++++-----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/openbr/core/eigenutils.cpp b/openbr/core/eigenutils.cpp
index 795c7d5..e0601d1 100644
--- a/openbr/core/eigenutils.cpp
+++ b/openbr/core/eigenutils.cpp
@@ -77,7 +77,7 @@ float eigStd(const Eigen::MatrixXf& x) {
     return sqrt((x.array() - mean).pow(2).sum() / (x.cols() * x.rows()));
 }
 
-MatrixXf removeRowCol(MatrixXf X, int row, int col) {
+MatrixXf removeRowCol(const MatrixXf X, int row, int col) {
     MatrixXf Y(X.rows() - 1,X.cols() - 1);
 
     for (int i1 = 0, i2 = 0; i1 < X.rows(); i1++) {
@@ -96,7 +96,7 @@ MatrixXf removeRowCol(MatrixXf X, int row, int col) {
     return Y;
 }
 
-MatrixXf pointsToMatrix(QList<QPointF> points, bool isAffine) {
+MatrixXf pointsToMatrix(const QList<QPointF> points, bool isAffine) {
     MatrixXf P(points.size(), isAffine ? 3 : 2);
     for (int i = 0; i < points.size(); i++) {
         P(i, 0) = points[i].x();
@@ -107,7 +107,7 @@ MatrixXf pointsToMatrix(QList<QPointF> points, bool isAffine) {
     return P;
 }
 
-QList<QPointF> matrixToPoints(MatrixXf P) {
+QList<QPointF> matrixToPoints(const Eigen::MatrixXf P) {
     QList<QPointF> points;
     for (int i = 0; i < P.rows(); i++)
         points.append(QPointF(P(i, 0), P(i, 1)));
@@ -115,7 +115,7 @@ QList<QPointF> matrixToPoints(MatrixXf P) {
 }
 
 //Converts x y points in a single vector to two column matrix
-Eigen::MatrixXf vectorToMatrix(Eigen::MatrixXf vector) {
+Eigen::MatrixXf vectorToMatrix(const Eigen::MatrixXf vector) {
     int n = vector.rows();
     Eigen::MatrixXf matrix(n / 2, 2);
     for (int i = 0; i < n / 2; i++) {
@@ -126,7 +126,7 @@ Eigen::MatrixXf vectorToMatrix(Eigen::MatrixXf vector) {
     return matrix;
 }
 
-Eigen::MatrixXf matrixToVector(Eigen::MatrixXf matrix) {
+Eigen::MatrixXf matrixToVector(const Eigen::MatrixXf matrix) {
     int n2 = matrix.rows();
     Eigen::MatrixXf vector(n2 * 2, 1);
     for (int i = 0; i < n2; i++) {
diff --git a/openbr/core/eigenutils.h b/openbr/core/eigenutils.h
index 382c87f..0bbc013 100644
--- a/openbr/core/eigenutils.h
+++ b/openbr/core/eigenutils.h
@@ -30,15 +30,15 @@ void printEigen(Eigen::MatrixXf X);
 void printSize(Eigen::MatrixXf X);
 
 //Converts x y points in a single vector to two column matrix
-Eigen::MatrixXf vectorToMatrix(Eigen::MatrixXf vector);
-Eigen::MatrixXf matrixToVector(Eigen::MatrixXf matrix);
+Eigen::MatrixXf vectorToMatrix(const Eigen::MatrixXf vector);
+Eigen::MatrixXf matrixToVector(const Eigen::MatrixXf matrix);
 
 //Remove row and column from the matrix:
-Eigen::MatrixXf removeRowCol(Eigen::MatrixXf X, int row, int col);
+Eigen::MatrixXf removeRowCol(const Eigen::MatrixXf X, int row, int col);
 
 //Convert a point list into a matrix:
-Eigen::MatrixXf pointsToMatrix(QList<QPointF> points, bool isAffine=false);
-QList<QPointF> matrixToPoints(Eigen::MatrixXf P);
+Eigen::MatrixXf pointsToMatrix(const QList<QPointF> points, bool isAffine=false);
+QList<QPointF> matrixToPoints(const Eigen::MatrixXf P);
 
 template<typename _Scalar, int _Rows, int _Cols, int _Options, int _MaxRows, int _MaxCols>
 inline QDataStream &operator<<(QDataStream &stream, const Eigen::Matrix< _Scalar, _Rows, _Cols, _Options, _MaxRows, _MaxCols > &mat)
--
libgit2 0.21.4