From f44551e957a4d9cb4c5af268c5189e033b088bdc Mon Sep 17 00:00:00 2001 From: Andrew Meadows Date: Tue, 9 Jul 2019 15:14:18 -0700 Subject: [PATCH] more readable code --- libraries/workload/src/workload/RegionState.cpp | 2 +- libraries/workload/src/workload/RegionState.h | 5 ++++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/libraries/workload/src/workload/RegionState.cpp b/libraries/workload/src/workload/RegionState.cpp index ade7e47e4d..509c00a048 100644 --- a/libraries/workload/src/workload/RegionState.cpp +++ b/libraries/workload/src/workload/RegionState.cpp @@ -30,7 +30,7 @@ void RegionState::run(const workload::WorkloadContextPointer& renderContext, con // inputs[2N + 1] = vector of ids entering region N // // But we only pass inputs for R1 through R3 - assert(inputs.size() == 2 * (int32_t)(Region::R3 + 1)); + assert(inputs.size() == 2 * RegionState::NUM_REGIONS_TRACKED); // The id's in each vector are sorted in ascending order // because the source vectors are scanned in ascending order. diff --git a/libraries/workload/src/workload/RegionState.h b/libraries/workload/src/workload/RegionState.h index 9552709d8e..d941a5046c 100644 --- a/libraries/workload/src/workload/RegionState.h +++ b/libraries/workload/src/workload/RegionState.h @@ -53,8 +53,11 @@ namespace workload { using Inputs = IndexVectors; using JobModel = workload::Job::ModelI; + // we track Proxies in regions R1 through R3 + const uint32_t NUM_REGIONS_TRACKED = uint32_t(Region::R3 - Region::R1 + 1); + RegionState() { - _state.resize(Region::R3 + 1); + _state.resize(NUM_REGIONS_TRACKED); } void configure(const Config& config);