RFR: Added tests to represent bad stream handling in jdec and correct one in jdis [v3]
Leonid Kuskov
lkuskov at openjdk.org
Wed Jun 22 18:21:16 UTC 2022
On Mon, 20 Jun 2022 15:48:09 GMT, Jiří Vaněk <jvanek at openjdk.org> wrote:
>> The tests are now maven only. I do not knwo how to add them properly to the ant task, without introducing ivy to download junit jupiter runner, or to add burden of some wgetted dependencies.
>>
>> WDYT?
>>
>> The tests represents the issue with messe dusp tream in jdecoder, and correct behaviour in jdis.
>> I hoped I will provide fix asap once the tests ar ehere, but the inheritance there was quite a surprised, so now I'm double checking what would be actually broken by the "swap" of the jdec streams back to "normal".
>> Do you have some quick guidance how to force somethng to loggerOutput please?
>
> Jiří Vaněk has refreshed the contents of this pull request, and previous commits have been removed. The incremental views will show differences compared to the previous content of the PR. The pull request contains one new commit since the last revision:
>
> Added tests to represent bad stream handling in jdec and correctone in jdis
Thank you for adding unit tests to the project. I wanted to create something like this for ages.
LGFM
-------------
PR: https://git.openjdk.org/asmtools/pull/23
More information about the asmtools-dev
mailing list