On Tuesday, 7 July 2015 at 08:50:45 UTC, Manu wrote:
Most iterators are already size_t.
.length should have been int but it's too late.
It's not because it's a "size" that it should be size_t.
Indexes shoulds be size_t for conventions sake, otherwise you
cast everywhere.
Disagree. I think it's the other way around.
And image coordinates are not especially indices.
Indices should be unsigned, otherwise you need to do 2 tests for
validity, rather than just one; (x >= 0 && x < length) rather
than
just (x < length).
You can force an unsigned comparison with (cast(uint)x < length).
Signed indices optimize better in loops because signed overflow
is undefined behaviour.
http://stackoverflow.com/questions/18795453/why-prefer-signed-over-unsigned-in-c
