I'm a moron and like to swear in ticket titles.

[Pascal258]

Inside of _ArrayDelete you get "Local $iUBound = UBound($avArray, 1) - 1". In the loop in Case 1 you also substract -1 which is wrong.
"For $i = $iElement To $iUBound - 1" should be "For $i = $iElement To $iUBound"

greetings,
Pascal257

[Valik]

Ticket title changed. Seriously, was that necessary?

[anonymous]

Replying to Valik:

Ticket title changed. Seriously, was that necessary?

So and your comment and title is/was also necessary?

[Gary]

I believe the loop is correct, if it is a 1 dimension array then you move everything up the array by one, being it is moving everything by one it needs to stop 2 before the end of the array. Same goes for a 2 dimension array.

[Valik]

Replying to anonymous:

Replying to Valik:

Ticket title changed. Seriously, was that necessary?

So and your comment and title is/was also necessary?

I don't really think that's relevant. You created a ticket for something that isn't a bug. Can you demonstrate the code, as written, presents a problem?. To top it off you do so by creating a ticket with a profane title. Why shouldn't I go out of my way to discourage you from ever wasting my time using the issue tracker again? You're not off to an auspicious start here. And your behavior has earned you a vacation from the forum.

The lesson(s) you need to take from this is/are 1) Think before you act; 2) If you're going to be an ass-hat, wear it somewhere else, please.

I'm closing this. Quite obviously, and as Gary pointed out, this is not a bug. The next array element ($i + 1) is read in the loop which means the loop must stop at UBound() - 2 which is what the code reflects. Otherwise the code will attempt to index out of bounds.