Skip to content

Pull - compilation changes...#2196

Merged
mpilquist merged 1 commit intotypelevel:developfrom
diesalbla:compileGo_no_attempt_or_Buildr
Dec 29, 2020
Merged

Pull - compilation changes...#2196
mpilquist merged 1 commit intotypelevel:developfrom
diesalbla:compileGo_no_attempt_or_Buildr

Conversation

@diesalbla
Copy link
Copy Markdown
Collaborator

  • WE remove the use of the BuildR subclass of Run, as well as the
    continuation-like CallRun type alias.
  • We remove the use of F.attempt after recursive calls to go,
    since errors are now also processed within the
  • For the ViewRunner, to avoid a stack overflow in one of the texts,
    we modify the method out to include a manual unrolling loop.

- WE remove the use of the `BuildR` subclass of Run, as well as the
  continuation-like `CallRun` type alias.

- We remove the use of `F.attempt` after recursive calls to `go`,
  since errors are now also processed within the

- For the ViewRunner, to avoid a stack overflow in one of the texts,
  we modify the method `out` to include a manual unrolling loop.
@mpilquist mpilquist merged commit 1ea8e42 into typelevel:develop Dec 29, 2020
@diesalbla diesalbla deleted the compileGo_no_attempt_or_Buildr branch December 29, 2020 15:45
diesalbla added a commit to diesalbla/fs2 that referenced this pull request May 14, 2021
We fix a memory leak that was introduced in a previous Pull Request
typelevel#2196
Note that some of the changes in that PR had already been reverted
to fix another memory leak.
@diesalbla diesalbla mentioned this pull request Jun 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants