Details
-
Type:
Story
-
Status: Won't Fix
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: jointcal
-
Labels:
-
Story Points:1
-
Epic Link:
-
Team:Alert Production
Description
Paul Price claims that DM-12207 fixed the need to copy refCats in jointcal to ensure memory contiguity. While attempting to test that claim, I ran into at least one exception related to DM-5855 that made it impossible to actually test that claim.
After DM-5855 is fixed, I should try removing those refCat.copy() statements again (just search for "contiguity" throughout jointcal for the 7 places to fix it) to see if it is safe or not.
It looks like there is now only one mention of "contiguity" and that does an isContiguous check before deciding to copy the catalog.