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