This is an unrelated bug fix: basically, tmp.data may (or may not) be filled in, it's essentially unrelated and probably a hold-over from some old version of the code, I didn't bother tracking it back.   cval.{len,str} is the list of targets, and better shows where the problem is.