| Anonymous | Login | Signup for a new account | 2013-05-18 11:27 CEST | ![]() |
| Main | My View | View Issues | Change Log | Roadmap |
| View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | |||||||||||
| ID | Project | Category | View Status | Date Submitted | Last Update | |||||||
| 0005718 | OCaml | OCaml typing | public | 2012-08-03 20:05 | 2012-08-03 22:32 | |||||||
| Reporter | prevosto | |||||||||||
| Assigned To | frisch | |||||||||||
| Priority | normal | Severity | tweak | Reproducibility | always | |||||||
| Status | resolved | Resolution | fixed | |||||||||
| Platform | OS | OS Version | ||||||||||
| Product Version | 4.00.0 | |||||||||||
| Target Version | Fixed in Version | |||||||||||
| Summary | 0005718: warning 37 is a bit too enthusiastic when flagging constructor of a private type as unused | |||||||||||
| Description | When compiling the file in attachment with warning 37 enabled, we get File "spurious_warning.ml", line 3, characters 7-27: Warning 37: constructor I is never used to build values. (However, this constructor appears in patterns.) But this is precisely the point of having a private type. | |||||||||||
| Steps To Reproduce | untar this attached file ocamlc -c spurious_warning.mli ocamlc -c -w+37 spurious_warning.ml | |||||||||||
| Additional Information | Note that the warning does not appear if we don't open M (and prefix inject and I with M. accordingly). | |||||||||||
| Tags | No tags attached. | |||||||||||
| Attached Files | ||||||||||||
Notes |
|
|
(0007892) frisch (developer) 2012-08-03 22:32 |
Thanks! It was already fixed in trunk (commit 12711). I've applied the same fix to 4.00 (commit 12820). |
Issue History |
|||
| Date Modified | Username | Field | Change |
| 2012-08-03 20:05 | prevosto | New Issue | |
| 2012-08-03 20:05 | prevosto | File Added: spurious_warning.tar | |
| 2012-08-03 22:32 | frisch | Note Added: 0007892 | |
| 2012-08-03 22:32 | frisch | Status | new => resolved |
| 2012-08-03 22:32 | frisch | Resolution | open => fixed |
| 2012-08-03 22:32 | frisch | Assigned To | => frisch |
| Copyright © 2000 - 2011 MantisBT Group |