You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Original bug ID: 5310 Reporter: rbonichon Assigned to:@damiendoligez Status: closed (set by @damiendoligez on 2012-03-31T22:15:58Z) Resolution: fixed Priority: normal Severity: tweak Version: 3.12.0 Fixed in version: 4.00.0+dev Category: ~DO NOT USE (was: OCaml general) Related to:#4866
Bug description
In (undocumented) Ratio module:
create_ratio normalizes its ratio at create time (and possibly sets normalized to true)
create_normalized_ratio does not (it supposes it creates something normalized).
I would have expected the exact contrary to happen (normalized_ratio should normalize and create should not).
If you feel the current naming is appropriate, could you at least document the implemented behavior of these functions in the interface of ratio?
(And while you are at it, have ocamldoc generate a html file for this module too, because it is isconcerting to browse the doc of the num library and see this Ratio module appears without documentation).
Additional information
The referred behavior can be instrumented with the attached ml file.
Original bug ID: 5310
Reporter: rbonichon
Assigned to: @damiendoligez
Status: closed (set by @damiendoligez on 2012-03-31T22:15:58Z)
Resolution: fixed
Priority: normal
Severity: tweak
Version: 3.12.0
Fixed in version: 4.00.0+dev
Category: ~DO NOT USE (was: OCaml general)
Related to: #4866
Bug description
In (undocumented) Ratio module:
I would have expected the exact contrary to happen (normalized_ratio should normalize and create should not).
If you feel the current naming is appropriate, could you at least document the implemented behavior of these functions in the interface of ratio?
(And while you are at it, have ocamldoc generate a html file for this module too, because it is isconcerting to browse the doc of the num library and see this Ratio module appears without documentation).
Additional information
The referred behavior can be instrumented with the attached ml file.
File attachments
The text was updated successfully, but these errors were encountered: