RFR(XS) : 8166164 : compiler/compilercontrol/share/processors/LogProcessor.java does not close Scanner

Igor Ignatyev igor.ignatyev at oracle.com
Tue Sep 20 14:37:14 UTC 2016


Pavel,

thanks for review.

— Igor
> On Sep 19, 2016, at 3:18 PM, Pavel Punegov <pavel.punegov at oracle.com> wrote:
> 
> Hi Igor,
> 
> the change looks good. Thanks for fixing.
> 
> — Pavel.
> 
>> On 19 Sep 2016, at 12:38, Igor Ignatyev <igor.ignatyev at oracle.com> wrote:
>> 
>> http://cr.openjdk.java.net/~iignatyev/8166164/webrev.00/
>>> 16 lines changed: 2 ins; 0 del; 14 mod;
>> 
>> Hi all,
>> 
>> could you please review this small patch which fixes resource leak in compiler/compilercontrol tests? 
>> LogProcessor::getScanner creates a new Scanner, but there is no code which closes it. This leak leads to 'failed to clean up files after test’ error from jtreg.
>> 
>> the fix was tested by running :hotspot_compiler test group.
>> 
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8166164
>> webrev: http://cr.openjdk.java.net/~iignatyev/8166164/webrev.00/
>> 
>> Thanks,
>> — Igor
>> 
> 



More information about the hotspot-compiler-dev mailing list