RFR: Added tests to represent bad stream handling in jdec and correct one in jdis [v3]
Jiří Vaněk
jvanek at openjdk.org
Mon Jun 20 15:48:09 UTC 2022
> 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
-------------
Changes:
- all: https://git.openjdk.org/asmtools/pull/23/files
- new: https://git.openjdk.org/asmtools/pull/23/files/2b4b8dc7..527cb9cf
Webrevs:
- full: https://webrevs.openjdk.org/?repo=asmtools&pr=23&range=02
- incr: https://webrevs.openjdk.org/?repo=asmtools&pr=23&range=01-02
Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
Patch: https://git.openjdk.org/asmtools/pull/23.diff
Fetch: git fetch https://git.openjdk.org/asmtools pull/23/head:pull/23
PR: https://git.openjdk.org/asmtools/pull/23
More information about the asmtools-dev
mailing list