Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006228OCamlOCaml generalpublic2013-11-08 00:472014-03-26 16:36
Assigned To 
PlatformOSOS Version
Product Version4.01.0 
Target VersionFixed in Version 
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>
Attached Filespatch file icon detect-cycles.patch [^] (6,684 bytes) 2013-11-08 00:47 [Show Content]

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

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

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

- 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

Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker