General: Replace loop with resize where applicable

Instead of default constructing in a loop for std::vector, we can just
use resize or its size constructor to do the same thing.
This commit is contained in:
Lioncash 2019-09-05 19:54:14 -04:00
parent fa56ba5db6
commit 4e71b955f6
2 changed files with 3 additions and 9 deletions

View File

@ -525,9 +525,7 @@ bool Table::CellView::reset(size_t c, size_t r) {
std::vector<Table::ColumnPool>& Table::ensureCellPools(size_t rows, size_t cols, ViewResources& res) { std::vector<Table::ColumnPool>& Table::ensureCellPools(size_t rows, size_t cols, ViewResources& res) {
if (m_cellPools.size() < cols) { if (m_cellPools.size() < cols) {
m_cellPools.reserve(cols); m_cellPools.resize(cols);
for (size_t i = m_cellPools.size(); i < cols; ++i)
m_cellPools.emplace_back();
m_ensuredRows = 0; m_ensuredRows = 0;
} }
if (m_ensuredRows < rows) { if (m_ensuredRows < rows) {
@ -569,9 +567,7 @@ void Table::_updateData() {
if (newViewChildren) { if (newViewChildren) {
m_headerViews.clear(); m_headerViews.clear();
m_cellPools.clear(); m_cellPools.clear();
m_headerViews.reserve(m_columns); m_headerViews.resize(m_columns);
for (size_t c = 0; c < m_columns; ++c)
m_headerViews.emplace_back();
m_ensuredRows = 0; m_ensuredRows = 0;
} }
ensureCellPools(m_rows, m_columns, res); ensureCellPools(m_rows, m_columns, res);

View File

@ -20,11 +20,9 @@ void Toolbar::Resources::init(boo::IGraphicsDataFactory::Context& ctx, const ITh
Toolbar::Toolbar(ViewResources& res, View& parentView, Position tbPos, unsigned units) Toolbar::Toolbar(ViewResources& res, View& parentView, Position tbPos, unsigned units)
: View(res, parentView) : View(res, parentView)
, m_units(units) , m_units(units)
, m_children(units)
, m_nomGauge(res.pixelFactor() * SPECTER_TOOLBAR_GAUGE * units) , m_nomGauge(res.pixelFactor() * SPECTER_TOOLBAR_GAUGE * units)
, m_padding(res.pixelFactor() * TOOLBAR_PADDING) { , m_padding(res.pixelFactor() * TOOLBAR_PADDING) {
m_children.reserve(units);
for (unsigned u = 0; u < units; ++u)
m_children.emplace_back();
commitResources(res, [&](boo::IGraphicsDataFactory::Context& ctx) -> bool { commitResources(res, [&](boo::IGraphicsDataFactory::Context& ctx) -> bool {
buildResources(ctx, res); buildResources(ctx, res);
m_tbBlockBuf = res.m_viewRes.m_bufPool.allocateBlock(res.m_factory); m_tbBlockBuf = res.m_viewRes.m_bufPool.allocateBlock(res.m_factory);