Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006228OCamlOCaml generalpublic2013-11-08 00:472014-05-30 11:23
Reporterlpw25 
Assigned Toshinwell 
PrioritynormalSeverityminorReproducibilityalways
StatusresolvedResolutionfixed 
PlatformOSOS Version
Product Version4.01.0 
Target VersionFixed in Version4.02.0+beta1 / +rc1 
Summary0006228: Toplevel printer does not detect cycles
DescriptionThe 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>
Tagspatch
Attached Filespatch file icon detect-cycles.patch [^] (6,684 bytes) 2013-11-08 00:47 [Show Content]

- Relationships
duplicate of 0001231resolvedshinwell Feature wish: better pretty-printing for cyclic values 

-  Notes
(0011106)
lpw25 (developer)
2014-03-26 16:36

For slightly easier review I've added a pull request:

https://github.com/ocaml/ocaml/pull/23 [^]
(0011583)
shinwell (developer)
2014-05-30 11:23

This has been merged.

- Issue History
Date Modified Username Field Change
2013-11-08 00:47 lpw25 New Issue
2013-11-08 00:47 lpw25 File Added: detect-cycles.patch
2013-11-08 00:47 lpw25 Tag Attached: patch
2014-02-08 13:13 yallop Relationship added duplicate of 0001231
2014-03-26 16:36 lpw25 Note Added: 0011106
2014-05-30 11:23 shinwell Note Added: 0011583
2014-05-30 11:23 shinwell Status new => resolved
2014-05-30 11:23 shinwell Fixed in Version => 4.02.0+beta1 / +rc1
2014-05-30 11:23 shinwell Resolution open => fixed
2014-05-30 11:23 shinwell Assigned To => shinwell


Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker