Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006286OCamlstandard librarypublic2014-01-06 03:412015-02-09 17:49
Assigned To 
PrioritynormalSeverityfeatureReproducibilityhave not tried
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0006286: I think isprintf is missing from the Printf module of the stdlib
DescriptionFor the sake of completeness, several printers in the
Printf module don't have an i-equivalent
(that does not print anything but is useful to ignore some material when conditionally printing).

While working on a lazy logger, I was in need of the isprintf.
Tagsjunior_job, patch
Attached Filesdiff file icon printf.diff [^] (4,238 bytes) 2014-06-22 17:19 [Show Content]
diff file icon format.diff [^] (2,286 bytes) 2014-06-22 17:19 [Show Content]
diff file icon printf_more_tests_.diff [^] (5,425 bytes) 2014-06-23 21:21 [Show Content]
diff file icon printf_more_tests2.diff [^] (5,425 bytes) 2014-06-23 21:27 [Show Content]

- Relationships
related to 0006017closedgasche A new format implementation based on GADTs 

-  Notes
fabien (reporter)
2014-06-22 17:19

Here is a patch for the module printf and the module format.

Some remaining points:
* I didn't know what "@since version" I should put in the mli
* I didn't add i-equivalent functions for the deprecated ones in format
* I didn't add tests in tformat because the only one present was for the deprecated bprintf
* how should printf behave with malformed format ?
fabien (reporter)
2014-06-23 21:23
edited on: 2014-06-23 21:27

I've attached a new patch that contains more tests (ibprintf) (the former one can be deleted).
I did not add tests for the continuation printf because I don't know what to do.
Any suggestions would be welcome.

Edit: Please use printf_more_tests2

- Issue History
Date Modified Username Field Change
2014-01-06 03:41 berenger New Issue
2014-02-19 17:20 doligez Relationship added related to 0006017
2014-06-19 17:44 gasche Tag Attached: junior_job
2014-06-22 17:19 fabien Note Added: 0011769
2014-06-22 17:19 fabien File Added: printf.diff
2014-06-22 17:19 fabien File Added: format.diff
2014-06-23 21:21 fabien File Added: printf_more_tests_.diff
2014-06-23 21:23 fabien Note Added: 0011775
2014-06-23 21:27 fabien Note Edited: 0011775 View Revisions
2014-06-23 21:27 fabien File Added: printf_more_tests2.diff
2014-07-16 14:50 doligez Tag Attached: patch
2014-07-16 14:51 doligez Status new => acknowledged
2017-02-23 16:43 doligez Category OCaml standard library => standard library

Copyright © 2000 - 2011 MantisBT Group
Powered by Mantis Bugtracker