summaryrefslogtreecommitdiff
path: root/src/mbgl/renderer/line_bucket.cpp
diff options
context:
space:
mode:
authorBruno de Oliveira Abinader <bruno@mapbox.com>2015-07-07 19:37:22 +0300
committerBruno de Oliveira Abinader <bruno@mapbox.com>2015-07-29 11:03:47 +0300
commit58b93553c10b77eedd573104b56d63916c635cbb (patch)
tree368648b8ec4af1c82f5dde5f66747975fd04d86c /src/mbgl/renderer/line_bucket.cpp
parent807d87f0f5883e1c697433eca381c860a74596f2 (diff)
downloadqtlocation-mapboxgl-58b93553c10b77eedd573104b56d63916c635cbb.tar.gz
Replace size() with empty() whenever possible
Before C++11, std::list's implementation of size was O(n). It should be all O(1) now, but it is probably still a good idea to use empty() to emphasize the intend.
Diffstat (limited to 'src/mbgl/renderer/line_bucket.cpp')
-rw-r--r--src/mbgl/renderer/line_bucket.cpp3
1 files changed, 1 insertions, 2 deletions
diff --git a/src/mbgl/renderer/line_bucket.cpp b/src/mbgl/renderer/line_bucket.cpp
index 446c75de76..24bda6d0a9 100644
--- a/src/mbgl/renderer/line_bucket.cpp
+++ b/src/mbgl/renderer/line_bucket.cpp
@@ -308,8 +308,7 @@ void LineBucket::addGeometry(const std::vector<Coordinate>& vertices) {
// Store the triangle/line groups.
{
- if (!triangleGroups.size() ||
- (triangleGroups.back()->vertex_length + vertexCount > 65535)) {
+ if (triangleGroups.empty() || (triangleGroups.back()->vertex_length + vertexCount > 65535)) {
// Move to a new group because the old one can't hold the geometry.
triangleGroups.emplace_back(std::make_unique<TriangleGroup>());
}