your sample code is not processing the very last pixel in your png example.

[bvdeenen]

Hi Pedro
I was trying the Zig array out of bounds check while having built your png example and just added +2 to the buffer[index + ..] statements.
To my surprise the array bounds checker didn't trigger.
A bit of searching led me the conclusion your while loop limit is wrong. It should just be index<len. The limit is checked before the loop body is executed, the increment (index+=4) after the loop body has been executed.

Thanks for the great work!

[pedropark99]

Thank you @bvdeenen! I will take a look at it

[bvdeenen]

Thank you @bvdeenen! I will take a look at it

I just checked to make sure, and I'm right:

    var index: u32 = 0;
    const len = 12;
    while (index < len - 4) : (index += 4) {
        try stdout.print("index={},  ", .{index});
    }

This prints 0 and 4, whereas the last one should be 8.

[pedropark99]

Solved by #98

[bvdeenen]

I'm pretty sure the issue still exists after the merge. Probably forgotten in the refactor. @pedropark99

[pedropark99]

Thanks, fixed.