Commit 165d8e0d authored by Alexander Wiebel's avatar Alexander Wiebel
Browse files

[CHANGE] small code improvements (added consts and changed from char size_t)

parent 8a075c1a
......@@ -49,7 +49,7 @@ public:
* Constructs a value set with values of type T. Sets order and dimension
* to allow to interprete the values as tensors of a certain order and dimension.
*/
WValueSet( char order, char dimension, const std::vector< T > data, dataType inDataType )
WValueSet( size_t order, size_t dimension, const std::vector< T > data, dataType inDataType )
: WValueSetBase( order, dimension, inDataType ),
m_data( data )
{
......
......@@ -28,7 +28,7 @@
#include "WValueSetBase.h"
WValueSetBase::WValueSetBase( char order, char dimension, dataType inDataType )
WValueSetBase::WValueSetBase( size_t order, size_t dimension, dataType inDataType )
: m_order( order ),
m_dimension( dimension ),
m_dataType( inDataType )
......
......@@ -39,7 +39,7 @@ public:
* Despite this is an abstract class all subclasses should have an order
* and dimension.
*/
WValueSetBase( char order, char dimension, dataType inDataType );
WValueSetBase( size_t order, size_t dimension, dataType inDataType );
/**
* Dummy since each class with virtual member functions needs one.
......@@ -59,7 +59,7 @@ public:
/**
* \return Dimension of the values in this ValueSet
*/
virtual size_t dimension()
virtual size_t dimension() const
{
return m_dimension;
}
......@@ -67,7 +67,7 @@ public:
/**
* \return Order of the values in this ValueSet
*/
virtual size_t order()
virtual size_t order() const
{
return m_dimension;
}
......@@ -75,7 +75,7 @@ public:
/**
* \return Dimension of the values in this ValueSet
*/
virtual dataType getDataType()
virtual dataType getDataType() const
{
return m_dataType;
}
......@@ -84,12 +84,12 @@ protected:
/**
* The order of the tensors for this ValueSet
*/
char m_order;
size_t m_order;
/**
* The dimension of the tensors for this ValueSet
*/
char m_dimension;
size_t m_dimension;
/**
* The data type of the values' elements.
......
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