Deep copy about lossypng HOT 1 OPEN

Please disregard my comment about the deep copy, I was mistaken. I now understand that the loops I linked to are supposed to shift the rows, not copy their values. However, I do think there's a bug because the loops effectively assign colorError[0] to colorError[1] and colorError[2] resulting in the issue with the 999.

Moreover, I think the last row needs to be zero'd before continuing with the calculations as it hasn't been visited by the algorithm yet.

Here is what I think should happen (Java):

// Remember first row
int[][] firstRow = colorError[0];

// Move rows one level up
for (int y = 1; y < colorError.length; y++) {
    colorError[y - 1] = colorError[y];
}

// Set new last row
colorError[colorError.length - 1] = firstRow;

// Zero last row
for (int x = 0; x < firstRow.length; x++) {
    Arrays.fill(firstRow[x], 0);
}

from lossypng.