Deleting segments in forward loop safe?

[rutledj]

I've been using the below code to remove OBR\OBX blocks where the OBR is a cancel. This seems to work fine but someone else suggested that deleting in a forward moving loop was not safe, that I should be using an indexed loop going through the segments in reverse.
Does this seem like it would cause an issue?

for each(obr in msgIn..OBR)
	{
		if(obr['OBR.25']['OBR.25.1'].toString() == 'X')
		{
			//Get all the segments after the OBR
			var segsAfterOBR = getSegmentsAfter(msgIn, msgIn.children()[obr.childIndex()],'OBX',false,'OBR');
			for each(seg in segsAfterOBR)
				delete msgIn.children()[seg.childIndex()];				
		}
	}

[pacmano1]

short answer - for each is fine. A for loop with a counter is what people are taking about when they say do it backwards and the forum discusses that in many places.