Actions
Bug #13934
closedadd code comment to avoid reversion of 13788
Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Documentation
Target version:
Start date:
08/01/2018
Due date:
% Done:
100%
Estimated time:
(Total: 0.00 h)
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.
Actions