This is a smallish thing, but it could be worth switching over from yield and to a list that is populated and returned. I know the syntax might be a little more verbose, but we have had several memory leaks introduced over time and they all had to do with yields. Is that ok?