RFR: CODETOOLS-7902240: Remove dependence on TestResult from ReadAheadIterator

dmitry.bessonov at oracle.com dmitry.bessonov at oracle.com
Wed Jul 25 17:57:43 UTC 2018


On 25/07/2018 00:17, dmitry.bessonov at oracle.com wrote:
> On 24/07/2018 23:39, Jonathan Gibbons wrote:
>> An arguably more abstract way of doing this would be to declare
>>
>>    36 public class ReadAheadIterator<T> implements Iterator<T>
>>
>> so that there is no unnecessary dependence on TestResult in this class.
>>
>> The dependence on TestResult can then be expressed at the use site.
> 
> Agreed.
> Will implement as part of a follow-up pack of changes.

Please review the fix:

JBS: https://bugs.openjdk.java.net/browse/CODETOOLS-7902240
Webrev: http://cr.openjdk.java.net/~dbessono/7902240/webrev/


dmitry



More information about the jtharness-dev mailing list