core.async exception rewriting can result in an incorrect return value

Description

The following test case passes in core.async 0.2.395, but fails on 0.3.443 due to an incorrect return value from an exception catch. See comments in the test for details.

Environment

None

Activity

Show:
Alex Miller
January 10, 2020, 6:48 PM

Released in 0.7.559

Alex Miller
January 9, 2020, 12:47 PM

Patch applied

Kevin Downey
January 4, 2020, 12:20 AM

rebased my local branch and generated a new patch

Kevin Downey
June 11, 2019, 9:59 PM

I suspect the above failing test is the same as which has a patch which builds on this one

import
February 1, 2019, 11:27 AM

Comment made by: alesguzik

I have encountered a very similar case, and it is broken even with the patch above:

The result is

It doesn't matter whether I catch clojure.lang.ExceptionInfo or Throwable in any of those places in the test.

Fixed
Your pinned fields
Click on the next to a field label to start pinning.

Assignee

Kevin Downey

Reporter

import

Labels

Patch

Code and Test