[10] RFR 8187551: MessageFormat.setFormat(int, Format) AIOOBE not thrown when documented (original) (raw)

Roger Riggs Roger.Riggs at Oracle.com
Fri Dec 1 15:10:55 UTC 2017


Hi Nishit,

Please add the new test to test/jdk/java/text/Format/MessageFormat/MessageRegression.java instead of creating a new test.

Also the convention for test names should be a functional description of the test; bug numbers are uninformative.

(I'm a bit surprised there are few/no regression tests for setFormat).

Thanks, Roger

On 12/1/2017 3:17 AM, Nishit Jain wrote:

Hi,

Please review the fix for JDK-8187551. Bug: https://bugs.openjdk.java.net/browse/JDK-8187551 Webrev: http://cr.openjdk.java.net/~nishjain/8187551/webrev.02/ Fix: As documented, updated MessageFormat.setFormat() to throw AIOOBE with invalid format element index. Thanks Martin Buchholz for filing the issue and suggesting the fix. Regards, Nishit Jain



More information about the core-libs-dev mailing list