Misplaced (**/**) directives #222

Closed
agarwal opened this Issue Jan 18, 2012 · 1 comment

Projects

None yet

2 participants

@agarwal
ocaml-batteries-team member

Issue #221 fixes a misplaced (**/**) directive in batString.mli, but I suspect this is happening in other places too. I believe batInt32.mli and batInt64.mli are affected also. I looked through several other modules in an attempt to fix all cases, but realized I don't know several modules well enough to confidently start moving code around occurrences of (**/**).

Can someone else please grep for (**/**) and try to fix this. It might be nice to have a policy of putting all hidden elements at the very end of an mli file. But be careful, some hidden elements are at the end of a sub-module, and due to lack of indentation, this is not always so easy to see. You might accidentally be moving an element out of a sub-module, which is not good.

@c-cube c-cube added a commit to c-cube/batteries-included that referenced this issue Jan 6, 2014
@c-cube c-cube try to fix some missing (**/**) directives (#222) 3f6c83e
@c-cube
ocaml-batteries-team member

Closing unless someone finds another unmatched (/)

@c-cube c-cube closed this Jan 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment