Skip to content

Commit 8c7bcab

Browse files
committed
Style, Histo: Fix -pedantic warnings (extra ;)
1 parent a571614 commit 8c7bcab

File tree

1 file changed

+11
-11
lines changed

1 file changed

+11
-11
lines changed

external/histo/include/histo.hpp

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ std::vector<PRECI> GenerateBreaksFromRangeAndBins(
7777
breaks[i] = low + i * width;
7878
}
7979
return breaks;
80-
};
80+
}
8181
/** @brief @sa GenerateBreaksFromRangeAndBins() */
8282
template <typename PRECI = double>
8383
std::vector<PRECI>
@@ -86,7 +86,7 @@ GenerateBreaksFromRangeAndBins(const std::pair<PRECI, PRECI> &range_low_upper,
8686
auto low = range_low_upper.first;
8787
auto upper = range_low_upper.second;
8888
return GenerateBreaksFromRangeAndBins<PRECI>(low, upper, bins);
89-
};
89+
}
9090

9191
/**
9292
*
@@ -113,7 +113,7 @@ std::vector<PRECI> GenerateBreaksFromRangeAndWidth(const PRECI &low,
113113
br += width;
114114
}
115115
return breaks;
116-
};
116+
}
117117

118118
/** @brief @sa GenerateBreaksFromRangeAndWidth() */
119119
template <typename PRECI = double>
@@ -123,7 +123,7 @@ GenerateBreaksFromRangeAndWidth(const std::pair<PRECI, PRECI> &range_low_upper,
123123
auto low = range_low_upper.first;
124124
auto upper = range_low_upper.second;
125125
return GenerateBreaksFromRangeAndWidth<PRECI>(low, upper, width);
126-
};
126+
}
127127
/** @} */
128128

129129
/** Exception class for Histo */
@@ -209,7 +209,7 @@ struct Histo {
209209
bins = static_cast<decltype(bins)>(breaks.size() - 1);
210210
ResetCounts();
211211
FillCounts(data);
212-
};
212+
}
213213

214214
/**
215215
* @brief Constructor with fixed input range.
@@ -226,7 +226,7 @@ struct Histo {
226226
bins = static_cast<decltype(bins)>(breaks.size() - 1);
227227
ResetCounts();
228228
FillCounts(data);
229-
};
229+
}
230230
/**
231231
* @brief Constructor that accepts a vector of breaks.
232232
* You can use @sa histo::GenerateBreaksFromRangeAndBins
@@ -245,7 +245,7 @@ struct Histo {
245245
bins = static_cast<decltype(bins)>(breaks.size() - 1);
246246
ResetCounts();
247247
FillCounts(data);
248-
};
248+
}
249249

250250
/********* PUBLIC METHODS ***********/
251251
BreaksType ComputeBinCenters() const {
@@ -354,7 +354,7 @@ struct Histo {
354354
}
355355

356356
return lo;
357-
};
357+
}
358358

359359
/** @brief Resize counts and reset value to zero. */
360360
void ResetCounts() {
@@ -377,7 +377,7 @@ struct Histo {
377377
counts[IndexFromValue(v)]++;
378378
}
379379
return counts;
380-
};
380+
}
381381

382382
/** \defgroup CountsManipulation Counts Safe Manipulation */
383383
/** @{
@@ -454,7 +454,7 @@ struct Histo {
454454
throw histo_error("CalculateBreaks: No Valid Method selected to "
455455
"calculate breaks.");
456456
}
457-
};
457+
}
458458

459459
bool
460460
CheckBreaksAreEquidistant(const BreaksType &input_breaks) const {
@@ -594,7 +594,7 @@ struct Histo {
594594
// << breaks[bins] << std::endl; std::for_each(std::begin(breaks),
595595
// std::end(breaks), [](const T& v) {std::cout<<v <<std::endl;});
596596
return this->breaks;
597-
};
597+
}
598598
};
599599

600600
template <typename PRECI = double, typename PRECI_INTEGER = size_t>

0 commit comments

Comments
 (0)