Thierry Louge / gammalib

Commit 1829c8f18fa75606f50af484af65a2a13512da84

Authored by Jürgen Knödlseder
1 parent a1742ade

Use const pointers instead of using const_casts.

Inline Side-by-side
Showing 1 changed file with 8 additions and 14 deletions
inst/cta/src/GCTAResponseTable.cpp
  View file @1829c8f
... ... @@ -1673,13 +1673,11 @@ void GCTAResponseTable::read_pars(const GFitsTable& hdu)
1673 1673 * that are used for linear interpolation.
1674 1674 *
1675 1675 * @todo Write down formula
1676   - *
1677   - * @todo Makes GNodeArray::set_value method const and use mutable members
1678 1676 ***************************************************************************/
1679 1677 void GCTAResponseTable::update(const double& arg) const
1680 1678 {
1681   - // Get pointer to node array (circumvent const correctness)
1682   - GNodeArray* nodes = const_cast<GNodeArray*>(&(m_axis_nodes[0]));
  1679 + // Get pointer to node array
  1680 + const GNodeArray* nodes = &(m_axis_nodes[0]);
1683 1681  
1684 1682 // Set value for node array
1685 1683 nodes->set_value(arg);
... ... @@ -1706,14 +1704,12 @@ void GCTAResponseTable::update(const double&amp; arg) const
1706 1704 * that are used for bilinear interpolation.
1707 1705 *
1708 1706 * @todo Write down formula
1709   - *
1710   - * @todo Makes GNodeArray::set_value method const and use mutable members
1711 1707 ***************************************************************************/
1712 1708 void GCTAResponseTable::update(const double& arg1, const double& arg2) const
1713 1709 {
1714   - // Get pointers to node arrays (circumvent const correctness)
1715   - GNodeArray* nodes1 = const_cast<GNodeArray*>(&(m_axis_nodes[0]));
1716   - GNodeArray* nodes2 = const_cast<GNodeArray*>(&(m_axis_nodes[1]));
  1710 + // Get pointers to node arrays
  1711 + const GNodeArray* nodes1 = &(m_axis_nodes[0]);
  1712 + const GNodeArray* nodes2 = &(m_axis_nodes[1]);
1717 1713  
1718 1714 // Set values for node arrays
1719 1715 nodes1->set_value(arg1);
... ... @@ -1753,16 +1749,14 @@ void GCTAResponseTable::update(const double&amp; arg1, const double&amp; arg2) const
1753 1749 * that are used for bilinear interpolation.
1754 1750 *
1755 1751 * @todo Write down formula
1756   - *
1757   - * @todo Makes GNodeArray::set_value method const and use mutable members
1758 1752 ***************************************************************************/
1759 1753 void GCTAResponseTable::update(const double& arg1, const double& arg2,
1760 1754 const double& arg3) const
1761 1755 {
1762 1756 // Get pointers to node arrays (circumvent const correctness)
1763   - GNodeArray* nodes1 = const_cast<GNodeArray*>(&(m_axis_nodes[0]));
1764   - GNodeArray* nodes2 = const_cast<GNodeArray*>(&(m_axis_nodes[1]));
1765   - GNodeArray* nodes3 = const_cast<GNodeArray*>(&(m_axis_nodes[2]));
  1757 + const GNodeArray* nodes1 = &(m_axis_nodes[0]);
  1758 + const GNodeArray* nodes2 = &(m_axis_nodes[1]);
  1759 + const GNodeArray* nodes3 = &(m_axis_nodes[2]);
1766 1760  
1767 1761 // Set values for node arrays
1768 1762 nodes1->set_value(arg1);
... ...