We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Original bug ID: 6228 Reporter: @lpw25 Assigned to: @mshinwell Status: closed (set by @xavierleroy on 2015-12-11T18:27:43Z) Resolution: fixed Priority: normal Severity: minor Version: 4.01.0 Fixed in version: 4.02.0+beta1 / +rc1 Category: ~DO NOT USE (was: OCaml general) Tags: patch Duplicate of: #3421 Monitored by: @gasche @diml @hcarty
The toplevel's value printer does not detect cycles in its output. This means that a simple recursive definition like:
let rec x = `Foo x
results in a one hundred line response.
Attached is a patch from Stephen Dolan which adds a check for cycles in the printer so that you instead get something like:
x = `Foo <cycle>
The text was updated successfully, but these errors were encountered:
Comment author: @lpw25
For slightly easier review I've added a pull request:
#23
Sorry, something went wrong.
Comment author: @mshinwell
This has been merged.
mshinwell
No branches or pull requests
Original bug ID: 6228
Reporter: @lpw25
Assigned to: @mshinwell
Status: closed (set by @xavierleroy on 2015-12-11T18:27:43Z)
Resolution: fixed
Priority: normal
Severity: minor
Version: 4.01.0
Fixed in version: 4.02.0+beta1 / +rc1
Category: ~DO NOT USE (was: OCaml general)
Tags: patch
Duplicate of: #3421
Monitored by: @gasche @diml @hcarty
Bug description
The toplevel's value printer does not detect cycles in its output. This means that a simple recursive definition like:
results in a one hundred line response.
Attached is a patch from Stephen Dolan which adds a check for cycles in the printer so that you instead get something like:
File attachments
The text was updated successfully, but these errors were encountered: