RFR: WIP adding support for stdin [v3]

Jiří Vaněk jvanek at openjdk.org
Sun Aug 21 15:16:25 UTC 2022


> Now Added support for stdin in jdis. Before merging, I would like to add support to all four (not sure with jcdec) tools.
> This PR serves for early feedback in approach. Hintsvery  welcomed. 
> 
> I personally miss stdin support in this toolchain. Also I think, that for any (test) wrapper, stdin will be much more comfortable then constantly create tmp files
> 
> As for the possibel future (test) wrapper, most likely custom stream implementation of ToolInput will arrive sooner or later.

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:

  stdin is now read also by jasm,jcod,jdec
  
  more tests needed

-------------

Changes:
  - all: https://git.openjdk.org/asmtools/pull/32/files
  - new: https://git.openjdk.org/asmtools/pull/32/files/0a50dc2d..c78c4b85

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=asmtools&pr=32&range=02
 - incr: https://webrevs.openjdk.org/?repo=asmtools&pr=32&range=01-02

  Stats: 25 lines in 1 file changed: 24 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/asmtools/pull/32.diff
  Fetch: git fetch https://git.openjdk.org/asmtools pull/32/head:pull/32

PR: https://git.openjdk.org/asmtools/pull/32


More information about the asmtools-dev mailing list