On Tue, 21 Sep 2021 12:47:00 GMT, Naoto Sato <na...@openjdk.org> wrote:

> Fixing an AIOOBE on normalizing the month value.

Marked as reviewed by joehw (Reviewer).

src/java.base/share/classes/sun/util/calendar/BaseCalendar.java line 281:

> 279:             month = 13 - (xm % 12);
> 280:             if (month == 13) {
> 281:                 year ++;

Not that it matters, just happen to see it ;-). Is it more common to have no 
space before the increment operator? Code convention seems to suggest no space 
before increment and their operands. 
No update needed should you decide to remove it.

-------------

PR: https://git.openjdk.java.net/jdk/pull/5611

Reply via email to