Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
42 changes: 21 additions & 21 deletions src/LinearAlgebra/SparsityPattern/Variable.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -35,8 +35,8 @@ namespace Sparse
*/
Variable()
: value_(0.0),
variableNumber_(INVALID_VAR_NUMBER),
isFixed_(false),
variable_number_(INVALID_VAR_NUMBER),
is_fixed_(false),
dependencies_(new DependencyMap)
{
}
Expand All @@ -46,8 +46,8 @@ namespace Sparse
*/
explicit Variable(double value)
: value_(value),
variableNumber_(INVALID_VAR_NUMBER),
isFixed_(false),
variable_number_(INVALID_VAR_NUMBER),
is_fixed_(false),
dependencies_(new DependencyMap)
{
}
Expand All @@ -57,22 +57,22 @@ namespace Sparse
number.

*/
Variable(double value, size_t variableNumber)
Variable(double value, size_t variable_number)
: value_(value),
variableNumber_(variableNumber),
isFixed_(false),
variable_number_(variable_number),
is_fixed_(false),
dependencies_(new DependencyMap)
{
(*dependencies_)[variableNumber_] = 1.0;
(*dependencies_)[variable_number_] = 1.0;
}

/*!
\brief Copy constructor.
*/
Variable(const Variable& v)
: value_(v.value_),
variableNumber_(INVALID_VAR_NUMBER),
isFixed_(false),
variable_number_(INVALID_VAR_NUMBER),
is_fixed_(false),
dependencies_(new DependencyMap(*v.dependencies_))
{
}
Expand Down Expand Up @@ -115,7 +115,7 @@ namespace Sparse
value_ = rhs.value_;

// if rhs is a constant so is *this
setFixed(rhs.isFixed());
setFixed(rhs.is_fixed());

// set dependencies from rhs
dependencies_->clear(); // clear map just in case
Expand Down Expand Up @@ -181,17 +181,17 @@ namespace Sparse
*/
size_t getVariableNumber() const
{
return variableNumber_;
return variable_number_;
}

/*!
\brief Sets the variable number.
*/
void setVariableNumber(size_t variableNumber)
void setVariableNumber(size_t variable_number)
{
dependencies_->clear();
variableNumber_ = variableNumber;
(*dependencies_)[variableNumber_] = 1.0;
variable_number_ = variable_number;
(*dependencies_)[variable_number_] = 1.0;
}

/*!
Expand All @@ -203,23 +203,23 @@ namespace Sparse
*/
bool isRegistered() const
{
return variableNumber_ != INVALID_VAR_NUMBER;
return variable_number_ != INVALID_VAR_NUMBER;
}

/*!
\brief Checks whether the variable is fixed or not.
*/
bool isFixed() const
bool is_fixed() const
{
return isFixed_;
return is_fixed_;
}

/*
\brief Turns variable into parameter, or vice versa.
*/
void setFixed(bool b = false)
{
isFixed_ = b;
is_fixed_ = b;
}


Expand Down Expand Up @@ -259,8 +259,8 @@ namespace Sparse

private:
double value_; ///< Value of the variable.
size_t variableNumber_; ///< Independent variable ID
bool isFixed_; ///< Constant parameter flag.
size_t variable_number_; ///< Independent variable ID
bool is_fixed_; ///< Constant parameter flag.

mutable DependencyMap* dependencies_;
static const size_t INVALID_VAR_NUMBER = static_cast<const size_t>(-1);
Expand Down
10 changes: 5 additions & 5 deletions src/LinearAlgebra/SparsityPattern/VariableImplementation.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -59,15 +59,15 @@ namespace Sparse
{
os << value_;

if (isFixed_)
if (is_fixed_)
{
os << " (fixed)";
return;
}

if (variableNumber_ != INVALID_VAR_NUMBER)
if (variable_number_ != INVALID_VAR_NUMBER)
{
os << " (variable " << variableNumber_ << ")";
os << " (variable " << variable_number_ << ")";
return;
}

Expand Down Expand Up @@ -113,7 +113,7 @@ namespace Sparse

// -=
/*!
\brief Compound substraction-assignment operator. Right hand
\brief Compound subtraction-assignment operator. Right hand
side is a built-in double type.
*/
Variable& Variable::operator-=(const double& rhs)
Expand All @@ -123,7 +123,7 @@ namespace Sparse
}

/*!
\brief Compound substraction-assignment operator. Right hand
\brief Compound subtraction-assignment operator. Right hand
side is a Variable type.
*/
Variable& Variable::operator-=(const Variable& rhs)
Expand Down