chizeta/cnorxz
chizeta/cnorxz
0
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

anonymous range: sreplace -> const; return new range instead

Browse source
This commit is contained in:
Christian Zimmermann 2019-01-18 17:49:58 +01:00
parent 8048c573fa
commit 75ead6fdba
2 changed files with 11 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/include/ranges/anonymous_range.h
View file

@ -93,8 +93,8 @@ namespace MultiArrayTools
const std::vector<std::shared_ptr<RangeBase> >& orig() const;
void sreplace(const std::shared_ptr<RangeBase> in, size_t num);
void sreplace(const std::vector<std::shared_ptr<RangeBase>>& in, size_t num);
std::shared_ptr<AnonymousRange> sreplace(const std::shared_ptr<RangeBase> in, size_t num) const;
std::shared_ptr<AnonymousRange> sreplace(const std::vector<std::shared_ptr<RangeBase>>& in, size_t num) const;
bool isEmpty() const;

13
src/lib/ranges/anonymous_range.cc
View file

@ -159,13 +159,16 @@ namespace MultiArrayTools
return mOrig.at(num);
}
void AnonymousRange::sreplace(const std::shared_ptr<RangeBase> in, size_t num)
std::shared_ptr<AnonymousRange> AnonymousRange::sreplace(const std::shared_ptr<RangeBase> in, size_t num) const
{
assert(mOrig[num]->size() == in->size());
mOrig[num] = in;
auto tmp = mOrig;
tmp[num] = in;
AnonymousRangeFactory arf(tmp);
return std::dynamic_pointer_cast<AnonymousRange>(arf.create());
}
void AnonymousRange::sreplace(const std::vector<std::shared_ptr<RangeBase>>& in, size_t num)
std::shared_ptr<AnonymousRange> AnonymousRange::sreplace(const std::vector<std::shared_ptr<RangeBase>>& in, size_t num) const
{
size_t nsize = 1;
for(auto& x: in){
@ -183,7 +186,9 @@ namespace MultiArrayTools
for(size_t i = num + 1 ; i < mOrig.size(); ++i){
norig[in.size()+i-1] = mOrig[i];
}
mOrig = std::move( norig );
//mOrig = std::move( norig );
AnonymousRangeFactory arf(norig);
return std::dynamic_pointer_cast<AnonymousRange>(arf.create());
}
const std::vector<std::shared_ptr<RangeBase> >& AnonymousRange::orig() const