Skip to content

Commit

Permalink
Refs #9244. Made changes according to code review
Browse files Browse the repository at this point in the history
used m_values.back() instead of first *m_values.rbegin() at line 840;
removed the outdated comment at line 1182;
  • Loading branch information
wdzhou committed Apr 23, 2014
1 parent 87c9d0c commit 9320183
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions Code/Mantid/Framework/Kernel/src/TimeSeriesProperty.cpp
Expand Up @@ -837,7 +837,7 @@ namespace Mantid
m_propSortedFlag = TimeSeriesSortStatus::TSUNKNOWN;
}
else if (m_propSortedFlag == TimeSeriesSortStatus::TSSORTED &&
*m_values.rbegin() < *(m_values.rbegin()+1) )
m_values.back() < *(m_values.rbegin()+1) )
{
// Previously sorted but last added is not in order
m_propSortedFlag = TimeSeriesSortStatus::TSUNSORTED;
Expand Down Expand Up @@ -1179,7 +1179,6 @@ namespace Mantid
std::size_t num = new_values.size();
for (std::size_t i=0; i < num; i++)
{
// By providing a guess iterator to the insert method, it speeds inserting up by a good amount.
TimeValueUnit<TYPE> newentry(new_times[i], new_values[i]);
m_values.push_back(newentry);
}
Expand Down

0 comments on commit 9320183

Please sign in to comment.