2 – c_compiler-ubuntu
82%
main: 82%

Ran 08 Jun 2023 07:24AM UTC

Files 18

Run time 0s

Badge

Embed ▾

Committed 08 Jun 2023 07:18AM UTC coverage: 73.62% (+0.2%) from 73.457%

Subproject # c_compiler-ubuntu – 5208477058.2

Build Type

push

github

Committed by

web-flow

Commit Message

Handle errors from process output (#63)

Remove the `stderrCompleter` and `stdoutCompleter` variables in favor of
holding the `StreamSubscription` and using `asFuture()`. This ensures
that any errors from the stream (for instance UTF8 decodign errors) are
captured by the awaited `Future` instead of bubbling up as unhandled
async errors.

Use the `.wait` extension for records so error handlers are eagerly
added for the `exitCode` and output futures instead of not listening to
the other futures until after the exit code has completed, and errors
may have already bubbled up unhandled.

Also refactor the use of a map literal - don't use an extra `...{}`
literal, move the for-in element to the outer collection literal and use
`...?` over a conditional element with a null check. Skip the extra
intermediate variable `newEnvironment`.

Check `captureOutput` eagerly and avoid a closure in favor of a tear-off
when it is false. When the logger is null this allows an ignorable
`null` callback instead of always invoking a callback that does nothing.

Run Details

600 of 815 relevant lines covered (73.62%)

4.98 hits per line

dart-lang / native / 5208477058 / 2 – c_compiler-ubuntu
82%
main: 82%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job c_compiler-ubuntu - 5208477058.2

dart-lang / native / 5208477058 / 2 – c_compiler-ubuntu 82% main: 82%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job c_compiler-ubuntu - 5208477058.2

dart-lang / native / 5208477058 / 2 – c_compiler-ubuntu
82%
main: 82%

README BADGES
x