On Thu, 16 Jan 2020 11:56:10 GMT, Michael Paus <mp...@openjdk.org> wrote:
>> Assuming `Nb` in `verticalTileNb` stands for number, I would recommend to >> change the names as `numVerticalTiles` and `numHorizontalTiles` > > I think the proposed code changes are wrong in case that `height / > maxTextureSize` is an exact integer. (Same for width). I normally add the 1 > only if `height % maxTextureSize != 0` Indeed, substituting the floating math for integer math requires an extra test on the remainder to be correct. Since this code isn't located in a tight loop, I'm not sure this optimization is worth making the code less straight forward. What do you all think? ------------- PR: https://git.openjdk.java.net/jfx/pull/68