Review request for JDK-8030642: Add golden files to javac/limits

Jonathan Gibbons jonathan.gibbons at
Tue Dec 17 13:27:58 PST 2013

Looks mostly OK.

One important thing to consider in tests like this is whether the 
messages are as good as they can/should be.   In this case, I am 
reminded that there is another bug in the system to make all limit 
messages more friendly, by including the actual value of the limit.  
Right now the messages are all of the simple form "you have exceeded the 
limit".  What you have done is OK, but you might want to look at 
updating javac, the messages, and the tests, to dynamically include the 
value of the limit in the message. (Meaning, the value should come from 
the same place/constant as used in javac, and should not be hard-wired 
into the message itself.).

I had to check the full non-raw form of the diag for string.overflow, 
but that one looks OK, along with the rest.


-- Jon (Reviewer)

On 12/17/2013 12:32 PM, Eric McCorkle wrote:
> Approved, though I am not a capital-R reviewer.
> On 12/17/13 15:23, Paul Govereau wrote:
>> Hello,
>> Please review this patch adding expected output files for the negative
>> tests in langtools/test/tools/javac/limits.
>> The bug report for this patch is here:
>> The webrev is here:
>> Thank you,
>> Paul

More information about the compiler-dev mailing list