Project

General

Profile

Actions

Bug #13934

closed

add code comment to avoid reversion of 13788

Added by Joshua Randall almost 6 years ago. Updated over 5 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Documentation
Target version:
Story points:
-
Release:
Release relationship:
Auto

Description

While working on 13933, I inadvertently reverted the concurrent map access bug that was fixed in 13788 (by moving the list var declaration out of the loop so that I could use it for the initial `limit: 0` `count: exact` call).

I would suggest adding a comment to the var declaration to make a particular point of it being absolutely necessary to declare it afresh in each loop iteration, as I suspect it would otherwise be likely that the same mistake will be made again.


Subtasks 1 (0 open1 closed)

Task #13946: Review 13934-guard-commentClosedTom Morris08/01/2018Actions
Actions

Also available in: Atom PDF