use reserve for std::vector

[schiller-manuel]

No description provided.

[wcandillon]

nice 🙌🏼 silly question, could this be done in the vector constructor?

[schiller-manuel]

Yes, the constructor can be called with a size. However, this would default initialize each element, and those default initialized elements would then have to be overwritten:

Example:

std::vector<SkPoint> points;
points.reserve(pointsSize);
for (int i = 0; i < pointsSize; i++) {
   points.push_back(...);
}

Instead:

std::vector<SkPoint> points(pointsSize);
for (int i = 0; i < pointsSize; i++) {
   points[i] = ...
}

To avoid the unecessary default constructor calls, I'd go with reserve.