Skip to content

Commit 771f951

Browse files
committed
QPR-12140 cleanup and avoid illegal access
1 parent a61680d commit 771f951

1 file changed

Lines changed: 2 additions & 7 deletions

File tree

ql/math/randomnumbers/burley2020sobolrsg.cpp

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -109,18 +109,14 @@ namespace QuantLib {
109109
const auto& seq =
110110
sobolRsg_->skipTo(nested_uniform_scramble(nextSequenceCounter_, group4Seeds_[0]));
111111
std::copy(seq.begin(), seq.end(), integerSequence_.begin());
112-
bool updating = true;
113112
Size i = 0, group = 0;
114113
do {
115114
Size seed = group4Seeds_[group++];
116-
for (Size g = 0; g < 4; ++g) {
115+
for (Size g = 0; g < 4 && i < dimensionality_; ++g, ++i) {
117116
boost::hash_combine(seed, g);
118117
integerSequence_[i] = nested_uniform_scramble(integerSequence_[i], seed);
119-
if (++i >= dimensionality_) {
120-
updating = false;
121-
}
122118
}
123-
} while (updating);
119+
} while (i < dimensionality_);
124120
++nextSequenceCounter_;
125121
return integerSequence_;
126122
}
@@ -132,5 +128,4 @@ namespace QuantLib {
132128
sequence_.value[k] = static_cast<double>(v[k]) / 4294967296.0;
133129
return sequence_;
134130
}
135-
136131
}

0 commit comments

Comments
 (0)